[DevTools] Removed InspectorTest.invokePageFunctionPromise

third_party/WebKit/LayoutTests/http/tests/inspector/inspector-test.js

 if (window.GCController)
     GCController.collectAll();
 var initialize_InspectorTest = function() {
 
 var results = [];
 
 function consoleOutputHook(messageType)
 {
     InspectorTest.addResult(messageType + ": " + Array.prototype.slice.call(arguments, 1));
 }
@@ skipping 38 matching lines @@
             callback(InspectorTest.runtimeModel.createRemoteObject(result), wasThrown);
     }
     InspectorTest.RuntimeAgent.evaluate(code, "console", false, mycallback);
 }
 
 InspectorTest.evaluateInPagePromise = function(code)
 {
     return new Promise(succ => InspectorTest.evaluateInPage(code, succ));
 }
 
+InspectorTest.evaluateInPageAsync = function(code)
+{
+    var callback;
+    var promise = new Promise((fulfill) => callback = fulfill);
+    InspectorTest.RuntimeAgent.evaluate(code,
+        "console",
+        /* includeCommandLineAPI */ false,
+        /* doNotPauseOnExceptionsAndMuteConsole */ undefined,
+        /* contextId */ undefined,
+        /* returnByValue */ undefined,
+        /* generatePreview */ undefined,
+        /* userGesture */ undefined,
+        /* awaitPromise */ true,
+        mycallback);
+
+    function mycallback(error, result, wasThrown)
+    {
+        if (!error)

[dgozman, 2016/08/03 22:46:58]

Let's log something and completeTest otherwise, or we'll get timeouts without
any results. And do the same in evaluateInPage.

[kozy, 2016/08/03 23:48:39]

Done.

+            callback(InspectorTest.runtimeModel.createRemoteObject(result), wasThrown);

[dgozman, 2016/08/03 22:46:58]

There is no way to pass two arguments to a promise callback. Let's handle
wasThrown as an error.

[kozy, 2016/08/03 23:48:39]

Done.

+    }
+    return promise;
+}
+
+InspectorTest.callFunctionInPageAsync = function(name, args)
+{
+    args = args || [];
+    return InspectorTest.evaluateInPageAsync(name + "(" + args.map(JSON.stringify).join(",") + ")");
+}
+
 InspectorTest.evaluateInPageWithTimeout = function(code)
 {
     // FIXME: we need a better way of waiting for chromium events to happen
     InspectorTest.evaluateInPage("setTimeout(unescape('" + escape(code) + "'), 1)");
 }
 
 InspectorTest.evaluateFunctionInOverlay = function(func, callback)
 {
     var expression = "testRunner.evaluateInWebInspectorOverlay(\"(\" + " + func + " + \")()\")";
     var mainContext = InspectorTest.runtimeModel.executionContexts()[0];
     mainContext.evaluate(expression, "", false, false, true, false, false, wrapCallback);
 
     function wrapCallback(val, err, result)
     {
         callback(result.value)
     }
 }
 
 InspectorTest.waitForOverlayRepaint = function(callback)
 {
     InspectorTest.invokePageFunctionAsync("requestAnimationFrame", callback);
 }
 
 var lastEvalId = 0;
 var pendingEvalRequests = {};
 
-var lastPromiseEvalId = 0;
-var pendingPromiseEvalRequests = {};
-
-/**
- * The given function should take two callback paraters before the arguments:
- *  * resolve - called when successful (with optional result)
- *  * reject  - called when there was a failure (with optional error)
- */
-InspectorTest.invokePageFunctionPromise = function(functionName, parameters)
-{
-    return new Promise(function(resolve, reject) {
-        var id = ++lastPromiseEvalId;
-        pendingPromiseEvalRequests[id] = { resolve: InspectorTest.safeWrap(resolve), reject: InspectorTest.safeWrap(reject) };
-
-        var jsonParameters = [];
-        for (var i = 0; i < parameters.length; ++i)
-            jsonParameters.push(JSON.stringify(parameters[i]));
-        var asyncEvalWrapper = function(callId, functionName, argumentsArray)
-        {
-            function evalCallbackResolve(result)
-            {
-                testRunner.evaluateInWebInspector(evalCallbackCallId, "InspectorTest.didInvokePageFunctionPromise(" + callId + ", " + JSON.stringify(result) + ", true);");
-            }
-
-            function evalCallbackReject(result)
-            {
-                testRunner.evaluateInWebInspector(evalCallbackCallId, "InspectorTest.didInvokePageFunctionPromise(" + callId + ", " + JSON.stringify(result) + ", false);");
-            }
-
-            var args = [evalCallbackResolve, evalCallbackReject].concat(argumentsArray.map(JSON.stringify));
-            var functionCall = functionName + ".call(null, " + args.join(", ") + ")";
-            try {
-                eval(functionCall);
-            } catch(e) {
-                InspectorTest.addResult("Error: " + e);
-                evalCallbackReject(e);
-            }
-        }
-        var pageRequest = "(" + asyncEvalWrapper.toString() + ")(" + id + ", unescape('" + escape(functionName) + "'), [" + jsonParameters.join(", ") + "])";
-        InspectorTest.evaluateInPage(pageRequest);
-    });
-}
-
-
-InspectorTest.didInvokePageFunctionPromise = function(callId, value, didResolve)
-{
-    var callbacks = pendingPromiseEvalRequests[callId];
-    if (!callbacks) {
-        InspectorTest.addResult("Missing callback for async eval " + callId + ", perhaps callback invoked twice?");
-        return;
-    }
-    var callback = didResolve ? callbacks.resolve : callbacks.reject;
-    delete pendingPromiseEvalRequests[callId];
-    callback(value);
-}
-
 /**
  * @param {string} functionName
  * @param {...} varArgs
  * @param {function()} callback
  */
 InspectorTest.invokePageFunctionAsync = function(functionName, varArgs)
 {
     var id = ++lastEvalId;
     var args = Array.prototype.slice.call(arguments, 1);
     var callback = args.pop();
@@ skipping 1067 matching lines @@
     _output("[page] " + text);
 }
 
 function _output(result)
 {
     if (!outputElement)
         createOutputElement();
     outputElement.appendChild(document.createTextNode(result));
     outputElement.appendChild(document.createElement("br"));
 }