Perf scripts: improves PrintPerfResult documentation.

Perf scripts: improves PrintPerfResult documentation.

No functional changes, just add a bit more info about the parameters.

BUG=

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

[bulach, 2013-10-02 14:03:38]

ptal

[Dominik Grewe, 2013-10-02 14:11:59]

LGTM with one nit. Thanks!

https://codereview.chromium.org/25634008/diff/1/build/util/lib/common/perf_te...
File build/util/lib/common/perf_tests_results_helper.py (right):

https://codereview.chromium.org/25634008/diff/1/build/util/lib/common/perf_te...
build/util/lib/common/perf_tests_results_helper.py:96:
http://src.chromium.org/viewvc/chrome/trunk/tools/build/scripts/slave/process...
Do you want to link to the log of the file or the file itself?

[bulach, 2013-10-02 16:50:04]

https://codereview.chromium.org/25634008/diff/1/build/util/lib/common/perf_te...
File build/util/lib/common/perf_tests_results_helper.py (right):

https://codereview.chromium.org/25634008/diff/1/build/util/lib/common/perf_te...
build/util/lib/common/perf_tests_results_helper.py:96:
http://src.chromium.org/viewvc/chrome/trunk/tools/build/scripts/slave/process...
On 2013/10/02 14:11:59, Dominik Grewe wrote:
> Do you want to link to the log of the file or the file itself?

ahn, yeah, the file itself :)
done, thanks!

[commit-bot: I haz the power, 2013-10-02 16:51:34]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[bulach, 2013-10-03 08:28:32]

let's see if ross' account is setup by now :) ptal

[rmcilroy, 2013-10-03 08:56:30]

lgtm

Fingers crossed ;)

[commit-bot: I haz the power, 2013-10-03 09:05:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/25634008/5001

[commit-bot: I haz the power, 2013-10-03 10:55:39]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[commit-bot: I haz the power, 2013-10-03 17:20:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/25634008/5001

[commit-bot: I haz the power, 2013-10-03 17:25:00]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...

[commit-bot: I haz the power, 2013-10-03 19:53:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/25634008/5001

[commit-bot: I haz the power, 2013-10-03 20:11:06]

Change committed as 226827