[test] Refactoring - Make perf suite definitions stateless regarding measurements.

[test] Refactoring - Make perf suite definitions stateless regarding measurements.

This prepares for making multiple measurements of one trace.
For this, the suite/trace configurations need to be
independent of the measurement instances.

BUG=chromium:507213
LOG=n
NOTRY=true

Committed: https://crrev.com/373d7396503212d8d939e18a2e55436ae57998a2
Cr-Commit-Position: refs/heads/master@{#29531}

[Michael Achenbach, 2015-07-08 09:58:37]

machenbach@chromium.org changed reviewers:
+ tandrii@chromium.org, ulan@chromium.org

[Michael Achenbach, 2015-07-08 10:03:19]

PTAL

https://codereview.chromium.org/1227033002/diff/1/tools/run_perf.py
File tools/run_perf.py (right):

https://codereview.chromium.org/1227033002/diff/1/tools/run_perf.py#newcode173
tools/run_perf.py:173: """Represents a series of results of one trace.
Basically copy from line 259.

https://codereview.chromium.org/1227033002/diff/1/tools/run_perf.py#newcode219
tools/run_perf.py:219: """Iterates over the output of multiple benchmark reruns
and accumulates
Basically copy from line 320.

https://codereview.chromium.org/1227033002/diff/1/tools/run_perf.py#newcode262
tools/run_perf.py:262: """Iterates over the output of multiple benchmark reruns
and accumulates
Basically copy from line 365.

[Michael Achenbach, 2015-07-08 12:14:05]

Patch 2 does some renaming.

[ulan, 2015-07-08 12:26:32]

lgtm

https://codereview.chromium.org/1227033002/diff/20001/tools/run_perf.py
File tools/run_perf.py (right):

https://codereview.chromium.org/1227033002/diff/20001/tools/run_perf.py#newco...
tools/run_perf.py:236: res = reduce(lambda r, t: r + t.GetResults(),
measurements, Results())
Nit: maybe use m instead of t in lambda?

[Michael Achenbach, 2015-07-08 12:29:53]

https://codereview.chromium.org/1227033002/diff/20001/tools/run_perf.py
File tools/run_perf.py (right):

https://codereview.chromium.org/1227033002/diff/20001/tools/run_perf.py#newco...
tools/run_perf.py:236: res = reduce(lambda r, t: r + t.GetResults(),
measurements, Results())
On 2015/07/08 12:26:32, ulan wrote:
> Nit: maybe use m instead of t in lambda?

Done.

[Michael Achenbach, 2015-07-08 12:29:59]

The CQ bit was checked by machenbach@chromium.org

[Michael Achenbach, 2015-07-08 12:29:59]

The patchset sent to the CQ was uploaded after l-g-t-m from ulan@chromium.org
Link to the patchset: https://codereview.chromium.org/1227033002/#ps40001
(title: "Review")

[commit-bot: I haz the power, 2015-07-08 12:30:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1227033002/40001

[commit-bot: I haz the power, 2015-07-08 12:31:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-07-08 12:31:52]

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