Clamp legacy bisect param max_time_minutes to be in the range [1, 60].

tools/run-bisect-perf-regression.py

 #!/usr/bin/env python
 # Copyright (c) 2013 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.
 
 """Run Performance Test Bisect Tool
 
 This script is used by a try bot to run the bisect script with the parameters
 specified in the bisect config file. It checks out a copy of the depot in
 a subdirectory 'bisect' of the working directory provided, annd runs the
@@ skipping 181 matching lines @@
   opts_dict['command'] = config['command']
   opts_dict['metric'] = config.get('metric')
 
   if config['repeat_count']:
     opts_dict['repeat_test_count'] = int(config['repeat_count'])
 
   if config['truncate_percent']:
     opts_dict['truncate_percent'] = int(config['truncate_percent'])
 
   if config['max_time_minutes']:
-    opts_dict['max_time_minutes'] = int(config['max_time_minutes'])
+    opts_dict['max_time_minutes'] = _Clamp(
+        int(config['max_time_minutes']), low=1, high=60)
 
   if config.has_key('use_goma'):
     opts_dict['use_goma'] = config['use_goma']
   if config.has_key('goma_dir'):
     opts_dict['goma_dir'] = config['goma_dir']
 
   if config.has_key('improvement_direction'):
     opts_dict['improvement_direction'] = int(config['improvement_direction'])
 
   if config.has_key('target_arch'):
@@ skipping 18 matching lines @@
     if 'android-chromium' in config['command']:
       opts_dict['target_platform'] = 'android'
     elif 'android-chrome' in config['command']:
       opts_dict['target_platform'] = 'android-chrome'
     else:
       opts_dict['target_platform'] = 'android'
 
   return bisect_perf_regression.BisectOptions.FromDict(opts_dict)
 
 
+def _Clamp(n, low, high):
+  """Clamps a value to a range."""
+  return min(high, max(low, n))
+
+
 def _ParseCloudLinksFromOutput(output):
   html_results_pattern = re.compile(
       r'\s(?P<VALUES>http://storage.googleapis.com/' +
           'chromium-telemetry/html-results/results-[a-z0-9-_]+)\s',
       re.MULTILINE)
   profiler_pattern = re.compile(
       r'\s(?P<VALUES>https://console.developers.google.com/' +
           'm/cloudstorage/b/[a-z-]+/o/profiler-[a-z0-9-_.]+)\s',
       re.MULTILINE)
 
@@ skipping 611 matching lines @@
     return _SetupAndRunPerformanceTest(
         config={}, path_to_goma=opts.path_to_goma, is_cq_tryjob=True)
 
   print ('Error: Could not load config file. Double check your changes to '
          'auto_bisect/bisect.cfg or run-perf-test.cfg for syntax errors.\n')
   return 1
 
 
 if __name__ == '__main__':
   sys.exit(main())