Passing hit breakpoint IDs to ScriptDebugServer.

Source/bindings/v8/DebuggerScript.js

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 253 matching lines @@
     if (eventData.script().data() === "injected-script")
         return;
     eventData.script().setSource(source);
 }
 
 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.
     var func;
     try {
         func = frameMirror.func();
     } catch(e) {
     }
     var functionName;
     if (func)
@@ skipping 75 matching lines @@
         break;
     case ScopeType.Block:
         // Unsupported yet. Mustn't be reachable.
         break;
     }
     return scopeObject;
 }
 
 return DebuggerScript;
 })();