Changeset 117270 in webkit

Timestamp:

May 16, 2012 5:48:50 AM (12 years ago)

Author:

pfeldman@chromium.org

Message:

Web Inspector: follow up to r117267 - fixing typo and removing unnecessary dispatch.
​https://bugs.webkit.org/show_bug.cgi?id=86613

Reviewed by Vsevolod Vlasov.

• inspector/front-end/ResourceScriptMapping.js:

(WebInspector.ResourceScriptMapping.prototype._uiSourceCodeAdded):
(WebInspector.ResourceScriptMapping.prototype._uiSourceCodeReplaced):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/ResourceScriptMapping.js (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-05-16  Pavel Feldman  <pfeldman@chromium.org>
+
+        Web Inspector: follow up to r117267 - fixing typo and removing unnecessary dispatch.
+        https://bugs.webkit.org/show_bug.cgi?id=86613
+
+        Reviewed by Vsevolod Vlasov.
+
+        * inspector/front-end/ResourceScriptMapping.js:
+        (WebInspector.ResourceScriptMapping.prototype._uiSourceCodeAdded):
+        (WebInspector.ResourceScriptMapping.prototype._uiSourceCodeReplaced):
+
 2012-05-16  Vsevolod Vlasov  <vsevik@chromium.org>
 

trunk/Source/WebCore/inspector/front-end/ResourceScriptMapping.js

@@ -145 +145 @@
     {
         this._rawSourceCodeForUISourceCode.put(uiSourceCode, rawSourceCode);
+        this.dispatchEventToListeners(WebInspector.ScriptMapping.Events.UISourceCodeAdded, uiSourceCode);
+    },
+
+    /**
+     * @param {WebInspector.RawSourceCode} rawSourceCode
+     * @param {WebInspector.UISourceCode} oldUISourceCode
+     * @param {WebInspector.UISourceCode} uiSourceCode
+     */
+    _uiSourceCodeReplaced: function(rawSourceCode, oldUISourceCode, uiSourceCode)
+    {
+        this._rawSourceCodeForUISourceCode.remove(oldUISourceCode);
+        this._rawSourceCodeForUISourceCode.put(uiSourceCode, rawSourceCode);
 
         for (var i = 0; i < rawSourceCode._scripts.length; ++i)
             rawSourceCode._scripts[i].setSourceMapping(this);
 
-        this.dispatchEventToListeners(WebInspector.ScriptMapping.Events.UISourceCodeAdded, uiSourceCode);
-    },
-
-    /**
-     * @param {WebInspector.RawSourceCode} rawSourceCode
-     * @param {WebInspector.UISourceCode} oldUISourceCode
-     * @param {WebInspector.UISourceCode} uiSourceCode
-     */
-    _uiSourceCodeReplaced: function(rawSourceCode, oldUISourceCode, uiSourceCode)
-    {
-        this._rawSourceCodeForUISourceCode.remove(oldUISourceCode);
-        this._rawSourceCodeForUISourceCode.put(uiSourceCode, rawSourceCode);
-
-        for (var i = 0; i < rawSourceCode._scripts.length; ++i)
-            rawSourceCode._scripts[i].setSourceMapping(this);
-
-        var data = { oldUISourceCode: uiSourceCode, uiSourceCode: uiSourceCode };
+        var data = { oldUISourceCode: oldUISourceCode, uiSourceCode: uiSourceCode };
         this.dispatchEventToListeners(WebInspector.ScriptMapping.Events.UISourceCodeReplaced, data);
     },