By default don't output results.html for recordings.

By default don't output results.html for recordings.

BUG=

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

[chrishenry, 2014-08-08 19:45:04]

lgtm

https://codereview.chromium.org/459433002/diff/1/tools/telemetry/telemetry/pa...
File tools/telemetry/telemetry/page/record_wpr.py (right):

https://codereview.chromium.org/459433002/diff/1/tools/telemetry/telemetry/pa...
tools/telemetry/telemetry/page/record_wpr.py:112: default_options =
{'output_format': 'none'}
Let's just inline this?

[ariblue, 2014-08-08 19:54:40]

https://codereview.chromium.org/459433002/diff/1/tools/telemetry/telemetry/pa...
File tools/telemetry/telemetry/page/record_wpr.py (right):

https://codereview.chromium.org/459433002/diff/1/tools/telemetry/telemetry/pa...
tools/telemetry/telemetry/page/record_wpr.py:112: default_options =
{'output_format': 'none'}
On 2014/08/08 19:45:04, chrishenry wrote:
> Let's just inline this?

Done.

[nednguyen, 2014-08-08 20:01:52]

lgtm

[commit-bot: I haz the power, 2014-08-08 20:04:46]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ariblue@google.com/459433002/20001

[nednguyen, 2014-08-08 20:06:53]

https://codereview.chromium.org/459433002/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/page/record_wpr.py (right):

https://codereview.chromium.org/459433002/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/page/record_wpr.py:149:
self._benchmark.SetArgumentDefaults(self._parser)
Actually, what would happen if this benchmark set output_format default to s.t
like html? Does the later one take precedence?

[ariblue, 2014-08-08 20:13:59]

https://codereview.chromium.org/459433002/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/page/record_wpr.py (right):

https://codereview.chromium.org/459433002/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/page/record_wpr.py:149:
self._benchmark.SetArgumentDefaults(self._parser)
On 2014/08/08 20:06:53, nednguyen wrote:
> Actually, what would happen if this benchmark set output_format default to s.t
> like html? Does the later one take precedence?

The later default set will take precedence, correct. I believe this is what we
want for the recording step (i.e. only ever output results if explicitly
specified as a flag when starting the recording).

[commit-bot: I haz the power, 2014-08-08 23:37:01]

Change committed as 288468