[DevTools] Removed InspectorTest.invokeFunctionInPageAsync

[DevTools] Removed InspectorTest.invokeFunctionInPageAsync

BUG=632875
R=caseq@chromium.org

Committed: https://crrev.com/a029bcf5871341046967c6e8f6d640eacb8c6134
Cr-Commit-Position: refs/heads/master@{#411813}

[kozy, 2016-08-04 00:41:02]

Dmitry, please take a look!

[kozy, 2016-08-04 00:46:06]

kozyatinskiy@chromium.org changed reviewers:
+ caseq@chromium.org

[kozy, 2016-08-04 00:46:07]

Andrey, please take a look at tracing tests.

[kozy, 2016-08-05 02:18:53]

ping?

[dgozman, 2016-08-05 02:22:37]

Description was changed from

==========
[DevTools] Removed InspectorTest.invokeFunctionInPageAsync

BUG=632875
R=dgozman@chromium.org
==========

to

==========
[DevTools] Removed InspectorTest.invokeFunctionInPageAsync

BUG=632875
R=caseq@chromium.org
==========

[dgozman, 2016-08-05 02:22:38]

dgozman@chromium.org changed reviewers:
- dgozman@chromium.org

[dgozman, 2016-08-05 02:26:14]

Harness looks good. @caseq: please take a look at tracing bits.

[caseq, 2016-08-05 17:56:43]

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js:48: return
fetch(url, requestInitializer).catch((e) => e);
e => e

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js:120:
InspectorTest.callFunctionInPageAsync("makeFetch", [ url, requestInitializer
]).then(callback);
[url, requestInitializer] (i.e. nuke spaces)

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-dispatchEvent.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-dispatchEvent.html:15:
return Promise.resolve();
just use InspectorTest.evaluateWithTimeline() below and drop promise altogether?

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-layout.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-layout.html:23:
InspectorTest.invokeAsyncWithTracing("performActions", finish);
ditto, evaluate with Timeline

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-raf.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-raf.html:13:
evaluateInFrontend("InspectorTest.testFunctionRequestAnimationFrame(" + rafId1 +
", " + rafId2 + ")");
nit: this could be re-written in a much better way now.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/elements/css-rule-hover-highlights-selectors.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/elements/css-rule-hover-highlights-selectors.html:15:
var promise = new Promise((fulfill) => callback = fulfill);
Just
return new Promise(fullfill => requestAnimationFrame(fullfill))

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/inspector/tracing-session-id.html (right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing-session-id.html:9:
InspectorTest.invokeWithTracing("(function() { return Promise.resolve(); })",
processTracingEvents);
InspectorTest.evaluateWithTimeline()

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/frame-model-instrumentation.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/frame-model-instrumentation.html:11:
var promise = new Promise((fulfill) => callback = fulfill);
return new Promise(fullfill => generateFrames(3, fullfill)) -- or change
generateFrames() to be a promise.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/inspector/tracing/scroll-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/scroll-invalidations.html:13:
testRunner.layoutAndPaintAsyncThen(callback);
if (window.testRunner)
    return new Promise(callback => testRunner.layoutAndPaintAsyncThen(callback))

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-js/timeline-gc-event.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-js/timeline-gc-event.html:13:
testRunner.layoutAndPaintAsyncThen(callback);
ditto -- only return promise here and inline this clause.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-event-causes.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-event-causes.html:22:
setTimeout(callback, 0);
inline into new Promise(...)

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-event-causes.html:52:
requestAnimationFrame(callback);
ditto.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-received-data.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-received-data.html:10:
var promise = new Promise((fulfill) => callback = fulfill);
promise => scriptLoadedPromise

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-received-data.html:13:
window._imageLoaded = true;
drop this and _window.scriptEvaluated, convert onImageLoad to
imageLoadedPromise, return Promise.all(scriptLoadedPromise, imageLoadedPromise)

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource-details.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource-details.html:19:
image.onload = onRequestFinished;
... or maybe let's introduce something like
InspectorTest.createLoadedPromise(element) and re-use it everywhere?

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource.html:11:
var callback;
drop this, assign directly to window.timelineNetworkResourceEvaluated

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/layer-tree.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/layer-tree.html:21:
generateFrames(3, callback);
Yeah, let's make generateFrames() return promise.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-layout-invalidations-on-deleted-node.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-layout-invalidations-on-deleted-node.html:10:
var promise = new Promise((fulfill) => callback = fulfill);
Let's rewrite this and a few following ones to just do their work immediately
(i.e. drop rAF, then return generateFrames(1) which would be a promise then)

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-style-recalc-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-style-recalc-invalidations.html:17:
return promise;
return generateFrames(1) instead.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-style-recalc-invalidations.html:29:
return promise;
ditto.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/update-layer-tree.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/update-layer-tree.html:24:
testRunner.layoutAndPaintAsyncThen(callback);
inline into new Promise(...)

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/parse-author-style-sheet.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/parse-author-style-sheet.html:17:
link.onload = callback.bind(this, null);
use createLoadPromise()

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:18:
return promise;
replace with generateFrames()

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:32:
return promise;
ditto.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:46:
return promise;
ditto

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:60:
return promise;
ditto.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:74:
return promise;
ditto.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:91:
return promise;
ditto.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidations.html:18:
return promise;
replace with generateFrames

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidations.html:43:
return promise;
ditto

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidations.html:57:
return promise;
ditto

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidator-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidator-invalidations.html:18:
return promise;
replace with generateFrames

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidator-invalidations.html:41:
return promise;
ditto

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidator-invalidations.html:65:
return promise;
ditto

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-time/timeline-timer-fired-from-eval-call-site.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-time/timeline-timer-fired-from-eval-call-site.html:11:
window.callWhenDone = callback;
assign directly within new Promise(...)

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/inspector/tracing/worker-events.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/worker-events.html:39: return
promise;
return new Promise(fulfill => worker2.onmessage = fulfill)

[kozy, 2016-08-10 01:20:49]

The CQ bit was checked by kozyatinskiy@chromium.org to run a CQ dry run

[kozy, 2016-08-10 01:21:15]

All done! Please take another look!

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js:48: return
fetch(url, requestInitializer).catch((e) => e);
On 2016/08/05 17:56:41, caseq wrote:
> e => e

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/inspector/network-test.js:120:
InspectorTest.callFunctionInPageAsync("makeFetch", [ url, requestInitializer
]).then(callback);
On 2016/08/05 17:56:41, caseq wrote:
> [url, requestInitializer] (i.e. nuke spaces)

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-dispatchEvent.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-dispatchEvent.html:15:
return Promise.resolve();
On 2016/08/05 17:56:41, caseq wrote:
> just use InspectorTest.evaluateWithTimeline() below and drop promise
altogether?

Acknowledged.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-layout.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-layout.html:23:
InspectorTest.invokeAsyncWithTracing("performActions", finish);
On 2016/08/05 17:56:42, caseq wrote:
> ditto, evaluate with Timeline

Acknowledged.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-raf.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector-protocol/timeline/timeline-raf.html:13:
evaluateInFrontend("InspectorTest.testFunctionRequestAnimationFrame(" + rafId1 +
", " + rafId2 + ")");
On 2016/08/05 17:56:42, caseq wrote:
> nit: this could be re-written in a much better way now.

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/elements/css-rule-hover-highlights-selectors.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/elements/css-rule-hover-highlights-selectors.html:15:
var promise = new Promise((fulfill) => callback = fulfill);
On 2016/08/05 17:56:42, caseq wrote:
> Just
> return new Promise(fullfill => requestAnimationFrame(fullfill))

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/inspector/tracing-session-id.html (right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing-session-id.html:9:
InspectorTest.invokeWithTracing("(function() { return Promise.resolve(); })",
processTracingEvents);
On 2016/08/05 17:56:42, caseq wrote:
> InspectorTest.evaluateWithTimeline()

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/frame-model-instrumentation.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/frame-model-instrumentation.html:11:
var promise = new Promise((fulfill) => callback = fulfill);
On 2016/08/05 17:56:42, caseq wrote:
> return new Promise(fullfill => generateFrames(3, fullfill)) -- or change
> generateFrames() to be a promise.

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/inspector/tracing/scroll-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/scroll-invalidations.html:13:
testRunner.layoutAndPaintAsyncThen(callback);
On 2016/08/05 17:56:42, caseq wrote:
> if (window.testRunner)
>     return new Promise(callback =>
testRunner.layoutAndPaintAsyncThen(callback))

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-js/timeline-gc-event.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-js/timeline-gc-event.html:13:
testRunner.layoutAndPaintAsyncThen(callback);
On 2016/08/05 17:56:42, caseq wrote:
> ditto -- only return promise here and inline this clause.

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-event-causes.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-event-causes.html:22:
setTimeout(callback, 0);
On 2016/08/05 17:56:42, caseq wrote:
> inline into new Promise(...)

We need to have good closure around setTimeout for checkStringContains.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-event-causes.html:52:
requestAnimationFrame(callback);
On 2016/08/05 17:56:42, caseq wrote:
> ditto.

Acknowledged.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-received-data.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-received-data.html:10:
var promise = new Promise((fulfill) => callback = fulfill);
On 2016/08/05 17:56:42, caseq wrote:
> promise => scriptLoadedPromise

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-received-data.html:13:
window._imageLoaded = true;
On 2016/08/05 17:56:42, caseq wrote:
> drop this and _window.scriptEvaluated, convert onImageLoad to
> imageLoadedPromise, return Promise.all(scriptLoadedPromise,
imageLoadedPromise)

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource-details.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource-details.html:19:
image.onload = onRequestFinished;
On 2016/08/05 17:56:42, caseq wrote:
> ... or maybe let's introduce something like
> InspectorTest.createLoadedPromise(element) and re-use it everywhere?

Acknowledged.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-network/timeline-network-resource.html:11:
var callback;
On 2016/08/05 17:56:42, caseq wrote:
> drop this, assign directly to window.timelineNetworkResourceEvaluated

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/layer-tree.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/layer-tree.html:21:
generateFrames(3, callback);
On 2016/08/05 17:56:42, caseq wrote:
> Yeah, let's make generateFrames() return promise.

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-layout-invalidations-on-deleted-node.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-layout-invalidations-on-deleted-node.html:10:
var promise = new Promise((fulfill) => callback = fulfill);
On 2016/08/05 17:56:42, caseq wrote:
> Let's rewrite this and a few following ones to just do their work immediately
> (i.e. drop rAF, then return generateFrames(1) which would be a promise then)

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-style-recalc-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-style-recalc-invalidations.html:17:
return promise;
On 2016/08/05 17:56:42, caseq wrote:
> return generateFrames(1) instead.

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-style-recalc-invalidations.html:29:
return promise;
On 2016/08/05 17:56:42, caseq wrote:
> ditto.

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/update-layer-tree.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/update-layer-tree.html:24:
testRunner.layoutAndPaintAsyncThen(callback);
On 2016/08/05 17:56:42, caseq wrote:
> inline into new Promise(...)

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/parse-author-style-sheet.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/parse-author-style-sheet.html:17:
link.onload = callback.bind(this, null);
On 2016/08/05 17:56:42, caseq wrote:
> use createLoadPromise()

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:18:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> replace with generateFrames()

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:32:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> ditto.

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:46:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> ditto

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:60:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> ditto.

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:74:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> ditto.

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:91:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> ditto.

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidations.html:18:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> replace with generateFrames

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidations.html:43:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> ditto

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidations.html:57:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> ditto

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidator-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidator-invalidations.html:18:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> replace with generateFrames

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidator-invalidations.html:41:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> ditto

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-with-invalidator-invalidations.html:65:
return promise;
On 2016/08/05 17:56:43, caseq wrote:
> ditto

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-time/timeline-timer-fired-from-eval-call-site.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-time/timeline-timer-fired-from-eval-call-site.html:11:
window.callWhenDone = callback;
On 2016/08/05 17:56:43, caseq wrote:
> assign directly within new Promise(...)

Done.

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/inspector/tracing/worker-events.html
(right):

https://codereview.chromium.org/2208963002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/inspector/tracing/worker-events.html:39: return
promise;
On 2016/08/05 17:56:43, caseq wrote:
> return new Promise(fulfill => worker2.onmessage = fulfill)

Done.

[commit-bot: I haz the power, 2016-08-10 01:21:29]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-10 02:35:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-10 02:35:53]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[kozy, 2016-08-10 03:07:44]

The CQ bit was checked by kozyatinskiy@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-10 03:08:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-10 04:37:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-10 04:37:50]

Dry run: This issue passed the CQ dry run.

[caseq, 2016-08-10 22:44:55]

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/http/tests/inspector/timeline-test.js
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/inspector/timeline-test.js:470:
.then(() => waitForFrame());
merge into above?

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/inspector/timeline-test.js:479: if
(window.testRunner)
inline into the Promise() constructor?

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/http/tests/inspector/tracing/websocket/timeline-websocket-event.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/inspector/tracing/websocket/timeline-websocket-event.html:12:
ws.onclose = callback;
return new Promise(fulfill => ws.onclose = fulfill)

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/inspector/tracing/decode-resize.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/decode-resize.html:45: .then(()
=> addImage(image))
drop () =>

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/decode-resize.html:46: .then(()
=> new Promise((fulfill) => testRunner.layoutAndPaintAsyncThen(fulfill)));
ditto

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-layout/timeline-layout-with-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-layout/timeline-layout-with-invalidations.html:11:
requestAnimationFrame(function() {
do we still need these actions to be done under rAF()?

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-event-causes.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-event-causes.html:22:
setTimeout(callback, 0);
return new Promise(fullfil => setTimeout(fullfil, 0))

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-and-multiple-style-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-and-multiple-style-invalidations.html:11:
requestAnimationFrame(function() {
remove rAF() and use waitForFrame()?

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-layout-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-layout-invalidations.html:11:
requestAnimationFrame(function() {
remove rAF() and use waitForFrame()?

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint.html:11:
requestAnimationFrame(function() {
remove rAF and use waitForFrame() instead?

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint.html:23:
requestAnimationFrame(function() {
ditto.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:53:
var forceStyleRecalc = document.body.offsetTop;
I think this one if wrong -- we should't force recalc after waitForFrame() (or
we should have another one after that)

[kozy, 2016-08-11 01:34:17]

Please take another look!

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/http/tests/inspector/timeline-test.js
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/inspector/timeline-test.js:470:
.then(() => waitForFrame());
On 2016/08/10 22:44:55, caseq wrote:
> merge into above?

Done.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/inspector/timeline-test.js:479: if
(window.testRunner)
On 2016/08/10 22:44:55, caseq wrote:
> inline into the Promise() constructor?

Acknowledged.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/http/tests/inspector/tracing/websocket/timeline-websocket-event.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/http/tests/inspector/tracing/websocket/timeline-websocket-event.html:12:
ws.onclose = callback;
On 2016/08/10 22:44:55, caseq wrote:
> return new Promise(fulfill => ws.onclose = fulfill)

Done.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/inspector/tracing/decode-resize.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/decode-resize.html:45: .then(()
=> addImage(image))
On 2016/08/10 22:44:55, caseq wrote:
> drop () =>

I'd like to upload image and then run testRunner.layoutAndPaintAsyncThen, load
next image and run testRunner.. Repeat original behavior, if I remove arrow
function here then images will be loaded in parallel.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/decode-resize.html:46: .then(()
=> new Promise((fulfill) => testRunner.layoutAndPaintAsyncThen(fulfill)));
On 2016/08/10 22:44:55, caseq wrote:
> ditto

Acknowledged.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-layout/timeline-layout-with-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-layout/timeline-layout-with-invalidations.html:11:
requestAnimationFrame(function() {
On 2016/08/10 22:44:55, caseq wrote:
> do we still need these actions to be done under rAF()?

Done.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-event-causes.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-event-causes.html:22:
setTimeout(callback, 0);
On 2016/08/10 22:44:55, caseq wrote:
> return new Promise(fullfil => setTimeout(fullfil, 0))

Done.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-and-multiple-style-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-and-multiple-style-invalidations.html:11:
requestAnimationFrame(function() {
On 2016/08/10 22:44:55, caseq wrote:
> remove rAF() and use waitForFrame()?

Done.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-layout-invalidations.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint-with-layout-invalidations.html:11:
requestAnimationFrame(function() {
On 2016/08/10 22:44:55, caseq wrote:
> remove rAF() and use waitForFrame()?

Done.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint.html:11:
requestAnimationFrame(function() {
On 2016/08/10 22:44:55, caseq wrote:
> remove rAF and use waitForFrame() instead?

Done.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/timeline-paint.html:23:
requestAnimationFrame(function() {
On 2016/08/10 22:44:55, caseq wrote:
> ditto.

Done.

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html
(right):

https://codereview.chromium.org/2208963002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/inspector/tracing/timeline-style/timeline-style-recalc-all-invalidator-types.html:53:
var forceStyleRecalc = document.body.offsetTop;
On 2016/08/10 22:44:55, caseq wrote:
> I think this one if wrong -- we should't force recalc after waitForFrame() (or
> we should have another one after that)

Done.

[kozy, 2016-08-11 01:34:37]

The CQ bit was checked by kozyatinskiy@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-11 01:35:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-11 02:54:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-11 02:54:52]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[kozy, 2016-08-11 15:29:53]

The CQ bit was checked by kozyatinskiy@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-11 15:30:17]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-11 17:57:19]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-11 17:57:21]

Dry run: This issue passed the CQ dry run.

[kozy, 2016-08-12 18:50:15]

friendly ping!

[caseq, 2016-08-12 21:43:40]

lgtm and thanks a lot for simplifying the tests!

[kozy, 2016-08-12 21:46:00]

The CQ bit was checked by kozyatinskiy@chromium.org

[commit-bot: I haz the power, 2016-08-12 21:46:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-12 23:24:55]

Description was changed from

==========
[DevTools] Removed InspectorTest.invokeFunctionInPageAsync

BUG=632875
R=caseq@chromium.org
==========

to

==========
[DevTools] Removed InspectorTest.invokeFunctionInPageAsync

BUG=632875
R=caseq@chromium.org
==========

[commit-bot: I haz the power, 2016-08-12 23:24:57]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-08-12 23:30:27]

Description was changed from

==========
[DevTools] Removed InspectorTest.invokeFunctionInPageAsync

BUG=632875
R=caseq@chromium.org
==========

to

==========
[DevTools] Removed InspectorTest.invokeFunctionInPageAsync

BUG=632875
R=caseq@chromium.org

Committed: https://crrev.com/a029bcf5871341046967c6e8f6d640eacb8c6134
Cr-Commit-Position: refs/heads/master@{#411813}
==========

[commit-bot: I haz the power, 2016-08-12 23:30:28]

Patchset 5 (id:??) landed as
https://crrev.com/a029bcf5871341046967c6e8f6d640eacb8c6134
Cr-Commit-Position: refs/heads/master@{#411813}