Support multiple URLs for latency testing

Support multiple URLs for latency testing

Adds support for testing multiple URLs per script run for automated VR latency
testing. Allows us to get latency results from more realistic scenarios instead
of just the synthetic flicker app without having to run the script and all the
setup overhead that comes with it multiple times.

Also fixes an issue caused by the recent swarming bot config change that resulted in the adb startup message appearing in the device name.

BUG=708747
Review-Url: https://codereview.chromium.org/2904583003
Cr-Commit-Position: refs/heads/master@{#476058}
Committed: https://chromium.googlesource.com/chromium/src/+/cfcb849246a5b77e9cac7ae92813f9b386f83b68

[bsheedy, 2017-05-24 00:56:49]

bsheedy@chromium.org changed reviewers:
+ klausw@chromium.org, leilei@chromium.org

[bsheedy, 2017-05-24 00:56:50]

+klausw for URL selection - can add more/change them around as desired
+leilei for overall review

I'll add an OWNER once everything looks fine to you two.

I haven't been able to test uploading of the results to the perf dashboard yet
since the move reset my IP. I'll get it re-whitelisted tomorrow and test then.

[Lei Lei, 2017-05-24 01:04:48]

lgtm

https://codereview.chromium.org/2904583003/diff/1/chrome/test/vr/perf/latency...
File chrome/test/vr/perf/latency/webvr_latency_test.py (right):

https://codereview.chromium.org/2904583003/diff/1/chrome/test/vr/perf/latency...
chrome/test/vr/perf/latency/webvr_latency_test.py:130: """Perform any
platform-specific setup before each test."""
s/Perform/Performs to keep consistent.

[bsheedy, 2017-05-25 01:07:04]

Description was changed from

==========
Support multiple URLs for latency testing

Adds support for testing multiple URLs per script run for automated VR latency
testing. Allows us to get latency results from more realistic scenarios instead
of just the synthetic flicker app without having to run the script and all the
setup overhead that comes with it multiple times.

BUG=708747
==========

to

==========
Support multiple URLs for latency testing

Adds support for testing multiple URLs per script run for automated VR latency
testing. Allows us to get latency results from more realistic scenarios instead
of just the synthetic flicker app without having to run the script and all the
setup overhead that comes with it multiple times.

Also fixes an issue caused by the recent swarming bot config change that
resulted in the adb startup message appearing in the device name.

BUG=708747
==========

[bsheedy, 2017-05-25 17:46:22]

bsheedy@chromium.org changed reviewers:
+ phajdan.jr@chromium.org

[bsheedy, 2017-05-25 17:46:23]

Tested to work properly with the Chrome performance dashboard.

+phajdan.jr for chrome/test OWNERS

https://codereview.chromium.org/2904583003/diff/1/chrome/test/vr/perf/latency...
File chrome/test/vr/perf/latency/webvr_latency_test.py (right):

https://codereview.chromium.org/2904583003/diff/1/chrome/test/vr/perf/latency...
chrome/test/vr/perf/latency/webvr_latency_test.py:130: """Perform any
platform-specific setup before each test."""
On 2017/05/24 01:04:47, Lei Lei wrote:
> s/Perform/Performs to keep consistent.

Done.

[bsheedy, 2017-05-31 21:51:19]

bsheedy@chromium.org changed reviewers:
+ mthiesse@chromium.org
- phajdan.jr@chromium.org

[bsheedy, 2017-05-31 21:51:20]

-phajdan.jr
+mthiesse for new VR-specific OWNERS

[mthiesse, 2017-05-31 22:03:13]

lgtm

[bsheedy, 2017-05-31 22:03:52]

The CQ bit was checked by bsheedy@chromium.org

[bsheedy, 2017-05-31 22:03:52]

The patchset sent to the CQ was uploaded after l-g-t-m from leilei@chromium.org
Link to the patchset: https://codereview.chromium.org/2904583003/#ps80001
(title: "Fix adb startup message appearing in device name")

[commit-bot: I haz the power, 2017-05-31 22:04: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-05-31 22:19:00]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1496268232255890,
"parent_rev": "0d38c3e37df38e4042922cead8abf1589264c226", "commit_rev":
"cfcb849246a5b77e9cac7ae92813f9b386f83b68"}

[commit-bot: I haz the power, 2017-05-31 22:19:09]

Description was changed from

==========
Support multiple URLs for latency testing

Adds support for testing multiple URLs per script run for automated VR latency
testing. Allows us to get latency results from more realistic scenarios instead
of just the synthetic flicker app without having to run the script and all the
setup overhead that comes with it multiple times.

Also fixes an issue caused by the recent swarming bot config change that
resulted in the adb startup message appearing in the device name.

BUG=708747
==========

to

==========
Support multiple URLs for latency testing

Adds support for testing multiple URLs per script run for automated VR latency
testing. Allows us to get latency results from more realistic scenarios instead
of just the synthetic flicker app without having to run the script and all the
setup overhead that comes with it multiple times.

Also fixes an issue caused by the recent swarming bot config change that
resulted in the adb startup message appearing in the device name.

BUG=708747

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

[commit-bot: I haz the power, 2017-05-31 22:19:10]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/cfcb849246a5b77e9cac7ae92813...