Title: [200649] trunk/Source/WebInspectorUI
Revision
200649
Author
[email protected]
Date
2016-05-10 16:18:12 -0700 (Tue, 10 May 2016)

Log Message

Web Inspector: Debugger sidebar should refresh when Debug UI enabled/disabled
https://bugs.webkit.org/show_bug.cgi?id=157540
<rdar://problem/26207064>

Reviewed by Timothy Hatcher.

* UserInterface/Views/DebuggerSidebarPanel.js:
(WebInspector.DebuggerSidebarPanel.prototype._scriptRemoved.removeScript):
(WebInspector.DebuggerSidebarPanel.prototype._scriptRemoved):
Remove element from both tree outlines, if it exists.

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (200648 => 200649)


--- trunk/Source/WebInspectorUI/ChangeLog	2016-05-10 23:11:52 UTC (rev 200648)
+++ trunk/Source/WebInspectorUI/ChangeLog	2016-05-10 23:18:12 UTC (rev 200649)
@@ -1,5 +1,18 @@
 2016-05-10  Matt Baker  <[email protected]>
 
+        Web Inspector: Debugger sidebar should refresh when Debug UI enabled/disabled
+        https://bugs.webkit.org/show_bug.cgi?id=157540
+        <rdar://problem/26207064>
+
+        Reviewed by Timothy Hatcher.
+
+        * UserInterface/Views/DebuggerSidebarPanel.js:
+        (WebInspector.DebuggerSidebarPanel.prototype._scriptRemoved.removeScript):
+        (WebInspector.DebuggerSidebarPanel.prototype._scriptRemoved):
+        Remove element from both tree outlines, if it exists.
+
+2016-05-10  Matt Baker  <[email protected]>
+
         Web Inspector: TimelineOverview assertion failed: Missing overview graph for timeline type undefined
         https://bugs.webkit.org/show_bug.cgi?id=157533
         <rdar://problem/26204033>

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js (200648 => 200649)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js	2016-05-10 23:11:52 UTC (rev 200648)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js	2016-05-10 23:18:12 UTC (rev 200649)
@@ -484,12 +484,16 @@
 
     _scriptRemoved(event)
     {
+        function removeScript(script, treeOutline)
+        {
+            let scriptTreeElement = treeOutline.getCachedTreeElement(script);
+            if (scriptTreeElement)
+                scriptTreeElement.parent.removeChild(scriptTreeElement);
+        }
+
         let script = event.data.script;
-        let scriptTreeElement = this._breakpointsContentTreeOutline.getCachedTreeElement(script);
-        if (!scriptTreeElement)
-            return;
-
-        scriptTreeElement.parent.removeChild(scriptTreeElement);
+        removeScript(script, this._breakpointsContentTreeOutline);
+        removeScript(script, this._scriptsContentTreeOutline);
     }
 
     _scriptsCleared(event)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to