New bisect for crbug.com/706941. [AA_AutofillCreditCardSigninPromo]

tools/perf/core/trybot_command.py

 # Copyright 2015 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 argparse
 import base64
 import gzip
 import io
 import json
 import logging
@@ skipping 72 matching lines @@
         'src': 'src/third_party/angle',
         'url': 'https://chromium.googlesource.com/angle/angle.git',
     },
     'catapult': {
         'src': 'src/third_party/catapult',
         'url': ('https://chromium.googlesource.com/external/github.com/'
                 'catapult-project/catapult.git')
     }
 }
 
+
 _MILO_MASTER_ENDPOINT = ('https://luci-milo.appspot.com/prpc/milo.Buildbot/'
                         'GetCompressedMasterJSON')
 
 _MILO_RESPONSE_PREFIX = ')]}\'\n'
 
 
 assert not set(DEFAULT_TRYBOTS) & set(EXCLUDED_BOTS), (
     'A trybot cannot present in both Default as well as Excluded bots lists.')
 
 
@@ skipping 274 matching lines @@
      --repo_path=$HOME/mydir/v8/v8 or --repo_path=v8/v8"""),
         metavar='<repo path>')
     parser.add_argument(
         '--deps_revision', type=str, default=None,
         help=('specify DEPS revision to modify DEPS entry in Chromium to a '
               'certain pushed revision.\n'
               'This revision overrides value in DEPS on TOT Chromium for the '
               'repo specified in --repo_path.\nIt is applied for both with and '
               'wihout patch.'),
         metavar='<deps revision>')
+    parser.add_argument(
+        '--tot_revision', type=str, default=None)
 
   def Run(self, options, extra_args=None):
     """Sends a tryjob to a perf trybot.
 
     This creates a branch, telemetry-tryjob, switches to that branch, edits
     the bisect config, commits it, uploads the CL to rietveld, and runs a
     tryjob on the given bot.
     """
     if extra_args is None:
       extra_args = []
     self._InitializeBuilderNames(options.trybot)
 
     return self._AttemptTryjob(options, extra_args)
 
-  def _GetPerfConfig(self, bot_platform, arguments):
+  def _GetPerfConfig(self, bot_platform, arguments, tot_revision):
     """Generates the perf config for try job.
 
     Args:
       bot_platform: Name of the platform to be generated.
       arguments: Command line arguments.
 
     Returns:
       A dictionary with perf config parameters.
     """
     # To make sure that we don't mutate the original args
@@ skipping 23 matching lines @@
       arguments.insert(1, '--browser=release')
 
     command = ' '.join(arguments)
 
     return {
         'command': command,
         'repeat_count': '1',
         'max_time_minutes': '120',
         'truncate_percent': '0',
         'target_arch': target_arch,
+        'good_revision': tot_revision,
+        'bad_revision': tot_revision,
     }
 
   def _GetRepoAndBranchName(self, repo_path):
     """Gets the repository name and working branch name.
 
     Args:
       repo_path: Path to the repository.
 
     Returns:
       Repository name and branch name as tuple.
@@ skipping 138 matching lines @@
       print ('\nRunning try job....\nview progress here %s.'
              '\n\tRepo Name: %s\n\tPath: %s\n\tBranch: %s' % (
                  rietveld_url, repo_name, repo_path, branch_name))
 
       for bot_platform in self._builder_names:
         if not self._builder_names[bot_platform]:
           logging.warning('No builder is found for %s', bot_platform)
           continue
         try:
           arguments = [options.benchmark_name] + extra_args
-          self._RunTryJob(bot_platform, arguments, deps_override)
+          self._RunTryJob(
+              bot_platform, arguments, deps_override, options.tot_revision)
         # Even if git cl try throws TrybotError exception for any platform,
         # keep sending try jobs to other platforms.
         except TrybotError, err:
           print err
     except TrybotError, error:
       print error
       return 1
     finally:
       # Restore to original working directory.
       os.chdir(original_workdir)
     return 0
 
-  def _RunTryJob(self, bot_platform, arguments, deps_override):
+  def _RunTryJob(self, bot_platform, arguments, deps_override, tot_revision):
     """Executes perf try job with benchmark test properties.
 
     Args:
       bot_platform: Name of the platform to be generated.
       arguments: Command line arguments.
       deps_override: DEPS revision if needs to be overridden.
 
     Raises:
       TrybotError: When trybot fails to upload CL or run git try.
     """
-    config = self._GetPerfConfig(bot_platform, arguments)
+    config = self._GetPerfConfig(bot_platform, arguments, tot_revision)
 
     # Generate git try command for available bots.
     git_try_command = ['cl', 'try', '-m', 'tryserver.chromium.perf']
 
      # Add Perf Test config to git try --properties arg.
     git_try_command.extend(['-p', 'perf_try_config=%s' % json.dumps(config)])
+    git_try_command.extend(['-p', 'parent_got_revision=%s' % tot_revision])
 
     error_msg_on_fail = 'Could not try CL for %s' % bot_platform
     # Add deps overrides to git try --properties arg.
     if deps_override:
       git_try_command.extend([
           '-p', 'deps_revision_overrides=%s' % json.dumps(deps_override)])
       error_msg_on_fail += ' with DEPS override (%s)' % deps_override
     for bot in self._builder_names[bot_platform]:
       git_try_command.extend(['-b', bot])
 
     RunGit(git_try_command, error_msg_on_fail)
     print 'Perf Try job sent to rietveld for %s platform.' % bot_platform