Prevent racey favicon loads in subresource filter browsertests.

Prevent racey favicon loads in subresource filter browsertests.

The favicon load was at race with the page load event, which caused flakiness
in tests that verified, at page load completion time, the number of evaluated
loads recorded into histograms.

BUG=677370
Committed: https://crrev.com/311f7c0a14700bd54a1ee50c3f530334579ea99d
Cr-Commit-Position: refs/heads/master@{#440956}

[engedy, 2016-12-29 13:42:54]

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

[commit-bot: I haz the power, 2016-12-29 13:43:04]

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

[engedy, 2016-12-29 13:43:31]

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

[engedy, 2016-12-29 13:43:31]

Fixing some flakiness. Tanja, could you please take a look?

[melandory, 2016-12-29 13:47:19]

On 2016/12/29 13:43:31, engedy wrote:
> Fixing some flakiness. Tanja, could you please take a look?

lgtm

[engedy, 2016-12-29 13:48:10]

The CQ bit was unchecked by engedy@chromium.org

[engedy, 2016-12-29 13:48:12]

The CQ bit was checked by engedy@chromium.org

[commit-bot: I haz the power, 2016-12-29 13:48:20]

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-12-29 14:46:17]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483019292825860,
"parent_rev": "ab93188658ca47da0358a72514666b906517d1ed", "commit_rev":
"530152220aba2a49ef4bff65fdb761642d8d2ba1"}

[commit-bot: I haz the power, 2016-12-29 14:46:45]

Description was changed from

==========
Prevent racey favicon loads in subresource filter browsertests.

The favicon load was at race with the page load event, which caused flakiness
in tests that verified, at page load completion time, the number of evaluated
loads recorded into histograms.

BUG=677370
==========

to

==========
Prevent racey favicon loads in subresource filter browsertests.

The favicon load was at race with the page load event, which caused flakiness
in tests that verified, at page load completion time, the number of evaluated
loads recorded into histograms.

BUG=677370

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

[commit-bot: I haz the power, 2016-12-29 14:46:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-02 15:52:17]

Description was changed from

==========
Prevent racey favicon loads in subresource filter browsertests.

The favicon load was at race with the page load event, which caused flakiness
in tests that verified, at page load completion time, the number of evaluated
loads recorded into histograms.

BUG=677370

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

to

==========
Prevent racey favicon loads in subresource filter browsertests.

The favicon load was at race with the page load event, which caused flakiness
in tests that verified, at page load completion time, the number of evaluated
loads recorded into histograms.

BUG=677370

Committed: https://crrev.com/311f7c0a14700bd54a1ee50c3f530334579ea99d
Cr-Commit-Position: refs/heads/master@{#440956}
==========

[commit-bot: I haz the power, 2017-01-02 15:52:18]

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