V8 Buildbot: Properly set revision property for triggered tryjobs.

scripts/slave/recipe_modules/v8/api.py

 # Copyright 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.
 
 import argparse
 import collections
 import datetime
 import math
 import re
 import urllib
@@ skipping 985 matching lines @@
       'merge perf results' + kwargs.pop('suffix', ''),
       self.resource('merge_perf_result_maps.py'),
       [self.m.json.input(results_file_map)],
       stdout=self.m.json.output(),
     ).stdout
 
   def maybe_trigger(self, **additional_properties):
     triggers = self.bot_config.get('triggers')
     if triggers:
       properties = {
-        'revision': self.revision,
         'parent_got_revision': self.revision,
         'parent_got_revision_cp': self.revision_cp,
       }
       if self.m.tryserver.is_tryserver:
         properties.update(
           category=self.m.properties['category'],
           issue=self.m.properties['issue'],
           master=str(self.m.properties['master']),
           patch_project=str(self.m.properties['patch_project']),
           patch_storage=str(self.m.properties['patch_storage']),
           patchset=str(self.m.properties['patchset']),
           reason=str(self.m.properties['reason']),
           requester=str(self.m.properties['requester']),
+          # On tryservers, set revision to the same as on the current bot,
+          # as CQ expects builders and testers to match the revision field.
+          revision=str(self.m.properties.get('revision', 'HEAD')),
           rietveld=str(self.m.properties['rietveld']),
         )
+      else:
+        # On non-tryservers, we can set the revision to whatever the
+        # triggering builder checked out.
+        properties['revision'] = self.revision
 
       # TODO(machenbach): Also set meaningful buildbucket tags of triggering
       # parent.
 
       swarm_hashes = self.m.isolate.isolated_tests
       if swarm_hashes:
         properties['swarm_hashes'] = swarm_hashes
       properties.update(**additional_properties)
       self.m.trigger(*[{
         'builder_name': builder_name,
@@ skipping 119 matching lines @@
   def report_culprits(self, culprit_range):
     assert culprit_range
     if len(culprit_range) > 1:
       text = 'Suspecting multiple commits'
     else:
       text = 'Suspecting %s' % culprit_range[0][:8]
 
     step_result = self.m.step(text, cmd=None)
     for culprit in culprit_range:
       step_result.presentation.links[culprit[:8]] = COMMIT_TEMPLATE % culprit