Fix stopped navigations not having the correct error code in their navigation handle.

Fix stopped navigations not having the correct error code in their navigation handle.

This fixes
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
ClientOnReceivedError2Test#testNotCalledOnStopLoading
ClientOnReceivedError2Test#testNotCalledOnStopLoading with {--webview-sandboxed-renderer}
with PlzNavigate.

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

Review-Url: https://codereview.chromium.org/2742853002
Cr-Commit-Position: refs/heads/master@{#455930}
Committed: https://chromium.googlesource.com/chromium/src/+/0299edae682a3dd0c5f4ffd3fddde60b124695db

[jam, 2017-03-09 21:46:29]

Description was changed from

==========
the # Enter a description of the change.
Fix ClientOnPageFinishedTest failures with PlzNavigate.

BUG=
==========

to

==========
the # Enter a description of the change.
Fix ClientOnPageFinishedTest failures with PlzNavigate.

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

[jam, 2017-03-09 21:46:46]

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

[commit-bot: I haz the power, 2017-03-09 21:47:14]

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

[jam, 2017-03-09 21:49:05]

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

[commit-bot: I haz the power, 2017-03-09 21:49:21]

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

[jam, 2017-03-09 22:01:10]

Description was changed from

==========
the # Enter a description of the change.
Fix ClientOnPageFinishedTest failures with PlzNavigate.

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

to

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_

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

[jam, 2017-03-09 22:13:33]

Description was changed from

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_

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

to

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect
org.chromium.android_webview.test.AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect_with__--webview-sandboxed-renderer_
org.chromium.android_webview.test.AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect
org.chromium.android_webview.test.AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect_with__--webview-sandboxed-renderer_

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

[jam, 2017-03-09 22:14:48]

Description was changed from

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect
org.chromium.android_webview.test.AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect_with__--webview-sandboxed-renderer_
org.chromium.android_webview.test.AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect
org.chromium.android_webview.test.AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect_with__--webview-sandboxed-renderer_

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

to

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect_with__--webview-sandboxed-renderer_
AwSettingsTest#testAppCacheWithTwoViews
AwSettingsTest#testAppCacheWithTwoViews with {--webview-sandboxed-renderer}

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

[jam, 2017-03-09 22:14:59]

Description was changed from

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect_with__--webview-sandboxed-renderer_
AwSettingsTest#testAppCacheWithTwoViews
AwSettingsTest#testAppCacheWithTwoViews with {--webview-sandboxed-renderer}

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

to

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect_with__--webview-sandboxed-renderer_
AwSettingsTest#testAppCacheWithTwoViews
AwSettingsTest#testAppCacheWithTwoViews with {--webview-sandboxed-renderer}
PolicyUrlFilteringTest#testWhitelistedUrl

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

[jam, 2017-03-09 22:17:03]

Description was changed from

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect_with__--webview-sandboxed-renderer_
AwSettingsTest#testAppCacheWithTwoViews
AwSettingsTest#testAppCacheWithTwoViews with {--webview-sandboxed-renderer}
PolicyUrlFilteringTest#testWhitelistedUrl

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

to

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect_with__--webview-sandboxed-renderer_
AwSettingsTest#testAppCacheWithTwoViews
AwSettingsTest#testAppCacheWithTwoViews with {--webview-sandboxed-renderer}
PolicyUrlFilteringTest#testWhitelistedUrl
AwSettingsTest#testUserAgentWithTestServer
AwSettingsTest#testUserAgentWithTestServer with {--webview-sandboxed-renderer}
AwSettingsTest#testUpdatingUserAgentWhileLoadingCausesReload
AwSettingsTest#testUpdatingUserAgentWhileLoadingCausesReload with
{--webview-sandboxed-renderer}



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

[jam, 2017-03-09 22:23:47]

Description was changed from

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect_with__--webview-sandboxed-renderer_
AwSettingsTest#testAppCacheWithTwoViews
AwSettingsTest#testAppCacheWithTwoViews with {--webview-sandboxed-renderer}
PolicyUrlFilteringTest#testWhitelistedUrl
AwSettingsTest#testUserAgentWithTestServer
AwSettingsTest#testUserAgentWithTestServer with {--webview-sandboxed-renderer}
AwSettingsTest#testUpdatingUserAgentWhileLoadingCausesReload
AwSettingsTest#testUpdatingUserAgentWhileLoadingCausesReload with
{--webview-sandboxed-renderer}



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

to

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect_with__--webview-sandboxed-renderer_
AwSettingsTest#testAppCacheWithTwoViews
AwSettingsTest#testAppCacheWithTwoViews with {--webview-sandboxed-renderer}
PolicyUrlFilteringTest#testWhitelistedUrl
AwSettingsTest#testUserAgentWithTestServer
AwSettingsTest#testUserAgentWithTestServer with {--webview-sandboxed-renderer}
AwSettingsTest#testUpdatingUserAgentWhileLoadingCausesReload
AwSettingsTest#testUpdatingUserAgentWhileLoadingCausesReload with
{--webview-sandboxed-renderer}
ClientOnReceivedError2Test#testNotCalledOnStopLoading
ClientOnReceivedError2Test#testNotCalledOnStopLoading with
{--webview-sandboxed-renderer}


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

[jam, 2017-03-09 22:31:34]

Description was changed from

==========
Fix ClientOnPageFinishedTest failures with PlzNavigate.

ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden
ClientOnPageFinishedTest_testCalledAfterRedirectedUrlIsOverridden_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect
AwContentsClientShouldOverrideUrlLoadingTest_testCalledOn302Redirect_with__--webview-sandboxed-renderer_
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect
AwContentsClientShouldOverrideUrlLoadingTest_testNullContentsClientWithServerRedirect_with__--webview-sandboxed-renderer_
AwSettingsTest#testAppCacheWithTwoViews
AwSettingsTest#testAppCacheWithTwoViews with {--webview-sandboxed-renderer}
PolicyUrlFilteringTest#testWhitelistedUrl
AwSettingsTest#testUserAgentWithTestServer
AwSettingsTest#testUserAgentWithTestServer with {--webview-sandboxed-renderer}
AwSettingsTest#testUpdatingUserAgentWhileLoadingCausesReload
AwSettingsTest#testUpdatingUserAgentWhileLoadingCausesReload with
{--webview-sandboxed-renderer}
ClientOnReceivedError2Test#testNotCalledOnStopLoading
ClientOnReceivedError2Test#testNotCalledOnStopLoading with
{--webview-sandboxed-renderer}


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

to

==========
Fix stopped navigations not having the correct error code in their navigation
handle.

This fixes
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
ClientOnReceivedError2Test#testNotCalledOnStopLoading
ClientOnReceivedError2Test#testNotCalledOnStopLoading with
{--webview-sandboxed-renderer}
with PlzNavigate.

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

[jam, 2017-03-09 22:33:11]

Description was changed from

==========
Fix stopped navigations not having the correct error code in their navigation
handle.

This fixes
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
ClientOnReceivedError2Test#testNotCalledOnStopLoading
ClientOnReceivedError2Test#testNotCalledOnStopLoading with
{--webview-sandboxed-renderer}
with PlzNavigate.

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

to

==========
Fix stopped navigations not having the correct error code in their navigation
handle.

This fixes
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
ClientOnReceivedError2Test#testNotCalledOnStopLoading
ClientOnReceivedError2Test#testNotCalledOnStopLoading with
{--webview-sandboxed-renderer}
with PlzNavigate.

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

[jam, 2017-03-09 22:33:12]

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

[jam, 2017-03-09 22:35:08]

Description was changed from

==========
Fix stopped navigations not having the correct error code in their navigation
handle.

This fixes
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
ClientOnReceivedError2Test#testNotCalledOnStopLoading
ClientOnReceivedError2Test#testNotCalledOnStopLoading with
{--webview-sandboxed-renderer}
with PlzNavigate.

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

to

==========
Fix stopped navigations not having the correct error code in their navigation
handle.

This fixes
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
ClientOnReceivedError2Test#testNotCalledOnStopLoading
ClientOnReceivedError2Test#testNotCalledOnStopLoading with
{--webview-sandboxed-renderer}
with PlzNavigate.

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

[commit-bot: I haz the power, 2017-03-09 22:39:45]

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

[commit-bot: I haz the power, 2017-03-09 22:39:46]

Dry run: This issue passed the CQ dry run.

[nasko, 2017-03-10 00:27:03]

LGTM

[jam, 2017-03-10 00:42:28]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2017-03-10 00:43:09]

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, 2017-03-10 00:49:33]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489106548529380,
"parent_rev": "258758c687d31cc18e1f9f0395a8714df0187202", "commit_rev":
"0299edae682a3dd0c5f4ffd3fddde60b124695db"}

