Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(343)

Unified Diff: scripts/slave/recipe_modules/pgo/api.py

Issue 2390753005: Double Builds for Blimp Integration Test Infrastructure on FYI bot (Closed)
Patch Set: Dirk comment Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/pgo/api.py
diff --git a/scripts/slave/recipe_modules/pgo/api.py b/scripts/slave/recipe_modules/pgo/api.py
index f938ab1401aec28385f767b7702b56e10f5c91aa..bc5158699caa9d70e5d01c8d9e64e7802d2b68db 100644
--- a/scripts/slave/recipe_modules/pgo/api.py
+++ b/scripts/slave/recipe_modules/pgo/api.py
@@ -56,7 +56,7 @@ class PGOApi(recipe_api.RecipeApi):
self.m.properties['mastername'],
self.m.properties['buildername'],
use_goma=False,
- phase=1)
+ phase='pgo_phase_1')
Dirk Pranke 2016/10/07 19:03:04 See my comments in https://codereview.chromium.org
# Remove the profile files from the previous builds.
self.m.file.rmwildcard('*.pg[cd]', str(self.m.chromium.output_dir))
self.m.chromium.compile(name='Compile: Instrumentation phase.')
@@ -95,7 +95,7 @@ class PGOApi(recipe_api.RecipeApi):
self.m.properties['mastername'],
self.m.properties['buildername'],
use_goma=False,
- phase=2)
+ phase='pgo_phase_2')
self.m.chromium.compile(name='Compile: Optimization phase.')
def compile_pgo(self, bot_config):

Powered by Google App Engine
This is Rietveld 408576698