Add --output-dir flag for page_runner.py.

tools/telemetry/telemetry/page/page_runner.py

Index: tools/telemetry/telemetry/page/page_runner.py
diff --git a/tools/telemetry/telemetry/page/page_runner.py b/tools/telemetry/telemetry/page/page_runner.py
index fa5c347cfffc56a19c375ae1cdf5c0f1c1b3bd95..11a532b9d8779ad9afe45e76ade938e9fef87a2c 100644
--- a/tools/telemetry/telemetry/page/page_runner.py
+++ b/tools/telemetry/telemetry/page/page_runner.py
@@ -124,7 +124,10 @@ class _RunState(object):
 
   def StartProfiling(self, page, finder_options):
     if not self.profiler_dir:
-      self.profiler_dir = tempfile.mkdtemp()
+      if finder_options.output_dir:
+        self.profiler_dir = finder_options.output_dir
+      else:

[nednguyen, 2014/10/01 04:37:36]

This is awkward since results.html will no longer be in the same directory as
results.html. If we expect all of the artifacts of the run are produced in a
same directory, let's make it consistent.

[ariblue, 2014/10/01 17:21:09]

I just wanted to keep the default behavior the same as it was for now, as to not
break anyone's workflow. Is this CL good as is, and then we can revisit what we
want the default behavior for files to be in a follow-up?

+        self.profiler_dir = tempfile.mkdtemp()
     output_file = os.path.join(self.profiler_dir, page.file_safe_name)
     is_repeating = (finder_options.page_repeat != 1 or
                     finder_options.pageset_repeat != 1)
@@ -219,6 +222,7 @@ def AddCommandLineArgs(parser):
 
 def ProcessCommandLineArgs(parser, args):
   page_filter.PageFilter.ProcessCommandLineArgs(parser, args)
+  results_options.ProcessCommandLineArgs(parser, args)
 
   # Page set options
   if args.pageset_shuffle_order_file and not args.pageset_shuffle: