Build delegation for CQ jobs

scripts/slave/recipe_modules/perf_try/api.py

Index: scripts/slave/recipe_modules/perf_try/api.py
diff --git a/scripts/slave/recipe_modules/perf_try/api.py b/scripts/slave/recipe_modules/perf_try/api.py
index d5cd4f1cefca5cfc4635799f88b0eff07205d70a..4cdb1ec0c8cf899b952d29802f5d98fac5d651ef 100644
--- a/scripts/slave/recipe_modules/perf_try/api.py
+++ b/scripts/slave/recipe_modules/perf_try/api.py
@@ -112,8 +112,14 @@ class PerfTryJobApi(recipe_api.RecipeApi):
           'There are no modifications to Telemetry benchmarks,'
           ' aborting the try job.')
       return
-    self._compile('With Patch', self.m.properties['mastername'],
-                  self.m.properties['buildername'], update_step, bot_db)
+    revision_hash = self.m.properties.get('parent_got_revision')
+    update_step = self._checkout_revision(update_step, bot_db, revision_hash)
+    if update_step.presentation.properties:
+      revision_hash = update_step.presentation.properties['got_revision']
+    revision = build_state.BuildState(self, revision_hash, True)
+    revision.request_build()
+    revision.wait_for()
+    revision.download_build(update_step, bot_db)
 
     if self.m.chromium.c.TARGET_PLATFORM == 'android':
       self.m.chromium_android.adb_install_apk('ChromePublic.apk')
@@ -169,7 +175,7 @@ class PerfTryJobApi(recipe_api.RecipeApi):
 
     return update_step
 
-  def _compile(self, name, mastername, buildername, update_step, bot_db):
+  def _compile(self, name, mastername, buildername, update_step, bot_db): # pragma: no cover

[RobertoCN, 2016/07/13 23:42:00]

Should we remove this method? cq was the last user of it, right?

[Ziqi Xiong, 2016/07/14 00:15:33]

Done.

     """Runs compile and related steps for given builder."""
     # TODO(phajdan.jr): Change this method to take bot_config as parameter.
     bot_config = self.m.chromium_tests.create_bot_config_object(