Replicate to all renderers the accepted languages set by a layout test.

Replicate to all renderers the accepted languages set by a layout test.

Moving |accept_languages_| from WebTestProxyBase into LayoutTestRuntimeFlags
helps to
1) replicate the value requested by a test into all renderers
2) prepare WebTestProxyBase to help with extraction of WebViewTestClient class.

BUG=587175, 595089
Committed: https://crrev.com/305f1739ea0dc08182da9d81193b859518f31bc7
Cr-Commit-Position: refs/heads/master@{#384089}

[Łukasz Anforowicz, 2016-03-24 22:08:23]

Description was changed from

==========
Replicate to all renderers the accepted languages set by a layout test.

Moving |accept_languages_| from WebTestProxyBase into LayoutTestRuntimeFlags
helps to
1) replicate the value requested by a test into all renderers
2) prepare WebTestProxyBase to help with extraction of WebViewTestClient class.

BUG=587175, 595089
==========

to

==========
Replicate to all renderers the accepted languages set by a layout test.

Moving |accept_languages_| from WebTestProxyBase into LayoutTestRuntimeFlags
helps to
1) replicate the value requested by a test into all renderers
2) prepare WebTestProxyBase to help with extraction of WebViewTestClient class.

Note that the new code doesn't compare old and new values during a chain of
SetAcceptLanguages calls, but unconditionally calls into
WebView::acceptLanguagesChanged.  This avoids a test-only logic in the code
(and also helps keep the code shorter).

BUG=587175, 595089
==========

[Łukasz Anforowicz, 2016-03-25 14:37:32]

Description was changed from

==========
Replicate to all renderers the accepted languages set by a layout test.

Moving |accept_languages_| from WebTestProxyBase into LayoutTestRuntimeFlags
helps to
1) replicate the value requested by a test into all renderers
2) prepare WebTestProxyBase to help with extraction of WebViewTestClient class.

Note that the new code doesn't compare old and new values during a chain of
SetAcceptLanguages calls, but unconditionally calls into
WebView::acceptLanguagesChanged.  This avoids a test-only logic in the code
(and also helps keep the code shorter).

BUG=587175, 595089
==========

to

==========
Replicate to all renderers the accepted languages set by a layout test.

Moving |accept_languages_| from WebTestProxyBase into LayoutTestRuntimeFlags
helps to
1) replicate the value requested by a test into all renderers
2) prepare WebTestProxyBase to help with extraction of WebViewTestClient class.

BUG=587175, 595089
==========

[Łukasz Anforowicz, 2016-03-25 14:37:58]

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

[Łukasz Anforowicz, 2016-03-25 14:37:59]

jochen@, could you please take a look?

https://codereview.chromium.org/1829253002/diff/40001/components/test_runner/...
File components/test_runner/test_runner.cc (right):

https://codereview.chromium.org/1829253002/diff/40001/components/test_runner/...
components/test_runner/test_runner.cc:2662: return;
Turns out that we really need to copy this check from WebTestProxyBase -
otherwise we get a failure in navigator_language/navigator_language.html.
Initially I thought that it is weird that a test ends up testing the code of
test harness, not what the product does, but I guess doing this check inside
WebView is difficult, because it doesn't know what the previous value was.

[jochen (gone - plz use gerrit), 2016-03-30 16:24:06]

lgtm

[Łukasz Anforowicz, 2016-03-30 18:03:40]

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

[commit-bot: I haz the power, 2016-03-30 18:04:00]

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

[commit-bot: I haz the power, 2016-03-30 18:07:25]

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

[commit-bot: I haz the power, 2016-03-30 18:07:26]

Dry run: Try jobs failed on following builders:
  ios_dbg_simulator_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_gn...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Łukasz Anforowicz, 2016-03-30 20:05:18]

The CQ bit was checked by lukasza@chromium.org

[Łukasz Anforowicz, 2016-03-30 20:05:20]

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/1829253002/#ps80001
(title: "Rebasing...")

[commit-bot: I haz the power, 2016-03-30 20:05:39]

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

[commit-bot: I haz the power, 2016-03-30 20:55:52]

Description was changed from

==========
Replicate to all renderers the accepted languages set by a layout test.

Moving |accept_languages_| from WebTestProxyBase into LayoutTestRuntimeFlags
helps to
1) replicate the value requested by a test into all renderers
2) prepare WebTestProxyBase to help with extraction of WebViewTestClient class.

BUG=587175, 595089
==========

to

==========
Replicate to all renderers the accepted languages set by a layout test.

Moving |accept_languages_| from WebTestProxyBase into LayoutTestRuntimeFlags
helps to
1) replicate the value requested by a test into all renderers
2) prepare WebTestProxyBase to help with extraction of WebViewTestClient class.

BUG=587175, 595089
==========

[commit-bot: I haz the power, 2016-03-30 20:55:54]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-03-30 20:58:07]

Description was changed from

==========
Replicate to all renderers the accepted languages set by a layout test.

Moving |accept_languages_| from WebTestProxyBase into LayoutTestRuntimeFlags
helps to
1) replicate the value requested by a test into all renderers
2) prepare WebTestProxyBase to help with extraction of WebViewTestClient class.

BUG=587175, 595089
==========

to

==========
Replicate to all renderers the accepted languages set by a layout test.

Moving |accept_languages_| from WebTestProxyBase into LayoutTestRuntimeFlags
helps to
1) replicate the value requested by a test into all renderers
2) prepare WebTestProxyBase to help with extraction of WebViewTestClient class.

BUG=587175, 595089

Committed: https://crrev.com/305f1739ea0dc08182da9d81193b859518f31bc7
Cr-Commit-Position: refs/heads/master@{#384089}
==========

[commit-bot: I haz the power, 2016-03-30 20:58:08]

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

[Łukasz Anforowicz, 2016-03-31 14:13:49]

A revert of this CL (patchset #5 id:80001) has been created in
https://codereview.chromium.org/1851483002/ by lukasza@chromium.org.

The reason for reverting is: Prevents clean reverting of an earlier CL
(https://codereview.chromium.org/1821923003/) that caused use after free related
to color chooser..