[DevTools] Use InspectorFrontendHost.readyForTest for layout tests.

[DevTools] Use InspectorFrontendHost.readyForTest for layout tests.

Attempt #2. First one made http/tests/inspector/resource-tree/resource-tree-reload.html flaky.

BUG=597572
Committed: https://crrev.com/bdb67c0b431bf1726f0d91626c952086a4346c5c
Cr-Commit-Position: refs/heads/master@{#383238}

[dgozman, 2016-03-22 00:57:20]

dgozman@chromium.org changed reviewers:
+ pfeldman@chromium.org

[dgozman, 2016-03-22 00:57:21]

Take a look please.

[pfeldman, 2016-03-23 19:02:49]

lgtm

[dgozman, 2016-03-23 19:17:46]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-03-23 19:18:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1819243002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1819243002/20001

[commit-bot: I haz the power, 2016-03-23 20:13:39]

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

[commit-bot: I haz the power, 2016-03-23 20:13:40]

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

[dgozman, 2016-03-23 22:57:25]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-03-23 22:57:26]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/1819243002/#ps40001
(title: "test fix")

[commit-bot: I haz the power, 2016-03-23 22:57:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1819243002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1819243002/40001

[commit-bot: I haz the power, 2016-03-24 02:12:41]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-24 02:14:06]

Description was changed from

==========
[DevTools] Use InspectorFrontendHost.readyForTest for layout tests.

BUG=none
==========

to

==========
[DevTools] Use InspectorFrontendHost.readyForTest for layout tests.

BUG=none

Committed: https://crrev.com/89f7238cebbf5d77e4caa84d0a2750f6faa7e983
Cr-Commit-Position: refs/heads/master@{#383015}
==========

[commit-bot: I haz the power, 2016-03-24 02:14:07]

Patchset 3 (id:??) landed as
https://crrev.com/89f7238cebbf5d77e4caa84d0a2750f6faa7e983
Cr-Commit-Position: refs/heads/master@{#383015}

[battre, 2016-03-24 11:46:28]

battre@chromium.org changed reviewers:
+ battre@chromium.org

[battre, 2016-03-24 11:46:29]

Please create bugs for features. I am going to revert this because I have seen
two consecutive failures of
http/tests/inspector/resource-tree/resource-tree-reload.html on WebKit Win
non-Oilpan (dbg) and could imagine that this is related.

[battre, 2016-03-24 11:51:47]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/1828983002/ by battre@chromium.org.

The reason for reverting is:
http/tests/inspector/resource-tree/resource-tree-reload.html is flaky

BUG=597572.

[dgozman, 2016-03-24 17:08:34]

On 2016/03/24 11:46:29, battre wrote:
> Please create bugs for features.
This is not a feature, but an internal refactoring.

> I am going to revert this because I have seen
> two consecutive failures of
> http/tests/inspector/resource-tree/resource-tree-reload.html on WebKit Win
> non-Oilpan (dbg) and could imagine that this is related.
That's fine. Thanks for doing this!

[dgozman, 2016-03-25 01:10:26]

Description was changed from

==========
[DevTools] Use InspectorFrontendHost.readyForTest for layout tests.

BUG=none

Committed: https://crrev.com/89f7238cebbf5d77e4caa84d0a2750f6faa7e983
Cr-Commit-Position: refs/heads/master@{#383015}
==========

to

==========
[DevTools] Use InspectorFrontendHost.readyForTest for layout tests.

Attempt #2. First one made
http/tests/inspector/resource-tree/resource-tree-reload.html flaky. 

BUG=597572
==========

[dgozman, 2016-03-25 01:13:10]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-03-25 01:13:10]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/1819243002/#ps60001
(title: "resource-tree-reload.html fix")

[commit-bot: I haz the power, 2016-03-25 01:13:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1819243002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1819243002/60001

[commit-bot: I haz the power, 2016-03-25 03:32:38]

Description was changed from

==========
[DevTools] Use InspectorFrontendHost.readyForTest for layout tests.

Attempt #2. First one made
http/tests/inspector/resource-tree/resource-tree-reload.html flaky. 

BUG=597572
==========

to

==========
[DevTools] Use InspectorFrontendHost.readyForTest for layout tests.

Attempt #2. First one made
http/tests/inspector/resource-tree/resource-tree-reload.html flaky. 

BUG=597572
==========

[commit-bot: I haz the power, 2016-03-25 03:32:39]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-03-25 03:33:53]

Description was changed from

==========
[DevTools] Use InspectorFrontendHost.readyForTest for layout tests.

Attempt #2. First one made
http/tests/inspector/resource-tree/resource-tree-reload.html flaky. 

BUG=597572
==========

to

==========
[DevTools] Use InspectorFrontendHost.readyForTest for layout tests.

Attempt #2. First one made
http/tests/inspector/resource-tree/resource-tree-reload.html flaky.

BUG=597572

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

[commit-bot: I haz the power, 2016-03-25 03:33:55]

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