Allow multiple performance values in endure_result_parser.py.

Allow multiple performance values in endure_result_parser.py.

It's similar to http://crrev.com/154247.


BUG=122119
NOTRY=true

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

[Dai Mikurube (NOT FULLTIME), 2012-09-20 07:03:35]

Hi Dennis,

Could you take a look as commented at http://codereview.chromium.org/10944003/?

[Dai Mikurube (NOT FULLTIME), 2012-09-24 22:56:29]

ping, just for reminding.

[dennis_jeffrey, 2012-09-26 00:34:29]

LGTM with 2 nits to consider before submitting.

http://codereview.chromium.org/10960009/diff/2001/chrome/test/functional/perf...
File chrome/test/functional/perf/endure_result_parser.py (right):

http://codereview.chromium.org/10960009/diff/2001/chrome/test/functional/perf...
chrome/test/functional/perf/endure_result_parser.py:347: key1 =
data['graph_name']
use a more descriptive variable name, such as "key_graph", instead of "key1"

http://codereview.chromium.org/10960009/diff/2001/chrome/test/functional/perf...
chrome/test/functional/perf/endure_result_parser.py:348: key2 =
data['description']
use a more descriptive variable name, such as "key_description" or "key_desc",
instead of "key2"

[Dai Mikurube (NOT FULLTIME), 2012-09-26 02:44:51]

Thanks for reviewing.  Fixed.

I'll be committing it after your CL is landed.

http://codereview.chromium.org/10960009/diff/2001/chrome/test/functional/perf...
File chrome/test/functional/perf/endure_result_parser.py (right):

http://codereview.chromium.org/10960009/diff/2001/chrome/test/functional/perf...
chrome/test/functional/perf/endure_result_parser.py:347: key1 =
data['graph_name']
On 2012/09/26 00:34:29, dennis_jeffrey wrote:
> use a more descriptive variable name, such as "key_graph", instead of "key1"

Done.

http://codereview.chromium.org/10960009/diff/2001/chrome/test/functional/perf...
chrome/test/functional/perf/endure_result_parser.py:348: key2 =
data['description']
On 2012/09/26 00:34:29, dennis_jeffrey wrote:
> use a more descriptive variable name, such as "key_description" or "key_desc",
> instead of "key2"

Done.

[commit-bot: I haz the power, 2012-09-26 03:39:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dmikurube@chromium.org/10960009/9001

[commit-bot: I haz the power, 2012-09-26 03:50:50]

Change committed as 158744