[DevTools] Move restartFrame and setCallFrameVariableValue to V8DebuggerAgent

third_party/WebKit/Source/platform/v8_inspector/InjectedScript.cpp

 /*
  * Copyright (C) 2012 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 151 matching lines @@
     function.appendArgument(callFrames);
     function.appendArgument(callFrameId);
     function.appendArgument(expression);
     function.appendArgument(objectGroup);
     function.appendArgument(includeCommandLineAPI);
     function.appendArgument(returnByValue);
     function.appendArgument(generatePreview);
     *result = makeEvalCall(errorString, function, wasThrown, exceptionDetails);
 }
 
-void InjectedScript::restartFrame(ErrorString* errorString, v8::Local<v8::Object> callFrames, const String16& callFrameId)
-{
-    v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(), "restartFrame");
-    function.appendArgument(callFrames);
-    function.appendArgument(callFrameId);
-    OwnPtr<protocol::Value> resultValue = makeCall(function);
-    if (resultValue) {
-        if (resultValue->type() == protocol::Value::TypeString) {
-            resultValue->asString(errorString);
-        } else {
-            bool value;
-            ASSERT_UNUSED(value, resultValue->asBoolean(&value) && value);
-        }
-        return;
-    }
-    *errorString = "Internal error";
-}
-
-void InjectedScript::setVariableValue(ErrorString* errorString,
-    v8::Local<v8::Object> callFrames,
-    const protocol::Maybe<String16>& callFrameIdOpt,
-    const protocol::Maybe<String16>&  functionObjectIdOpt,
-    int scopeNumber,
-    const String16& variableName,
-    const String16& newValueStr)
-{
-    v8::HandleScope handles(m_isolate);
-    V8FunctionCall function(m_manager->debugger(), context(), v8Value(), "setVariableValue");
-    if (callFrameIdOpt.isJust()) {
-        function.appendArgument(callFrames);
-        function.appendArgument(callFrameIdOpt.fromJust());
-    } else {
-        function.appendArgument(false);
-        function.appendArgument(false);
-    }
-    if (functionObjectIdOpt.isJust())
-        function.appendArgument(functionObjectIdOpt.fromJust());
-    else
-        function.appendArgument(false);
-    function.appendArgument(scopeNumber);
-    function.appendArgument(variableName);
-    function.appendArgument(newValueStr);
-    OwnPtr<protocol::Value> resultValue = makeCall(function);
-    if (!resultValue) {
-        *errorString = "Internal error";
-        return;
-    }
-    if (resultValue->type() == protocol::Value::TypeString) {
-        resultValue->asString(errorString);
-        return;
-    }
-    // Normal return.
-}
-
 void InjectedScript::getFunctionDetails(ErrorString* errorString, const String16& functionId, OwnPtr<FunctionDetails>* result)
 {
     v8::HandleScope handles(m_isolate);
     V8FunctionCall function(m_manager->debugger(), context(), v8Value(), "getFunctionDetails");
     function.appendArgument(functionId);
     OwnPtr<protocol::Value> resultValue = makeCall(function);
     protocol::ErrorSupport errors(errorString);
     *result = FunctionDetails::parse(resultValue.get(), &errors);
 }
 
@@ skipping 267 matching lines @@
             result = protocol::StringValue::create("Object has too long reference chain(must not be longer than " + String16::number(protocol::Value::maxDepth) + ")");
     }
     return result.release();
 }
 
 void InjectedScript::dispose()
 {
     m_manager->discardInjectedScript(m_contextId);
 }
 
+v8::MaybeLocal<v8::Value> InjectedScript::resolveCallArgument(ErrorString* errorString, protocol::Runtime::CallArgument* callArgument)
+{
+    if (callArgument->hasObjectId()) {
+        OwnPtr<RemoteObjectId> remoteObjectId = RemoteObjectId::parse(callArgument->getObjectId(""));
+        if (!remoteObjectId) {
+            *errorString = "Invalid object id in call argument";
+            return v8::MaybeLocal<v8::Value>();
+        }
+        if (remoteObjectId->contextId() != m_contextId) {
+            *errorString = "Argument should belong to the same JavaScript world as target object";
+            return v8::MaybeLocal<v8::Value>();
+        }
+        v8::Local<v8::Value> object = findObject(*remoteObjectId);
+        if (object.IsEmpty()) {
+            *errorString = "Could not find object with given id";
+            return v8::MaybeLocal<v8::Value>();
+        }
+        return object;
+    }
+    if (callArgument->hasValue()) {
+        String16 value = callArgument->getValue(nullptr)->toJSONString();
+        if (callArgument->getType(String16()) == "number")
+            value = "Number(" + value + ")";

[pfeldman, 2016/03/15 19:12:22]

What is this for?

[kozy, 2016/03/16 02:15:15]

It repeats InjectedScriptSource.js behavior for -Infinity, e.t.c.

+        v8::Local<v8::Value> object;
+        if (!m_manager->debugger()->compileAndRunInternalScript(context(), toV8String(m_isolate, value)).ToLocal(&object)) {
+            *errorString = "Couldn't parse value object in call argument";
+            return v8::MaybeLocal<v8::Value>();
+        }
+        return object;
+    }
+    return v8::Undefined(m_isolate);
+}
+
 } // namespace blink