Fix the flaky Warmup URL test in data reduction proxy

Fix the flaky Warmup URL test in data reduction proxy

Remove the part of the test that waits for the fetch to succeed. The test
is flaking because sometimes the URL may be fetched before the callback
is set. This causes the run loop to wait forever.

BUG=702748
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

Review-Url: https://codereview.chromium.org/2765763003
Cr-Commit-Position: refs/heads/master@{#458917}
Committed: https://chromium.googlesource.com/chromium/src/+/dfc3e7204e434185cbaa0d6cb48ae0b37481f007

[tbansal1, 2017-03-21 07:23:48]

Description was changed from

==========
warmup URL

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

to

==========
warmup URL

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[tbansal1, 2017-03-21 07:25:31]

Patchset #1 (id:1) has been deleted

[tbansal1, 2017-03-21 07:25:36]

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

[commit-bot: I haz the power, 2017-03-21 07:25:51]

Dry run: 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-21 10:29:49]

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

[commit-bot: I haz the power, 2017-03-21 10:29:50]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tbansal1, 2017-03-21 16:39:14]

Description was changed from

==========
warmup URL

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
Fix the flaky Warmup URL test in data reduction proxy

Remove the part of the test that waits for the fetch to succeed.

BUG=702748
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[tbansal1, 2017-03-21 16:40:36]

Description was changed from

==========
Fix the flaky Warmup URL test in data reduction proxy

Remove the part of the test that waits for the fetch to succeed.

BUG=702748
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
Fix the flaky Warmup URL test in data reduction proxy

Remove the part of the test that waits for the fetch to succeed. The test
is flaking because sometimes the URL may be fetched before the callback 
is set. This causes the run loop to wait forever.

BUG=702748
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[tbansal1, 2017-03-21 16:49:13]

Patchset #1 (id:20001) has been deleted

[tbansal1, 2017-03-21 16:49:17]

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

[tbansal1, 2017-03-21 16:49:33]

tbansal@chromium.org changed reviewers:
+ ryansturm@chromium.org

[tbansal1, 2017-03-21 16:49:34]

ryansturm: ptal. Thanks.

[commit-bot: I haz the power, 2017-03-21 16:50:04]

Dry run: 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-21 17:42:07]

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

[commit-bot: I haz the power, 2017-03-21 17:42:08]

Dry run: This issue passed the CQ dry run.

[RyanSturm, 2017-03-22 21:52:34]

lgtm

[tbansal1, 2017-03-22 21:54:07]

The CQ bit was checked by tbansal@chromium.org

[commit-bot: I haz the power, 2017-03-22 21:54:47]

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-22 23:01:41]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490219647138960,
"parent_rev": "597c3878ead7e6df748ca0fbe4ca6d9ad14bde85", "commit_rev":
"dfc3e7204e434185cbaa0d6cb48ae0b37481f007"}

[commit-bot: I haz the power, 2017-03-22 23:09:14]

Description was changed from

==========
Fix the flaky Warmup URL test in data reduction proxy

Remove the part of the test that waits for the fetch to succeed. The test
is flaking because sometimes the URL may be fetched before the callback 
is set. This causes the run loop to wait forever.

BUG=702748
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
Fix the flaky Warmup URL test in data reduction proxy

Remove the part of the test that waits for the fetch to succeed. The test
is flaking because sometimes the URL may be fetched before the callback
is set. This causes the run loop to wait forever.

BUG=702748
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

Review-Url: https://codereview.chromium.org/2765763003
Cr-Commit-Position: refs/heads/master@{#458917}
Committed:
https://chromium.googlesource.com/chromium/src/+/dfc3e7204e434185cbaa0d6cb48a...
==========

[commit-bot: I haz the power, 2017-03-22 23:09:16]

Committed patchset #1 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/dfc3e7204e434185cbaa0d6cb48a...