<webview>: Add additional WebRequest API test coverage.

<webview>: Add additional WebRequest API test coverage.

This CL adds a test which verifies that we read and change HTTP headers.

BUG=352293
TBR=rockot@chromium.org

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

[Fady Samuel, 2014-10-08 16:39:50]

fsamuel@chromium.org changed reviewers:
+ lfg@chromium.org

[lfg, 2014-10-08 17:23:31]

https://codereview.chromium.org/640603005/diff/1/chrome/browser/apps/web_view...
File chrome/browser/apps/web_view_browsertest.cc (right):

https://codereview.chromium.org/640603005/diff/1/chrome/browser/apps/web_view...
chrome/browser/apps/web_view_browsertest.cc:564:
embedded_test_server()->RegisterRequestHandler(
The request handler should be defined in web_view_apitest.cc as well, to work in
app_shell_browsertests.

https://codereview.chromium.org/640603005/diff/1/extensions/test/data/web_vie...
File extensions/test/data/web_view/apitest/main.js (right):

https://codereview.chromium.org/640603005/diff/1/extensions/test/data/web_vie...
extensions/test/data/web_view/apitest/main.js:1594: webview.src =
embedder.detectUserAgentURL;
This needs to be defined on top of this file.

[Fady Samuel, 2014-10-08 19:18:50]

Wow, I was pointing WebViewAPITest to testWebRequestAPI and not
testWebRequestAPIWithHeaders! The Appshell test was completely broken! Thanks!
PTAL!

https://codereview.chromium.org/640603005/diff/1/chrome/browser/apps/web_view...
File chrome/browser/apps/web_view_browsertest.cc (right):

https://codereview.chromium.org/640603005/diff/1/chrome/browser/apps/web_view...
chrome/browser/apps/web_view_browsertest.cc:564:
embedded_test_server()->RegisterRequestHandler(
On 2014/10/08 17:23:31, lfg wrote:
> The request handler should be defined in web_view_apitest.cc as well, to work
in
> app_shell_browsertests.

Done.

https://codereview.chromium.org/640603005/diff/1/extensions/test/data/web_vie...
File extensions/test/data/web_view/apitest/main.js (right):

https://codereview.chromium.org/640603005/diff/1/extensions/test/data/web_vie...
extensions/test/data/web_view/apitest/main.js:1594: webview.src =
embedder.detectUserAgentURL;
On 2014/10/08 17:23:31, lfg wrote:
> This needs to be defined on top of this file.

Done.

[lfg, 2014-10-08 19:21:00]

lgtm

[fsamuel, 2014-10-08 19:23:46]

The CQ bit was checked by fsamuel@google.com

[commit-bot: I haz the power, 2014-10-08 19:25:43]

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

[commit-bot: I haz the power, 2014-10-08 19:40:58]

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

[commit-bot: I haz the power, 2014-10-08 19:40:59]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[fsamuel, 2014-10-08 19:41:42]

fsamuel@google.com changed reviewers:
+ rockot@chromium.org

[fsamuel, 2014-10-08 19:41:42]

fsamuel@google.com changed reviewers:
+ rockot@chromium.org

[Fady Samuel, 2014-10-08 21:48:21]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2014-10-08 21:50:44]

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

[commit-bot: I haz the power, 2014-10-08 23:10:50]

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

[commit-bot: I haz the power, 2014-10-08 23:11:01]

Try jobs failed on following builders:
  mac_gpu_triggered_tests on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu_triggered...)

[Ken Rockot(use gerrit already), 2014-10-09 00:01:53]

lgtm

[Fady Samuel, 2014-10-10 01:36:03]

The CQ bit was checked by fsamuel@chromium.org

[commit-bot: I haz the power, 2014-10-10 01:37:59]

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

[commit-bot: I haz the power, 2014-10-10 02:41:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-10 02:42:24]

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