Add pyauto-based perf test for the SunSpider benchmark suite.

Add pyauto-based perf test for the SunSpider benchmark suite.

BUG=chromium-os:18457
TEST=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107834

[Nirnimesh, 2011-10-28 07:36:49]

LGTM

http://codereview.chromium.org/8413026/diff/1/chrome/test/functional/perf.py
File chrome/test/functional/perf.py (right):

http://codereview.chromium.org/8413026/diff/1/chrome/test/functional/perf.py#...
chrome/test/functional/perf.py:290: total = re.search(total_pattern,
results).group(1)
merge with previous line

http://codereview.chromium.org/8413026/diff/1/chrome/test/functional/perf.py#...
chrome/test/functional/perf.py:292: self._OutputPerfGraphValue('%s_%s' % ('ms',
'SunSpider-total'),
why not use 'ms_SunSpider-total' directly?

http://codereview.chromium.org/8413026/diff/1/chrome/test/functional/perf.py#...
chrome/test/functional/perf.py:305: '%s_%s-%s-%s' % ('ms', 'SunSpider',
category_name, result_name),
Include ms_Sunspider in the format itself.

[dennis_jeffrey, 2011-10-28 22:29:40]

Thank you for the review.  Now marking this as commit-ready.

http://codereview.chromium.org/8413026/diff/1/chrome/test/functional/perf.py
File chrome/test/functional/perf.py (right):

http://codereview.chromium.org/8413026/diff/1/chrome/test/functional/perf.py#...
chrome/test/functional/perf.py:290: total = re.search(total_pattern,
results).group(1)
On 2011/10/28 07:36:49, Nirnimesh wrote:
> merge with previous line

Done.

http://codereview.chromium.org/8413026/diff/1/chrome/test/functional/perf.py#...
chrome/test/functional/perf.py:292: self._OutputPerfGraphValue('%s_%s' % ('ms',
'SunSpider-total'),
On 2011/10/28 07:36:49, Nirnimesh wrote:
> why not use 'ms_SunSpider-total' directly?

Done.

http://codereview.chromium.org/8413026/diff/1/chrome/test/functional/perf.py#...
chrome/test/functional/perf.py:305: '%s_%s-%s-%s' % ('ms', 'SunSpider',
category_name, result_name),
On 2011/10/28 07:36:49, Nirnimesh wrote:
> Include ms_Sunspider in the format itself.

Done.

[commit-bot: I haz the power, 2011-10-28 22:29:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dennisjeffrey@chromium.org/8413026/3001

[commit-bot: I haz the power, 2011-10-29 00:46:33]

Change committed as 107834