Passing hit breakpoint IDs to ScriptDebugServer.

Source/bindings/v8/DebuggerScript.js

Index: Source/bindings/v8/DebuggerScript.js
diff --git a/Source/bindings/v8/DebuggerScript.js b/Source/bindings/v8/DebuggerScript.js
index 2eb43c61a9b47a2d933abea30a45f0513a790349..7861bc9a26aeb335e15333f157a6bda49f928e14 100644
--- a/Source/bindings/v8/DebuggerScript.js
+++ b/Source/bindings/v8/DebuggerScript.js
@@ -271,6 +271,20 @@ DebuggerScript.getScriptName = function(eventData)
     return eventData.script().script_.nameOrSourceURL();
 }
 
+DebuggerScript.getBreakpointNumbers = function(eventData)

[yurys, 2013/05/16 19:52:53]

getBreakpointsIds would be a better name

[SeRya, 2013/05/30 07:45:57]

I think the name is right. V8 uses term "number". InspectorDebuggerAgent and
external interface of ScriptDebugServer use "id". I think terminology and types
should be changed at the same place (where type change from v8::Number to
WTF::String).

+{
+    var breakpoints = eventData.breakPointsHit();
+    var numbers = [];

[yurys, 2013/05/16 19:52:53]

ids

[SeRya, 2013/05/30 07:45:57]

The same.

+    if (breakpoints) {
+        for (var i = 0; i < breakpoints.length; i++) {
+            var b = breakpoints[i];

[yurys, 2013/05/16 19:52:53]

b -> breakpoint we normally don't use abbreviations in blink code.

+            var scriptBreakPoint = b.script_break_point();
+            numbers.push(scriptBreakPoint ? scriptBreakPoint.number() : b.number());
+        }
+    }
+    return numbers;
+}
+
 DebuggerScript._frameMirrorToJSCallFrame = function(frameMirror, callerFrame)
 {
     // Get function name.