Changeset 223948 in webkit

Timestamp:

Oct 24, 2017 10:37:47 PM (7 years ago)

Author:

commit-queue@webkit.org

Message:

REGRESSION(r222181): Web Inspector: Augmenting Agents Domains are not getting activated
​https://bugs.webkit.org/show_bug.cgi?id=178768
<rdar://problem/35081334>

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-10-24
Reviewed by Brian Burg.

• UserInterface/Base/Main.js:

Simplify event dispatch.

• UserInterface/Controllers/AppController.js:

(WI.AppController.prototype.activateExtraDomains):
Pass the domains on.

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Base/Main.js (modified) (1 diff)
• UserInterface/Controllers/AppController.js (modified) (1 diff)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2017-10-24  Joseph Pecoraro  <pecoraro@apple.com>
+
+        REGRESSION(r222181): Web Inspector: Augmenting Agents Domains are not getting activated
+        https://bugs.webkit.org/show_bug.cgi?id=178768
+        <rdar://problem/35081334>
+
+        Reviewed by Brian Burg.
+
+        * UserInterface/Base/Main.js:
+        Simplify event dispatch.
+
+        * UserInterface/Controllers/AppController.js:
+        (WI.AppController.prototype.activateExtraDomains):
+        Pass the domains on.
+
 2017-10-24  Joseph Pecoraro  <pecoraro@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Base/Main.js

@@ -696 +696 @@
 WI.activateExtraDomains = function(domains)
 {
-    this.notifications.dispatchEventToListeners(WI.Notification.ExtraDomainsActivated, {"domains": domains});
+    this.notifications.dispatchEventToListeners(WI.Notification.ExtraDomainsActivated, {domains});
 
     WI.CSSCompletions.requestCSSCompletions();

trunk/Source/WebInspectorUI/UserInterface/Controllers/AppController.js

@@ -55 +55 @@
 
         // FIXME: all code within WI.activateExtraDomains should be distributed elsewhere.
-        WI.activateExtraDomains();
+        WI.activateExtraDomains(domains);
     }
 };