Fix <webview> subframe resource loads with PlzNavigate.

Fix <webview> subframe resource loads with PlzNavigate.

Also fix NavigationRequest not handling blocked requests.

This fixes
WebViewTests/WebViewTest.Shim_TestNestedSubframes/*
WebViewTests/WebViewTest.Shim_TestWebViewInsideFrame/*
with PlzNavigate.

BUG=504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/7e588d6b310e78e90280d46ad74f6cddcdb58f09
Cr-Commit-Position: refs/heads/master@{#426819}

[jam, 2016-10-20 22:23:33]

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

[commit-bot: I haz the power, 2016-10-20 22:24:07]

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

[jam, 2016-10-20 22:25:00]

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

[jam, 2016-10-20 22:25:01]

ExtensionNavigationThrottle is added for subframe navigations as well as main
frame.

[Fady Samuel, 2016-10-20 22:36:41]

lgtm

[jam, 2016-10-20 22:37:41]

The CQ bit was unchecked by jam@chromium.org

[jam, 2016-10-20 22:37:47]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2016-10-20 22:38:04]

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

[jam, 2016-10-21 00:08:14]

The CQ bit was unchecked by jam@chromium.org

[jam, 2016-10-21 06:47:09]

Description was changed from

==========
Fix <webview> subframe resource loads with PlzNavigate.

This fixes
WebViewTests/WebViewTest.Shim_TestNestedSubframes/*
WebViewTests/WebViewTest.Shim_TestWebViewInsideFrame/*
with PlzNavigate.

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

to

==========
Fix <webview> subframe resource loads with PlzNavigate.

This fixes
WebViewTests/WebViewTest.Shim_TestNestedSubframes/*
WebViewTests/WebViewTest.Shim_TestWebViewInsideFrame/*
with PlzNavigate.

BUG=504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[jam, 2016-10-21 06:47:14]

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

[commit-bot: I haz the power, 2016-10-21 06:47:27]

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-21 08:00:04]

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

[commit-bot: I haz the power, 2016-10-21 08:00:05]

Dry run: This issue passed the CQ dry run.

[jam, 2016-10-21 15:05:42]

Description was changed from

==========
Fix <webview> subframe resource loads with PlzNavigate.

This fixes
WebViewTests/WebViewTest.Shim_TestNestedSubframes/*
WebViewTests/WebViewTest.Shim_TestWebViewInsideFrame/*
with PlzNavigate.

BUG=504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Fix <webview> subframe resource loads with PlzNavigate.

Also fix NavigationRequest not handling blocked requests.

This fixes
WebViewTests/WebViewTest.Shim_TestNestedSubframes/*
WebViewTests/WebViewTest.Shim_TestWebViewInsideFrame/*
with PlzNavigate.

BUG=504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[jam, 2016-10-21 15:05:51]

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

[jam, 2016-10-21 15:05:51]

Nasko: ptal at patchset 1-> 2

[nasko, 2016-10-21 16:23:44]

LGTM

[jam, 2016-10-21 16:37:49]

The CQ bit was checked by jam@chromium.org

[jam, 2016-10-21 16:37:50]

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/2441843002/#ps20001
(title: "fix NavigationRequest to handle blocking, and fix unit tests")

[commit-bot: I haz the power, 2016-10-21 16:38:01]

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-21 16:56:35]

Description was changed from

==========
Fix <webview> subframe resource loads with PlzNavigate.

Also fix NavigationRequest not handling blocked requests.

This fixes
WebViewTests/WebViewTest.Shim_TestNestedSubframes/*
WebViewTests/WebViewTest.Shim_TestWebViewInsideFrame/*
with PlzNavigate.

BUG=504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Fix <webview> subframe resource loads with PlzNavigate.

Also fix NavigationRequest not handling blocked requests.

This fixes
WebViewTests/WebViewTest.Shim_TestNestedSubframes/*
WebViewTests/WebViewTest.Shim_TestWebViewInsideFrame/*
with PlzNavigate.

BUG=504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[commit-bot: I haz the power, 2016-10-21 16:56:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-21 17:05:58]

Description was changed from

==========
Fix <webview> subframe resource loads with PlzNavigate.

Also fix NavigationRequest not handling blocked requests.

This fixes
WebViewTests/WebViewTest.Shim_TestNestedSubframes/*
WebViewTests/WebViewTest.Shim_TestWebViewInsideFrame/*
with PlzNavigate.

BUG=504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Fix <webview> subframe resource loads with PlzNavigate.

Also fix NavigationRequest not handling blocked requests.

This fixes
WebViewTests/WebViewTest.Shim_TestNestedSubframes/*
WebViewTests/WebViewTest.Shim_TestWebViewInsideFrame/*
with PlzNavigate.

BUG=504347
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

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

[commit-bot: I haz the power, 2016-10-21 17:05:59]

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