monitor console command implemented.

Source/bindings/v8/custom/V8InjectedScriptHostCustom.cpp

 /*
  * Copyright (C) 2007-2011 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 345 matching lines @@
     v8::Handle<v8::Value> functionValue = args[0];
     int scopeIndex = args[1]->Int32Value();
     String variableName = toWebCoreStringWithUndefinedOrNullCheck(args[2]);
     v8::Handle<v8::Value> newValue = args[3];
 
     InjectedScriptHost* host = V8InjectedScriptHost::toNative(args.Holder());
     ScriptDebugServer& debugServer = host->scriptDebugServer();
     v8SetReturnValue(args, debugServer.setFunctionVariableValue(functionValue, scopeIndex, variableName, newValue));
 }
 
-static bool getFunctionLocation(const v8::FunctionCallbackInfo<v8::Value>& args, String* scriptId, int* lineNumber, int* columnNumber)
-{
-    if (args.Length() < 1)
-        return false;
-    v8::Handle<v8::Value> fn = args[0];
-    if (!fn->IsFunction())
-        return false;
-    v8::HandleScope handleScope;
-    v8::Handle<v8::Function> function = v8::Handle<v8::Function>::Cast(fn);
-    *lineNumber = function->GetScriptLineNumber();
-    *columnNumber = function->GetScriptColumnNumber();
-    if (*lineNumber == v8::Function::kLineOffsetNotFound || *columnNumber == v8::Function::kLineOffsetNotFound)
-        return false;
-    *scriptId = toWebCoreStringWithUndefinedOrNullCheck(function->GetScriptId());
-    return true;
-}
-
 void V8InjectedScriptHost::setBreakpointMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& args)
 {
-    String scriptId;
-    int lineNumber;
-    int columnNumber;
-    if (!getFunctionLocation(args, &scriptId, &lineNumber, &columnNumber))
+    if (args.Length() < 5) {

[vsevik, 2013/06/11 09:18:32]

Can we avoid having custom bindings at all?

+        v8::ThrowException(v8::Exception::Error(v8::String::New("5 argument expected.")));
         return;
+    }
+    if (!args[0]->IsString()) {
+        v8::ThrowException(v8::Exception::Error(v8::String::New("scriptId must be a string.")));
+        return;
+    }
+
+    String scriptId = toWebCoreStringWithUndefinedOrNullCheck(args[0]->ToString());
+    int lineNumber = args[1]->Int32Value();
+    int columnNumber = args[2]->Int32Value();
+    String source = toWebCoreStringWithUndefinedOrNullCheck(args[3]->ToString());
+    String condition = toWebCoreStringWithUndefinedOrNullCheck(args[4]->ToString());
 
     InjectedScriptHost* host = V8InjectedScriptHost::toNative(args.Holder());
-    host->setBreakpoint(scriptId, lineNumber, columnNumber);
+    host->setBreakpoint(scriptId, lineNumber, columnNumber, source, condition);
 }
 
 void V8InjectedScriptHost::removeBreakpointMethodCustom(const v8::FunctionCallbackInfo<v8::Value>& args)
 {
-    String scriptId;
-    int lineNumber;
-    int columnNumber;
-    if (!getFunctionLocation(args, &scriptId, &lineNumber, &columnNumber))
+    if (args.Length() < 4) {
+        v8::ThrowException(v8::Exception::Error(v8::String::New("4 argument expected.")));
         return;
+    }
+
+    String scriptId = toWebCoreStringWithUndefinedOrNullCheck(args[0]->ToString());
+    int lineNumber = args[1]->Int32Value();
+    int columnNumber = args[2]->Int32Value();
+    String source = toWebCoreStringWithUndefinedOrNullCheck(args[3]->ToString());
 
     InjectedScriptHost* host = V8InjectedScriptHost::toNative(args.Holder());
-    host->removeBreakpoint(scriptId, lineNumber, columnNumber);
+    host->removeBreakpoint(scriptId, lineNumber, columnNumber, source);
 }
 
-
 } // namespace WebCore