Fix for flaky MHTMLGenerationTest.ViewedMHTMLContainsNoStoreContentIfNoCacheControlPolicy test.

Fix for flaky MHTMLGenerationTest.ViewedMHTMLContainsNoStoreContentIfNoCacheControlPolicy test.

The problem is here: https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/download/mhtml_generation_browsertest.cc&l=61

The number of matches is checked at the first reply to the find request, without checking that the find operation actually finished. The number of matches should be checked only when |final_update| is true.

BUG=616110
Committed: https://crrev.com/d7c18f6c0bc2c601ee718e61fac29280e7059042
Cr-Commit-Position: refs/heads/master@{#396869}

[paulmeyer, 2016-05-31 17:11:20]

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

[commit-bot: I haz the power, 2016-05-31 17:11:37]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2026773002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2026773002/1

[paulmeyer, 2016-05-31 17:12:56]

paulmeyer@chromium.org changed reviewers:
+ nick@chromium.org

[dewittj, 2016-05-31 17:15:54]

dewittj@chromium.org changed reviewers:
+ dewittj@chromium.org

[dewittj, 2016-05-31 17:15:54]

not the OWNER, but lgtm.  Thanks for the fix!

[dewittj, 2016-05-31 17:17:05]

also, these tests were disabled on Android, could you re-enable them?

[ncarter (slow), 2016-05-31 17:37:57]

lgtm

[paulmeyer, 2016-05-31 18:08:45]

On 2016/05/31 17:17:05, dewittj wrote:
> also, these tests were disabled on Android, could you re-enable them?

I looked at the bug associated with the tests being disabled, and that issue is
unrelated, so they should remain disabled.

[commit-bot: I haz the power, 2016-05-31 18:15:30]

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

[commit-bot: I haz the power, 2016-05-31 18:15:31]

Dry run: This issue passed the CQ dry run.

[paulmeyer, 2016-05-31 18:17:56]

The CQ bit was checked by paulmeyer@chromium.org

[commit-bot: I haz the power, 2016-05-31 18:18:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2026773002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2026773002/1

[commit-bot: I haz the power, 2016-05-31 18:23:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-31 18:25:07]

Description was changed from

==========
Fix for flaky
MHTMLGenerationTest.ViewedMHTMLContainsNoStoreContentIfNoCacheControlPolicy
test.

The problem is here:
https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/do...

The number of matches is checked at the first reply to the find request, without
checking that the find operation actually finished. The number of matches should
be checked only when |final_update| is true.

BUG=616110
==========

to

==========
Fix for flaky
MHTMLGenerationTest.ViewedMHTMLContainsNoStoreContentIfNoCacheControlPolicy
test.

The problem is here:
https://code.google.com/p/chromium/codesearch#chromium/src/content/browser/do...

The number of matches is checked at the first reply to the find request, without
checking that the find operation actually finished. The number of matches should
be checked only when |final_update| is true.

BUG=616110

Committed: https://crrev.com/d7c18f6c0bc2c601ee718e61fac29280e7059042
Cr-Commit-Position: refs/heads/master@{#396869}
==========

[commit-bot: I haz the power, 2016-05-31 18:25:08]

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