(reland) Use stricter checking in UrlOverridingTest

(reland) Use stricter checking in UrlOverridingTest

Check the callback counts in all the test cases, and wait for the
fallback navigation to avoid early abortion.

The failures with PlzNavigate are not fixed yet.

BUG=699388
Review-Url: https://codereview.chromium.org/2770193004
Cr-Commit-Position: refs/heads/master@{#464316}
Committed: https://chromium.googlesource.com/chromium/src/+/1e31d646391e0b4a897025d83a4d7ba5fdbbc175

[wychen, 2017-03-24 23:32:31]

Patchset #2 (id:20001) has been deleted

[wychen, 2017-04-12 01:10:13]

Description was changed from

==========
Use stricter checking in UrlOverridingTest

Check the callback counts in all the test cases, and wait for the
fallback navigation to avoid early abortion.

The failures with PlzNavigate are not fixed yet.

BUG=699388
==========

to

==========
(reland) Use stricter checking in UrlOverridingTest

Check the callback counts in all the test cases, and wait for the
fallback navigation to avoid early abortion.

The failures with PlzNavigate are not fixed yet.

BUG=699388
==========

[wychen, 2017-04-12 01:10:13]

wychen@chromium.org changed reviewers:
+ mariakhomenko@chromium.org

[wychen, 2017-04-12 01:10:29]

PTAL

[Maria, 2017-04-12 17:16:41]

lgtm

[wychen, 2017-04-13 05:19:28]

The CQ bit was checked by wychen@chromium.org

[wychen, 2017-04-13 05:19:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
mariakhomenko@chromium.org
Link to the patchset: https://codereview.chromium.org/2770193004/#ps60001
(title: "fix merging, better error msg")

[commit-bot: I haz the power, 2017-04-13 05:19:59]

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-04-13 06:18:09]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1492060768381850,
"parent_rev": "fca615ce366c3d28d9a0a48522313a3710306be1", "commit_rev":
"1e31d646391e0b4a897025d83a4d7ba5fdbbc175"}

[commit-bot: I haz the power, 2017-04-13 06:19:03]

Description was changed from

==========
(reland) Use stricter checking in UrlOverridingTest

Check the callback counts in all the test cases, and wait for the
fallback navigation to avoid early abortion.

The failures with PlzNavigate are not fixed yet.

BUG=699388
==========

to

==========
(reland) Use stricter checking in UrlOverridingTest

Check the callback counts in all the test cases, and wait for the
fallback navigation to avoid early abortion.

The failures with PlzNavigate are not fixed yet.

BUG=699388

Review-Url: https://codereview.chromium.org/2770193004
Cr-Commit-Position: refs/heads/master@{#464316}
Committed:
https://chromium.googlesource.com/chromium/src/+/1e31d646391e0b4a897025d83a4d...
==========

[commit-bot: I haz the power, 2017-04-13 06:19:04]

Committed patchset #3 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/1e31d646391e0b4a897025d83a4d...