Telemetry: Use the leftmost argument to find the test to parse flags.

Telemetry: Use the leftmost argument to find the test to parse flags.

Previously it would check all of the arguments, and use the last test found.
This is causes problems as different tests can be used to create the parser and
accepts the parsed flags.

SHERIFFS: If errors related to parsing flags for run_measurement, run_benchmark
or run_gpu_test.py appear, this is likely the cause. It is safe to revert.

BUG=413334
Committed: https://crrev.com/5afe5a61f03743bd523dae66f662ad773f8befb9
Cr-Commit-Position: refs/heads/master@{#295738}

[jbroman, 2014-09-18 19:17:26]

jbroman@chromium.org changed reviewers:
+ ariblue@google.com, dtu@chromium.org, kbr@chromium.org, nednguyen@google.com,
tonyg@chromium.org

[jbroman, 2014-09-18 19:17:27]

This is an alternative fix for bug 413334.

https://codereview.chromium.org/560153004/ broke WebRTC, and
after discussion with nednguyen@, it seems easier to try and
fix this aspect of the bug first.

Ultimately a more thorough cleanup of command-line parsing
logic will be required.

[nednguyen, 2014-09-19 15:38:43]

LGTM
Thanks for fixing this! Maybe worth adding in the description that sheriff can
revert this change if errors related to flags show up.

https://codereview.chromium.org/582793002/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/test_runner.py (right):

https://codereview.chromium.org/582793002/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/test_runner.py:116: if len(sys.argv) > 2 and
'run'.startswith(sys.argv[1]):
+dtu: I wonder whether we can kill 'r', 'ru'.

[jbroman, 2014-09-19 17:19:28]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2014-09-19 17:19:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/582793002/20001

[commit-bot: I haz the power, 2014-09-19 18:11:54]

Committed patchset #2 (id:20001) as 50635a15c0b897991d34d0fb73633a8effb2f23e

[commit-bot: I haz the power, 2014-09-19 18:12:36]

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

[jbroman, 2014-09-19 19:31:15]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/584023003/ by jbroman@chromium.org.

The reason for reverting is: Broke Android debug gpu bot:
http://build.chromium.org/p/chromium.gpu/builders/Android%20Debug%20%28Nexus%...

This bot relies on being able to supply some flags before the test name; it
runs:
content/test/gpu/run_gpu_test.py --browser=android-content-shell
webgl_conformance --webgl-conformance-version=1.0.1.