Speculative fix for TestWebApkInstaller.Success on Lollipop Low-end Tester

Speculative fix for TestWebApkInstaller.Success on Lollipop Low-end Tester

BUG=635624
TBR=dfalcantara

Committed: https://crrev.com/dab69e22fde245ba5fb7b430e26f7573c159b0f2
Cr-Commit-Position: refs/heads/master@{#410662}

[pkotwicz, 2016-08-08 22:15:12]

Patchset #1 (id:1) has been deleted

[pkotwicz, 2016-08-08 22:15:57]

pkotwicz@chromium.org changed reviewers:
+ jbudorick@chromium.org

[pkotwicz, 2016-08-08 22:16:01]

jbudorick@ can you please take a look?

I wasn't able to repro the breakage on an Android One but I think that this CL
will fix the test breakage

[jbudorick, 2016-08-08 22:34:51]

While I'm fine with the fix, 20 -> 1000 is a big jump. Could a slightly shorter
timeout serve the same purpose?

[pkotwicz, 2016-08-08 23:32:45]

Ok, lets try 100ms

The goal of the timeout is that the timeout is long enough that the HTTP
requests to the EmbeddedTestServer have time to complete.

[jbudorick, 2016-08-08 23:33:38]

lgtm

On 2016/08/08 23:32:45, pkotwicz wrote:
> Ok, lets try 100ms

sgtm

> 
> The goal of the timeout is that the timeout is long enough that the HTTP
> requests to the EmbeddedTestServer have time to complete.

Makes sense.

[pkotwicz, 2016-08-09 03:45:36]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2016-08-09 03:46:03]

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

[jbudorick, 2016-08-09 03:47:09]

On 2016/08/09 03:46:03, commit-bot: I haz the power wrote:
> CQ is trying da patch. Follow status at
>  
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

beware that I'm not an owner here.

[commit-bot: I haz the power, 2016-08-09 03:51:58]

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

[commit-bot: I haz the power, 2016-08-09 03:51:59]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[pkotwicz, 2016-08-09 13:43:35]

Description was changed from

==========
Speculative fix for TestWebApkInstaller.Success on Lollipop Low-end Tester

BUG=635624
==========

to

==========
Speculative fix for TestWebApkInstaller.Success on Lollipop Low-end Tester

BUG=635624
TBR=dfalcantara
==========

[pkotwicz, 2016-08-09 13:43:35]

pkotwicz@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[pkotwicz, 2016-08-09 13:43:39]

The CQ bit was checked by pkotwicz@chromium.org

[commit-bot: I haz the power, 2016-08-09 13:43:57]

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-08-09 14:27:51]

Description was changed from

==========
Speculative fix for TestWebApkInstaller.Success on Lollipop Low-end Tester

BUG=635624
TBR=dfalcantara
==========

to

==========
Speculative fix for TestWebApkInstaller.Success on Lollipop Low-end Tester

BUG=635624
TBR=dfalcantara
==========

[commit-bot: I haz the power, 2016-08-09 14:27:52]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-08-09 14:29:26]

Description was changed from

==========
Speculative fix for TestWebApkInstaller.Success on Lollipop Low-end Tester

BUG=635624
TBR=dfalcantara
==========

to

==========
Speculative fix for TestWebApkInstaller.Success on Lollipop Low-end Tester

BUG=635624
TBR=dfalcantara

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

[commit-bot: I haz the power, 2016-08-09 14:29:27]

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