[Telemetry] Support downloading profiles from cloud storage

[Telemetry] Support download profiles from cloud storage

Needed for theme and many extension startup tests.

* Add benchmarks/startup_with_extensions.py - new theme and many_profile startup tests.
* Teach test.py to use .sha1 files for profile directories, these are downloaded from cloud storage, unzipped and then used as the profile directory by the browser.
* Add a function to browser_finder to allow querying the target OS.

BUG=None

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

[jeremy, 2014-01-06 06:57:07]

tonyg/dtu: review
aberent: WDYT about TargetBrowserOS()?

https://codereview.chromium.org/124733002/diff/20001/tools/perf/generated_pro...
File tools/perf/generated_profiles/darwin/many_extensions_profile.zip.sha1
(right):

https://codereview.chromium.org/124733002/diff/20001/tools/perf/generated_pro...
tools/perf/generated_profiles/darwin/many_extensions_profile.zip.sha1:1:
87f83fecf7b0838cf5f492e1977f6b32ef4b6a71
Is there a better place to put these?

https://codereview.chromium.org/124733002/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/core/browser_finder.py (right):

https://codereview.chromium.org/124733002/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/core/browser_finder.py:34: def
TargetBrowserOS(options):
Guessing that we want a more robust version of this, one option would be to
perform most of FindBrowser() here to resolve the browser object and then cache
it to pass it back in FindBrowser?

In addition to the above, we'd probably want Browser objects to have a method
returning their Target OS.

WDYT?

[aberent, 2014-01-06 10:44:52]

https://codereview.chromium.org/124733002/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/core/browser_finder.py (right):

https://codereview.chromium.org/124733002/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/core/browser_finder.py:34: def
TargetBrowserOS(options):
On 2014/01/06 06:57:07, jeremy wrote:
> Guessing that we want a more robust version of this, one option would be to
> perform most of FindBrowser() here to resolve the browser object and then
cache
> it to pass it back in FindBrowser?
> 
> In addition to the above, we'd probably want Browser objects to have a method
> returning their Target OS.
> 
> WDYT?
How difficult would it be to create the robust version immediately? Also could
the robust version simply call FindBrowser() and then ask the resulting browser
what the OS is?

[jeremy, 2014-01-06 11:23:57]

Tony/dtu: rest of CL is still reviewable, thanks!

https://codereview.chromium.org/124733002/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/core/browser_finder.py (right):

https://codereview.chromium.org/124733002/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/core/browser_finder.py:34: def
TargetBrowserOS(options):
Yes, was mainly looking for feedback before I make further changes here.

Per off-line discussion I plan to call FindBrowser() here and cache the value,
then add a static method to the Browser class to return the target OS.

The only extra bit is an assertion in FindBrowser to make sure that any later
calls have the same value for the --browser command line variable.

[tonyg, 2014-01-06 18:53:55]

lgtm when aberent is happy

https://codereview.chromium.org/124733002/diff/20001/tools/perf/benchmarks/st...
File tools/perf/benchmarks/startup_with_extensions.py (right):

https://codereview.chromium.org/124733002/diff/20001/tools/perf/benchmarks/st...
tools/perf/benchmarks/startup_with_extensions.py:9: class
StartupColdTheme(test.Test):
I recommend putting these in benchmarks/startup.py instead of a new file.

[jeremy, 2014-01-07 15:52:13]

aberent: all fixed, ready for another look :)

[aberent, 2014-01-07 17:47:24]

lgtm

[commit-bot: I haz the power, 2014-01-08 08:23:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jeremy@chromium.org/124733002/190001

[commit-bot: I haz the power, 2014-01-08 09:18:07]

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

[commit-bot: I haz the power, 2014-01-08 10:21:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jeremy@chromium.org/124733002/530001

[commit-bot: I haz the power, 2014-01-08 12:58:20]

Change committed as 243543