[commit-bot: I haz the power, 2017-03-10 00:50:16]

Description was changed from

==========
Fix stopped navigations not having the correct error code in their navigation
handle.

This fixes
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
ClientOnReceivedError2Test#testNotCalledOnStopLoading
ClientOnReceivedError2Test#testNotCalledOnStopLoading with
{--webview-sandboxed-renderer}
with PlzNavigate.

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

to

==========
Fix stopped navigations not having the correct error code in their navigation
handle.

This fixes
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad
ClientOnPageFinishedTest_testCalledOnCancelingProvisionalLoad_with__--webview-sandboxed-renderer_
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi
ClientOnPageFinishedTest_testNotCalledOnDomModificationWithJavascriptUrlAfterNonCommittedLoadFromApi_with__--webview-sandboxed-renderer_
ClientOnReceivedError2Test#testNotCalledOnStopLoading
ClientOnReceivedError2Test#testNotCalledOnStopLoading with
{--webview-sandboxed-renderer}
with PlzNavigate.

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

Review-Url: https://codereview.chromium.org/2742853002
Cr-Commit-Position: refs/heads/master@{#455930}
Committed:
https://chromium.googlesource.com/chromium/src/+/0299edae682a3dd0c5f4ffd3fddd...
==========

[commit-bot: I haz the power, 2017-03-10 00:50:16]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/0299edae682a3dd0c5f4ffd3fddd...

[jam, 2017-03-10 01:12:57]

Patchset #3 (id:40001) has been deleted