Move DumpBackForwardLists out of WebTestProxyBase.

Move DumpBackForwardLists out of WebTestProxyBase.

Iterating over all known test windows can be moved out of
web_test_proxy_base.cc and into blink_test_runner.cc.  This helps
minimize the amount of code in WebTestProxyBase (which should hopefully
be just a passive holder of WebView, WebViewTestClient, etc. - all logic
should be implemented inside WebViewTestClient and [future]
WebWidgetTestClient).

BUG=595089
Committed: https://crrev.com/7ab191b81769178cbbe438736b0144783ca40b49
Cr-Commit-Position: refs/heads/master@{#387465}

[Łukasz Anforowicz, 2016-04-14 00:55:28]

lukasza@chromium.org changed reviewers:
+ jochen@chromium.org

[Łukasz Anforowicz, 2016-04-14 00:55:29]

jochen@, could you please take a look?

[jochen (gone - plz use gerrit), 2016-04-14 13:27:20]

lgtm

[Łukasz Anforowicz, 2016-04-14 21:15:43]

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

[commit-bot: I haz the power, 2016-04-14 21:16:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1886103002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1886103002/80001

[commit-bot: I haz the power, 2016-04-14 22:29:06]

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

[commit-bot: I haz the power, 2016-04-14 22:29:07]

Dry run: This issue passed the CQ dry run.

[Łukasz Anforowicz, 2016-04-14 22:37:34]

The CQ bit was checked by lukasza@chromium.org

[Łukasz Anforowicz, 2016-04-14 22:37:34]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/1886103002/#ps80001
(title: "Rebasing...")

[commit-bot: I haz the power, 2016-04-14 22:37:56]

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

[commit-bot: I haz the power, 2016-04-14 22:43:34]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-04-14 22:44:36]

Description was changed from

==========
Move DumpBackForwardLists out of WebTestProxyBase.

Iterating over all known test windows can be moved out of
web_test_proxy_base.cc and into blink_test_runner.cc.  This helps
minimize the amount of code in WebTestProxyBase (which should hopefully
be just a passive holder of WebView, WebViewTestClient, etc. - all logic
should be implemented inside WebViewTestClient and [future]
WebWidgetTestClient).

BUG=595089
==========

to

==========
Move DumpBackForwardLists out of WebTestProxyBase.

Iterating over all known test windows can be moved out of
web_test_proxy_base.cc and into blink_test_runner.cc.  This helps
minimize the amount of code in WebTestProxyBase (which should hopefully
be just a passive holder of WebView, WebViewTestClient, etc. - all logic
should be implemented inside WebViewTestClient and [future]
WebWidgetTestClient).

BUG=595089

Committed: https://crrev.com/7ab191b81769178cbbe438736b0144783ca40b49
Cr-Commit-Position: refs/heads/master@{#387465}
==========

[commit-bot: I haz the power, 2016-04-14 22:44:37]

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