Add Suites results to Speedometer

Add Suites results to Speedometer

Display the results for the individual benchmark suites, this is
acheived by storing the full results set and quering it after
the benchmark has finished.

BUG=389257
Committed: https://crrev.com/14014307773800b452b475c15b7860039114314a
Cr-Commit-Position: refs/heads/master@{#291848}

[fmeawad, 2014-08-25 22:27:00]

fmeawad@chromium.org changed reviewers:
+ dtu@chromium.org, tonyg@chromium.org

[fmeawad, 2014-08-25 22:27:00]

I injected a script that keeps the results around, and then queried those
results at the end.
PTAL.

[tonyg, 2014-08-25 22:32:13]

lgtm w/ nits

https://codereview.chromium.org/503053005/diff/20001/tools/perf/benchmarks/sp...
File tools/perf/benchmarks/speedometer.py (right):

https://codereview.chromium.org/503053005/diff/20001/tools/perf/benchmarks/sp...
tools/perf/benchmarks/speedometer.py:43: benchmarkClient.didRunSuites = function
(measuredValues) {
uber-nit: no space after "function"

https://codereview.chromium.org/503053005/diff/20001/tools/perf/benchmarks/sp...
tools/perf/benchmarks/speedometer.py:54:
tab.EvaluateJavaScript('benchmarkClient._timeValues')))
There's an optional "important" arg. I forget the default, but we want to make
sure that Total is important and the other are not.

[fmeawad, 2014-08-25 22:49:24]

https://codereview.chromium.org/503053005/diff/20001/tools/perf/benchmarks/sp...
File tools/perf/benchmarks/speedometer.py (right):

https://codereview.chromium.org/503053005/diff/20001/tools/perf/benchmarks/sp...
tools/perf/benchmarks/speedometer.py:43: benchmarkClient.didRunSuites = function
(measuredValues) {
On 2014/08/25 22:32:13, tonyg wrote:
> uber-nit: no space after "function"

Done.

https://codereview.chromium.org/503053005/diff/20001/tools/perf/benchmarks/sp...
tools/perf/benchmarks/speedometer.py:54:
tab.EvaluateJavaScript('benchmarkClient._timeValues')))
On 2014/08/25 22:32:13, tonyg wrote:
> There's an optional "important" arg. I forget the default, but we want to make
> sure that Total is important and the other are not.

Default was true. Added in both to make it more readable.

[tonyg, 2014-08-25 22:51:13]

The CQ bit was checked by tonyg@chromium.org

[commit-bot: I haz the power, 2014-08-25 22:53:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fmeawad@chromium.org/503053005/40001

[commit-bot: I haz the power, 2014-08-26 01:52:48]

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

[commit-bot: I haz the power, 2014-08-26 01:52:48]

Failed to commit the patch.

[tonyg, 2014-08-26 03:12:21]

The CQ bit was checked by tonyg@chromium.org

[commit-bot: I haz the power, 2014-08-26 03:13:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fmeawad@chromium.org/503053005/40001

[commit-bot: I haz the power, 2014-08-26 03:15:06]

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

[commit-bot: I haz the power, 2014-08-26 03:15:07]

Failed to commit the patch.

[fmeawad, 2014-08-26 05:35:13]

The CQ bit was checked by fmeawad@chromium.org

[commit-bot: I haz the power, 2014-08-26 05:36:23]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/fmeawad@chromium.org/503053005/40001

[commit-bot: I haz the power, 2014-08-26 05:37:19]

Committed patchset #3 (40001) as ddf514b0c8b99c37149005a2b79f507060da948f

[commit-bot: I haz the power, 2014-09-10 02:40:46]

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