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

tools/telemetry/telemetry/page/record_wpr.py

Index: tools/telemetry/telemetry/page/record_wpr.py
diff --git a/tools/telemetry/telemetry/page/record_wpr.py b/tools/telemetry/telemetry/page/record_wpr.py
index ba25594793ac7285bdafd67c1288393fc0ec5e2d..71c279816b827abeaba7c80266d2225c1219d493 100755
--- a/tools/telemetry/telemetry/page/record_wpr.py
+++ b/tools/telemetry/telemetry/page/record_wpr.py
@@ -108,6 +108,7 @@ def _MaybeGetInstanceOfClass(target, base_dir, cls):
 
 
 class WprRecorder(object):
+
   def __init__(self, base_dir, target, args=None):
     action_names_to_run = FindAllActionNames(base_dir)
     self._record_page_test = RecorderPageTest(action_names_to_run)
@@ -146,6 +147,10 @@ class WprRecorder(object):
     if self._benchmark is not None:
       self._benchmark.AddCommandLineArgs(self._parser)
       self._benchmark.SetArgumentDefaults(self._parser)

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

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:58]

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).

+    self._SetArgumentDefaults()
+
+  def _SetArgumentDefaults(self):
+    self._parser.set_defaults(**{'output_format': 'none'})
 
   def _ParseArgs(self, args=None):
     args_to_parse = sys.argv[1:] if args is None else args