Telemetry: Allow running only a subset of pages.

tools/telemetry/telemetry/browser_options.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 import optparse
 import sys
 import shlex
 import logging
 import copy
 
 from telemetry import browser_finder
@@ skipping 85 matching lines @@
     group.add_option('--pageset-repeat', dest='pageset_repeat', default=1,
         help='Number of times to repeat the entire pageset ' +
         'before finishing.')
     group.add_option('--test-shuffle', action='store_true', dest='test_shuffle',
         help='Shuffle the order of pages within a pageset.')
     group.add_option('--test-shuffle-order-file',
         dest='test_shuffle_order_file', default=None,
         help='Filename of an output of a previously run test on the current ' +
         'pageset. The tests will run in the same order again, overriding ' +
         'what is specified by --page-repeat and --pageset-repeat.')
+    group.add_option('--page-filter', dest='page_filter',
+        help='Run only pages whose URLs match the given filter regexp.')

[nduca, 2013/01/09 21:47:19]

should we validate that the regex compiles down in the parse_args function lower
down?

[marja, 2013/01/10 13:39:59]

I added checks to _ShuffleAndFilterPageSet, since it already checks for the
--test-shuffle params, and raises if there is a problem. So it seemed to fit to
add this check there too.

     parser.add_option_group(group)
 
     # Debugging options
     group = optparse.OptionGroup(parser, 'When things go wrong')
     group.add_option(
       '--trace-dir', dest='trace_dir', default=None,
       help='Record traces and store them in this directory.')
     group.add_option(
       '-v', '--verbose', action='count', dest='verbosity',
       help='Increase verbosity level (repeat as needed)')
@@ skipping 31 matching lines @@
           self.extra_browser_args_as_string) # pylint: disable=E1101
         self.extra_browser_args.extend(tmp)
         delattr(self, 'extra_browser_args_as_string')
       return ret
     parser.parse_args = ParseArgs
     return parser
 
   def AppendExtraBrowserArg(self, arg):
     if arg not in self.extra_browser_args:
       self.extra_browser_args.append(arg)