Title: [125881] trunk/Source/WebCore
Revision
125881
Author
[email protected]
Date
2012-08-17 05:06:27 -0700 (Fri, 17 Aug 2012)

Log Message

Web Inspector: decouple extension server from the Elements panel
https://bugs.webkit.org/show_bug.cgi?id=94322

Reviewed by Pavel Feldman.

- relay WebInspector.ElementsTreeOutline.Events.SelectedNodeChanged received in ElementsPanel on WebInspector.notifications;

* inspector/front-end/ElementsPanel.js:
(WebInspector.ElementsPanel.prototype._selectedNodeChanged):
* inspector/front-end/ExtensionServer.js:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (125880 => 125881)


--- trunk/Source/WebCore/ChangeLog	2012-08-17 11:42:08 UTC (rev 125880)
+++ trunk/Source/WebCore/ChangeLog	2012-08-17 12:06:27 UTC (rev 125881)
@@ -1,3 +1,16 @@
+2012-08-17  Andrey Kosyakov  <[email protected]>
+
+        Web Inspector: decouple extension server from the Elements panel
+        https://bugs.webkit.org/show_bug.cgi?id=94322
+
+        Reviewed by Pavel Feldman.
+
+        - relay WebInspector.ElementsTreeOutline.Events.SelectedNodeChanged received in ElementsPanel on WebInspector.notifications;
+
+        * inspector/front-end/ElementsPanel.js:
+        (WebInspector.ElementsPanel.prototype._selectedNodeChanged):
+        * inspector/front-end/ExtensionServer.js:
+
 2012-08-17  Jan Keromnes  <[email protected]>
 
         Web Inspector: SourceFrame shouldn't be a View

Modified: trunk/Source/WebCore/inspector/front-end/ElementsPanel.js (125880 => 125881)


--- trunk/Source/WebCore/inspector/front-end/ElementsPanel.js	2012-08-17 11:42:08 UTC (rev 125880)
+++ trunk/Source/WebCore/inspector/front-end/ElementsPanel.js	2012-08-17 12:06:27 UTC (rev 125881)
@@ -206,6 +206,7 @@
             ConsoleAgent.addInspectedNode(selectedNode.id);
             this._lastValidSelectedNode = selectedNode;
         }
+        WebInspector.notifications.dispatchEventToListeners(WebInspector.ElementsTreeOutline.Events.SelectedNodeChanged);
     },
 
     _updateSidebars: function()

Modified: trunk/Source/WebCore/inspector/front-end/ExtensionServer.js (125880 => 125881)


--- trunk/Source/WebCore/inspector/front-end/ExtensionServer.js	2012-08-17 11:42:08 UTC (rev 125880)
+++ trunk/Source/WebCore/inspector/front-end/ExtensionServer.js	2012-08-17 12:06:27 UTC (rev 125881)
@@ -574,17 +574,10 @@
             WebInspector.workspace,
             WebInspector.UISourceCodeProvider.Events.UISourceCodeAdded,
             this._notifyResourceAdded);
-        this._registerSubscriptionHandler(WebInspector.extensionAPI.Events.ElementsPanelObjectSelected,
-            function()
-            {
-                WebInspector.panel("elements").treeOutline.addEventListener(WebInspector.ElementsTreeOutline.Events.SelectedNodeChanged, this._notifyElementsSelectionChanged, this);
-            }.bind(this),
-            function()
-            {
-                WebInspector.panel("elements").treeOutline.removeEventListener(WebInspector.ElementsTreeOutline.Events.SelectedNodeChanged, this._notifyElementsSelectionChanged, this);
-            }.bind(this)
-        );
-
+        this._registerAutosubscriptionHandler(WebInspector.extensionAPI.Events.ElementsPanelObjectSelected,
+            WebInspector.notifications,
+            WebInspector.ElementsTreeOutline.Events.SelectedNodeChanged,
+            this._notifyElementsSelectionChanged);
         this._registerAutosubscriptionHandler(WebInspector.extensionAPI.Events.ResourceContentCommitted,
             WebInspector.workspace,
             WebInspector.Workspace.Events.UISourceCodeContentCommitted,
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to