Adding 3 attempts at starting the browser for a gpu integration test.

Adding 3 attempts at starting the browser for a gpu integration test.

BUG=628022
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

[eyaich, 2016-07-25 14:22:15]

Description was changed from

==========
Adding 3 attempts at starting the browser for a gpu integration test.

BUG=628022
==========

to

==========
Adding 3 attempts at starting the browser for a gpu integration test.

BUG=628022
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[eyaich, 2016-07-25 14:24:40]

eyaich@google.com changed reviewers:
+ nednguyen@google.com

[nednguyen, 2016-07-25 15:13:31]

https://codereview.chromium.org/2181673002/diff/1/content/test/gpu/gpu_tests/...
File content/test/gpu/gpu_tests/gpu_integration_test.py (right):

https://codereview.chromium.org/2181673002/diff/1/content/test/gpu/gpu_tests/...
content/test/gpu/gpu_tests/gpu_integration_test.py:50:
self._RestartBrowser(reason)
Can you do a for loop instead of recursion?

[eyaich, 2016-07-26 13:15:33]

https://codereview.chromium.org/2181673002/diff/1/content/test/gpu/gpu_tests/...
File content/test/gpu/gpu_tests/gpu_integration_test.py (right):

https://codereview.chromium.org/2181673002/diff/1/content/test/gpu/gpu_tests/...
content/test/gpu/gpu_tests/gpu_integration_test.py:50:
self._RestartBrowser(reason)
On 2016/07/25 15:13:31, nednguyen wrote:
> Can you do a for loop instead of recursion?

Done.

[nednguyen, 2016-07-26 13:17:37]

lgtm

Can you add test coverage?

[eyaich, 2016-07-26 13:51:16]

done

[nednguyen, 2016-07-26 14:09:59]

lgtm

[eyaich, 2016-07-26 14:10:50]

The CQ bit was checked by eyaich@google.com

[commit-bot: I haz the power, 2016-07-26 14:11:07]

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-07-26 14:16:52]

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

[commit-bot: I haz the power, 2016-07-26 14:16:53]

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...)

[eyaich, 2016-07-26 14:22:03]

eyaich@google.com changed reviewers:
+ bajones@chromium.org, zmo@chromium.org

[eyaich, 2016-07-26 14:22:04]

Ken is out so could one of you take a look for directory ownership approval?

[bajones, 2016-07-26 17:49:17]

On 2016/07/26 14:22:04, eyaich wrote:
> Ken is out so could one of you take a look for directory ownership approval?

LGTM

[eyaich, 2016-07-26 17:49:57]

The CQ bit was checked by eyaich@google.com

[commit-bot: I haz the power, 2016-07-26 17:50:19]

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-07-26 17:55:01]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-26 17:57:08]

Description was changed from

==========
Adding 3 attempts at starting the browser for a gpu integration test.

BUG=628022
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Adding 3 attempts at starting the browser for a gpu integration test.

BUG=628022
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

[commit-bot: I haz the power, 2016-07-26 17:57:09]

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

[Ken Russell (switch to Gerrit), 2016-07-27 17:59:27]

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

[Ken Russell (switch to Gerrit), 2016-07-27 17:59:28]

Thank you for adding this! LGTM after the fact.