Changeset 69019 in webkit

Timestamp:

Oct 4, 2010 10:00:50 AM (14 years ago)

Author:

podivilov@chromium.org

Message:

2010-10-04 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: do not show breakpoint in front-end if it was not set in v8
​https://bugs.webkit.org/show_bug.cgi?id=46749

• bindings/v8/DebuggerScript.js: ():

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• bindings/v8/DebuggerScript.js (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-10-04  Pavel Podivilov  <podivilov@chromium.org>
+
+        Reviewed by Pavel Feldman.
+
+        Web Inspector: do not show breakpoint in front-end if it was not set in v8
+        https://bugs.webkit.org/show_bug.cgi?id=46749
+
+        * bindings/v8/DebuggerScript.js:
+        ():
+
 2010-10-04  Alejandro G. Castro  <alex@igalia.com>
 

trunk/WebCore/bindings/v8/DebuggerScript.js

@@ -99 +99 @@
 
     var locations = Debug.findBreakPointActualLocations(breakId);
-    var actualLineNumber = locations.length ? locations[0].line : args.lineNumber;
+    if (!locations.length)
+        return undefined;
+    var actualLineNumber = locations[0].line;
 
     var key = args.scriptId + ":" + actualLineNumber;