Web Inspector: Uncaught Exception: Unable to get object details - GlobalObject
authordrousso@apple.com <drousso@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Nov 2019 19:46:41 +0000 (19:46 +0000)
committerdrousso@apple.com <drousso@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 7 Nov 2019 19:46:41 +0000 (19:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=203930
<rdar://problem/56960937>

In r248286, calls to `Heap.snapshot`, `Heap.getPreview`, and `Heap.getRemoteObject` were
moved to the `WI.HeapManager` instead of being inside view code. During this move, extra
`WI.reportInternalError` calls were added, which wasn't entirely correct as it is fully
possible (and expected) that any of these commands are invoked with old heap snapshot nodes
that may no longer exist. As such, calling any error an "internal error" isn't strictly
correct, and instead should just be a log. Also, the frontend code that calls these commands
is able to handle errors, again because it is expected.

Reviewed by Brian Burg.

* UserInterface/Controllers/HeapManager.js:
(WI.HeapManager.prototype.snapshot):
(WI.HeapManager.prototype.getPreview):
(WI.HeapManager.prototype.getRemoteObject):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@252197 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Controllers/HeapManager.js

index c2cecb1..2f859e3 100644 (file)
@@ -1,3 +1,24 @@
+2019-11-07  Devin Rousso  <drousso@apple.com>
+
+        Web Inspector: Uncaught Exception: Unable to get object details - GlobalObject
+        https://bugs.webkit.org/show_bug.cgi?id=203930
+        <rdar://problem/56960937>
+
+        In r248286, calls to `Heap.snapshot`, `Heap.getPreview`, and `Heap.getRemoteObject` were
+        moved to the `WI.HeapManager` instead of being inside view code. During this move, extra
+        `WI.reportInternalError` calls were added, which wasn't entirely correct as it is fully
+        possible (and expected) that any of these commands are invoked with old heap snapshot nodes
+        that may no longer exist. As such, calling any error an "internal error" isn't strictly
+        correct, and instead should just be a log. Also, the frontend code that calls these commands
+        is able to handle errors, again because it is expected.
+
+        Reviewed by Brian Burg.
+
+        * UserInterface/Controllers/HeapManager.js:
+        (WI.HeapManager.prototype.snapshot):
+        (WI.HeapManager.prototype.getPreview):
+        (WI.HeapManager.prototype.getRemoteObject):
+
 2019-11-06  Nikita Vasilyev  <nvasilyev@apple.com>
 
         Web Inspector: Display color picker for p3 colors
index ca0e351..00d7ecc 100644 (file)
@@ -90,7 +90,7 @@ WI.HeapManager = class HeapManager extends WI.Object
         let target = WI.assumingMainTarget();
         target.HeapAgent.snapshot((error, timestamp, snapshotStringData) => {
             if (error)
-                WI.reportInternalError(error);
+                console.error(error);
             callback(error, timestamp, snapshotStringData);
         });
     }
@@ -103,7 +103,7 @@ WI.HeapManager = class HeapManager extends WI.Object
         let target = WI.assumingMainTarget();
         target.HeapAgent.getPreview(node.id, (error, string, functionDetails, preview) => {
             if (error)
-                WI.reportInternalError(error);
+                console.error(error);
             callback(error, string, functionDetails, preview);
         });
     }
@@ -116,7 +116,7 @@ WI.HeapManager = class HeapManager extends WI.Object
         let target = WI.assumingMainTarget();
         target.HeapAgent.getRemoteObject(node.id, objectGroup, (error, result) => {
             if (error)
-                WI.reportInternalError(error);
+                console.error(error);
             callback(error, result);
         });
     }