First version of the PGO recipe.

scripts/slave/recipe_modules/chromium/config.py

Index: scripts/slave/recipe_modules/chromium/config.py
diff --git a/scripts/slave/recipe_modules/chromium/config.py b/scripts/slave/recipe_modules/chromium/config.py
index 905b77543fd9bde73be1ba3fdf290e8a16e3f94a..26db9153d85bfcf4172a99428338f40daeccb6bf 100644
--- a/scripts/slave/recipe_modules/chromium/config.py
+++ b/scripts/slave/recipe_modules/chromium/config.py
@@ -413,3 +413,21 @@ def _android_common(c):
 def codesearch(c):
   gyp_defs = c.gyp_env.GYP_DEFINES
   gyp_defs['fastbuild'] = 1
+
+
+@config_ctx()

[iannucci, 2014/06/19 21:37:17]

you should probably depend on a compiler (ninja), 'default_compiler', and either
'shared_library' or 'static_library'

[Sébastien Marchand, 2014/07/03 19:13:17]

Done.

+def chrome_pgo_base(c):
+  c.gyp_env.GYP_DEFINES['buildtype'] = 'Official'
+  c.gyp_env.GYP_DEFINES['optimize'] = 'max'
+  c.gyp_env.GYP_DEFINES['use_goma'] = 0
+  c.gyp_env.GYP_DEFINES['fastbuild'] = 0

[iannucci, 2014/06/19 21:37:17]

I would set this by doing

fastbuild(c, invert=True)

This has some additional error-checking advantages (for example, if someone
later tried to apply the fastbuild config, it would be an error, instead of just
turning it on).

I see some others have done this wrong already in this file :)

[Sébastien Marchand, 2014/07/03 19:13:17]

Done.

+  c.compile_py.default_targets = ['chrome']
+
+#### 'Full' configurations
+@config_ctx(includes=['chrome_pgo_base'])
+def chrome_pgo_instrument(c):
+  c.gyp_env.GYP_DEFINES['chrome_pgo_phase'] = 1
+
+@config_ctx(includes=['chrome_pgo_base'])
+def chrome_pgo_optimize(c):
+  c.gyp_env.GYP_DEFINES['chrome_pgo_phase'] = 2