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..021c6ee5c0a2dea983fded493bce867a482122aa 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,21 +175,7 @@ class PerfTryJobApi(recipe_api.RecipeApi):
 
     return update_step
 
-  def _compile(self, name, mastername, buildername, update_step, bot_db):
-    """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(
-        mastername, buildername)
-    compile_targets = self.m.chromium_tests.get_compile_targets(
-        bot_config, bot_db, tests=[])
-    if self.m.chromium.c.TARGET_PLATFORM == 'android':
-      self.m.chromium_android.clean_local_files()
-      compile_targets = None
-    else:
-      # Removes any chrome temporary files or build.dead directories.
-      self.m.chromium.cleanup_temp()
-
-    if 'With Patch' in name:
+    if 'With Patch' in name:  # pragma: no cover
       # We've had some cases where a stale build directory was used on perf

[RobertoCN, 2016/07/14 22:12:54]

This removal is incomplete. probably a rebase artifact?

[Ziqi Xiong, 2016/07/14 22:43:59]

Done.

       # try job leading to unwanted cache and temp data. The best way to
       # ensure the old build directory is removed before doing any