Make sure the default CSP is used for <webview> with PlzNavigate.

Make sure the default CSP is used for <webview> with PlzNavigate.

This fixes
WebViewTests/WebViewTest.Shim_TestInlineScriptFromAccessibleResources/*
with PlzNavigate.

BUG=504347
Committed: https://crrev.com/059a6b4108cd8a6ade942d70f5de95df1a33f6e5
Cr-Commit-Position: refs/heads/master@{#426300}

[jam, 2016-10-19 06:59:26]

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

[commit-bot: I haz the power, 2016-10-19 09:31:49]

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-10-19 12:14:42]

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

[commit-bot: I haz the power, 2016-10-19 12:14:43]

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

[jam, 2016-10-19 15:17:36]

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

[Fady Samuel, 2016-10-19 15:27:24]

lgtm + nit

https://codereview.chromium.org/2432153003/diff/1/extensions/browser/url_requ...
File extensions/browser/url_request_util.cc (right):

https://codereview.chromium.org/2432153003/diff/1/extensions/browser/url_requ...
extensions/browser/url_request_util.cc:137: if
(WebViewRendererState::GetInstance()->IsGuest(info->GetChildID()))
Could you please add a comment why this may not produce the expected result?
Offline comment: GetChildId() might be invalid with PlzNavigate since we don't
have a process yet when you begin a navigation request?

[jam, 2016-10-19 19:19:53]

https://codereview.chromium.org/2432153003/diff/1/extensions/browser/url_requ...
File extensions/browser/url_request_util.cc (right):

https://codereview.chromium.org/2432153003/diff/1/extensions/browser/url_requ...
extensions/browser/url_request_util.cc:137: if
(WebViewRendererState::GetInstance()->IsGuest(info->GetChildID()))
On 2016/10/19 15:27:24, Fady Samuel wrote:
> Could you please add a comment why this may not produce the expected result?
> Offline comment: GetChildId() might be invalid with PlzNavigate since we don't
> have a process yet when you begin a navigation request?

Done.

[jam, 2016-10-19 19:19:57]

The CQ bit was checked by jam@chromium.org

[jam, 2016-10-19 19:19:58]

The patchset sent to the CQ was uploaded after l-g-t-m from fsamuel@chromium.org
Link to the patchset: https://codereview.chromium.org/2432153003/#ps20001
(title: "add comment")

[commit-bot: I haz the power, 2016-10-19 19:20:26]

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

[jam, 2016-10-19 20:41:33]

The CQ bit was unchecked by jam@chromium.org

[jam, 2016-10-19 20:41:47]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2016-10-19 20:42:14]

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-10-19 21:33:35]

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

[commit-bot: I haz the power, 2016-10-19 21:33:36]

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_...)

[jam, 2016-10-19 21:37:18]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2016-10-19 21:38:05]

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-10-19 22:08:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-21 13:12:01]

Description was changed from

==========
Make sure the default CSP is used for <webview> with PlzNavigate.

This fixes
WebViewTests/WebViewTest.Shim_TestInlineScriptFromAccessibleResources/*
with PlzNavigate.

BUG=504347
==========

to

==========
Make sure the default CSP is used for <webview> with PlzNavigate.

This fixes
WebViewTests/WebViewTest.Shim_TestInlineScriptFromAccessibleResources/*
with PlzNavigate.

BUG=504347

Committed: https://crrev.com/059a6b4108cd8a6ade942d70f5de95df1a33f6e5
Cr-Commit-Position: refs/heads/master@{#426300}
==========

[commit-bot: I haz the power, 2016-10-21 13:12:01]

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