Fix race in SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe.

Fix race in SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe.

Ensure server-supplied content for the 404 error page. This prevents showing
Chrome's built in error page, which would trigger a background request to
load navigation corrections (aka. Link Doctor), and once the results are back,
would make a navigation to a second error page containing the suggestions,
which, unfortunately, makes WaitForLoadStop() in NavigateToURL() racey.

BUG=677819
Committed: https://crrev.com/5fb9af565efc05fe461fb27117a5419a6f04de69
Cr-Commit-Position: refs/heads/master@{#441116}

[engedy, 2017-01-03 13:32:20]

Description was changed from

==========
Fix race condition in
SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe.

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

to

==========
Fix race in SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe.

Ensure server-supplied content for the 404 error page, to prevent showing
Chrome's built in error page, which would then trigger background request to
load navigation corrections (aka. Link Doctor), and once the results are back,
make a navigation to a second error page with the suggestions, which makes
WaitForLoadStop() in NavigateToURL() racey.

BUG=677819
==========

[engedy, 2017-01-03 13:32:20]

engedy@chromium.org changed reviewers:
+ melandory@chromium.org

[engedy, 2017-01-03 13:34:26]

Description was changed from

==========
Fix race in SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe.

Ensure server-supplied content for the 404 error page, to prevent showing
Chrome's built in error page, which would then trigger background request to
load navigation corrections (aka. Link Doctor), and once the results are back,
make a navigation to a second error page with the suggestions, which makes
WaitForLoadStop() in NavigateToURL() racey.

BUG=677819
==========

to

==========
Fix race in SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe.

Ensure server-supplied content for the 404 error page. This prevents showing 
Chrome's built in error page, which would trigger a background request to
load navigation corrections (aka. Link Doctor), and once the results are back,
would make a navigation to a second error page containing the suggestions,
which, unfortunately, makes WaitForLoadStop() in NavigateToURL() racey.

BUG=677819
==========

[engedy, 2017-01-03 13:36:07]

Patchset #1 (id:1) has been deleted

[engedy, 2017-01-03 13:36:28]

Tanja, PTAL.

[melandory, 2017-01-03 13:38:01]

On 2017/01/03 13:36:28, engedy wrote:
> Tanja, PTAL.

LGTM

[engedy, 2017-01-03 13:38:27]

The CQ bit was checked by engedy@chromium.org

[commit-bot: I haz the power, 2017-01-03 13:38:33]

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-01-03 14:14:48]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483450707766930,
"parent_rev": "c254e15859f20ac46b197885d24c3fc7dfdc82a2", "commit_rev":
"b69854ed5695aac269eeffba8bec4dbb968a5e64"}

[commit-bot: I haz the power, 2017-01-03 14:15:18]

Description was changed from

==========
Fix race in SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe.

Ensure server-supplied content for the 404 error page. This prevents showing 
Chrome's built in error page, which would trigger a background request to
load navigation corrections (aka. Link Doctor), and once the results are back,
would make a navigation to a second error page containing the suggestions,
which, unfortunately, makes WaitForLoadStop() in NavigateToURL() racey.

BUG=677819
==========

to

==========
Fix race in SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe.

Ensure server-supplied content for the 404 error page. This prevents showing 
Chrome's built in error page, which would trigger a background request to
load navigation corrections (aka. Link Doctor), and once the results are back,
would make a navigation to a second error page containing the suggestions,
which, unfortunately, makes WaitForLoadStop() in NavigateToURL() racey.

BUG=677819

Review-Url: https://codereview.chromium.org/2609163003
==========

[commit-bot: I haz the power, 2017-01-03 14:15:18]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2017-01-03 14:18:28]

Description was changed from

==========
Fix race in SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe.

Ensure server-supplied content for the 404 error page. This prevents showing 
Chrome's built in error page, which would trigger a background request to
load navigation corrections (aka. Link Doctor), and once the results are back,
would make a navigation to a second error page containing the suggestions,
which, unfortunately, makes WaitForLoadStop() in NavigateToURL() racey.

BUG=677819

Review-Url: https://codereview.chromium.org/2609163003
==========

to

==========
Fix race in SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe.

Ensure server-supplied content for the 404 error page. This prevents showing
Chrome's built in error page, which would trigger a background request to
load navigation corrections (aka. Link Doctor), and once the results are back,
would make a navigation to a second error page containing the suggestions,
which, unfortunately, makes WaitForLoadStop() in NavigateToURL() racey.

BUG=677819

Committed: https://crrev.com/5fb9af565efc05fe461fb27117a5419a6f04de69
Cr-Commit-Position: refs/heads/master@{#441116}
==========

[commit-bot: I haz the power, 2017-01-03 14:18:29]

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