Changeset 50880 in webkit

Timestamp:

Nov 12, 2009 9:00:22 AM (14 years ago)

Author:

eric@webkit.org

Message:

2009-11-12 Patrick Mueller <​Patrick_Mueller@us.ibm.com>

Reviewed by Timothy Hatcher.

Web Inspector: breakpoints in named evals are not restored after a reload
​https://bugs.webkit.org/show_bug.cgi?id=31375

Manual test added

• inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.addScript):
• manual-tests/inspector/bp-in-named-eval-after-reload.html: Added.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/ScriptsPanel.js (modified) (2 diffs)
• manual-tests/inspector/bp-in-named-eval-after-reload.html (added)

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-11-12  Patrick Mueller  <Patrick_Mueller@us.ibm.com>
+
+        Reviewed by Timothy Hatcher.
+
+        Web Inspector: breakpoints in named evals are not restored after a reload
+        https://bugs.webkit.org/show_bug.cgi?id=31375
+
+        Manual test added
+
+        * inspector/front-end/ScriptsPanel.js:
+        (WebInspector.ScriptsPanel.prototype.addScript):
+        * manual-tests/inspector/bp-in-named-eval-after-reload.html: Added.
+
 2009-11-12  Benjamin Poulain  <benjamin.poulain@nokia.com>
 

trunk/WebCore/inspector/front-end/ScriptsPanel.js

@@ -277 +277 @@
         }
 
+        sourceURL = script.sourceURL;
+
+        if (sourceID)
+            this._sourceIDMap[sourceID] = (resource || script);
+
         if (sourceURL in this._breakpointsURLMap && sourceID) {
             var breakpoints = this._breakpointsURLMap[sourceURL];
@@ -294 +299 @@
             }
         }
-
-        if (sourceID)
-            this._sourceIDMap[sourceID] = (resource || script);
 
         this._addScriptToFilesMenu(script);