Add a telemetry-based gpu_tests folder

Add a telemetry-based gpu_tests folder

BUG=143317
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=183915

[nduca, 2013-02-13 22:09:15]

ptal

[Ken Russell (switch to Gerrit), 2013-02-13 22:15:40]

LGTM. Thanks for putting this infrastructure in place.

https://codereview.chromium.org/12252020/diff/1/content/test/gpu/page_sets/pi...
File content/test/gpu/page_sets/pixel_tests.json (right):

https://codereview.chromium.org/12252020/diff/1/content/test/gpu/page_sets/pi...
content/test/gpu/page_sets/pixel_tests.json:6: "url":
"file:///../../data/gpu/pixel_webgl.html",
That's a strange URL...

[nduca, 2013-02-13 22:45:33]

Note, I have filed crbug.com/176139 to unify page_test_runner with the erstwhile
multipage_benchmark_runner.

[dtu, 2013-02-13 23:54:23]

https://chromiumcodereview.appspot.com/12252020/diff/1/content/test/gpu/page_...
File content/test/gpu/page_sets/__init__.py (right):

https://chromiumcodereview.appspot.com/12252020/diff/1/content/test/gpu/page_...
content/test/gpu/page_sets/__init__.py:1: # Copyright (c) 2012 The Chromium
Authors. All rights reserved.
Possibly move these into telemetry under discover.py and have it take a
directory as a parameter.

https://chromiumcodereview.appspot.com/12252020/diff/1/content/test/gpu/page_...
File content/test/gpu/page_sets/pixel_tests.json (right):

https://chromiumcodereview.appspot.com/12252020/diff/1/content/test/gpu/page_...
content/test/gpu/page_sets/pixel_tests.json:6: "url":
"file:///../../data/gpu/pixel_webgl.html",
On 2013/02/13 22:15:40, kbr wrote:
> That's a strange URL...

Bug filed: http://crbug.com/176143

https://chromiumcodereview.appspot.com/12252020/diff/1/tools/telemetry/teleme...
File tools/telemetry/telemetry/page_test_runner.py (right):

https://chromiumcodereview.appspot.com/12252020/diff/1/tools/telemetry/teleme...
tools/telemetry/telemetry/page_test_runner.py:38: test = tests[test_name]()
Print a "test not found" message instead of crashing if it's not a valid test
name.

https://chromiumcodereview.appspot.com/12252020/diff/1/tools/telemetry/teleme...
tools/telemetry/telemetry/page_test_runner.py:45: import page_sets # pylint:
disable=F0401
How does this import work? Also, tools/telemetry shouldn't depend on tools/perf
and content/test/gpu. Better if the caller passes in the list of page sets or
discovery function.

https://chromiumcodereview.appspot.com/12252020/diff/1/tools/telemetry/teleme...
tools/telemetry/telemetry/page_test_runner.py:66: sys.stdout.write('%i pages
succeed\n' % len(results.page_successes))
print

[nduca, 2013-02-20 05:03:13]

Most changes made, but page discovery still left as a copy. I dont think it
warrants code reuse imo.

[nduca, 2013-02-21 04:43:11]

@dtu, ping

[dtu, 2013-02-21 05:12:07]

lgtm

[commit-bot: I haz the power, 2013-02-21 05:18:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nduca@chromium.org/12252020/8001

[commit-bot: I haz the power, 2013-02-21 08:09:22]

Retried try job too often on android_clang_dbg for step(s) build
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[commit-bot: I haz the power, 2013-02-21 23:46:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nduca@chromium.org/12252020/8001

[commit-bot: I haz the power, 2013-02-22 00:05:16]

Change committed as 183915