Inspector: support simple objects for Runtime.callFunctionOn arguments

This patch adds objects support for Runtime.callFunctionOn arguments.

R=kozy

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I9e9ad000482aa556f10a632b89c2f91fdc21ff1e
Reviewed-on: https://chromium-review.googlesource.com/636353Reviewed-by: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#47624}

src/inspector/injected-script.cc

@@ -564,7 +564,7 @@ Response InjectedScript::resolveCallArgument(
   if (callArgument->hasValue() || callArgument->hasUnserializableValue()) {
     String16 value =
         callArgument->hasValue()
-            ? callArgument->getValue(nullptr)->serialize()
+            ? "(" + callArgument->getValue(nullptr)->serialize() + ")"
             : "Number(\"" + callArgument->getUnserializableValue("") + "\")";
     if (!m_context->inspector()
              ->compileAndRunInternalScript(

src/inspector/js_protocol.json

@@ -140,7 +140,7 @@
                 "type": "object",
                 "description": "Represents function call argument. Either remote object id <code>objectId</code>, primitive <code>value</code>, unserializable primitive value or neither of (for undefined) them should be specified.",
                 "properties": [
-                    { "name": "value", "type": "any", "optional": true, "description": "Primitive value." },
+                    { "name": "value", "type": "any", "optional": true, "description": "Primitive value or serializable javascript object." },
                     { "name": "unserializableValue", "$ref": "UnserializableValue", "optional": true, "description": "Primitive value which can not be JSON-stringified." },
                     { "name": "objectId", "$ref": "RemoteObjectId", "optional": true, "description": "Remote object handle." }
                 ]

test/inspector/runtime/call-function-on-async-expected.txt

@@ -13,6 +13,17 @@ Running test: testArguments
     }
 }
 
+Running test: testComplexArguments
+{
+    id : <messageId>
+    result : {
+        result : {
+            type : string
+            value : bar
+        }
+    }
+}
+
 Running test: testSyntaxErrorInFunction
 {
     id : <messageId>

test/inspector/runtime/call-function-on-async.js

@@ -27,6 +27,17 @@ InspectorTest.runAsyncTestSuite([
     }));
   },
 
+  async function testComplexArguments() {
+    InspectorTest.logMessage(await callFunctionOn({
+      objectId: remoteObject1.objectId,
+      functionDeclaration: 'function(arg) { return arg.foo; }',
+      arguments: prepareArguments([{foo: 'bar'}]),
+      returnByValue: true,
+      generatePreview: false,
+      awaitPromise: false
+    }));
+  },
+
   async function testSyntaxErrorInFunction() {
     InspectorTest.logMessage(await callFunctionOn({
       objectId: remoteObject1.objectId,