Fix compile on the Blimp Client Engine Integration bot

scripts/slave/recipes/blimp/integration.py

Index: scripts/slave/recipes/blimp/integration.py
diff --git a/scripts/slave/recipes/blimp/integration.py b/scripts/slave/recipes/blimp/integration.py
index a07e7ba2129d421e05bec0ea525ac1638ae83c68..9b93fa8f47853dc550ed2a0b1df80af03b0dc8db 100644
--- a/scripts/slave/recipes/blimp/integration.py
+++ b/scripts/slave/recipes/blimp/integration.py
@@ -39,11 +39,10 @@ BUILDERS = freeze({
 REPO_URL = 'https://chromium.googlesource.com/chromium/src.git'
 
 def RunSteps(api):
-  android_build = api.path['slave_build'].join('src', 'out-android', 'Debug')
-  linux_build = api.path['slave_build'].join('src', 'out-linux', 'Debug')
   mastername = api.properties.get('mastername')
   buildername = api.properties.get('buildername')
   builder = BUILDERS[mastername][buildername]
+  api.chromium.set_config('chromium')
   api.chromium_android.configure_from_properties(
       builder['config'],
       REPO_NAME='src',
@@ -51,7 +50,6 @@ def RunSteps(api):
       INTERNAL=False,
       BUILD_CONFIG=builder['target'])
 
-  api.chromium.set_config('chromium')
   api.gclient.set_config(builder['gclient_config'])
   api.gclient.apply_config(builder['gclient_apply_config'])
   api.bot_update.ensure_checkout()
@@ -59,6 +57,13 @@ def RunSteps(api):
   api.chromium_android.clean_local_files()
   api.chromium.runhooks()
 
+  android_build = api.path['checkout'].join(
+                      'out-android',
+                      api.chromium.c.build_config_fs)
+  linux_build = api.path['checkout'].join(
+                    'out-linux',
+                    api.chromium.c.build_config_fs)

[mikecase (-- gone --), 2016/10/14 19:04:14]

Indenting should be either ....


linux_build = api.path['checkout'].join(
    'out-linux', api.chromium.c.build_config_fs)


OR

linux_build = api.path['checkout'].join('out-linux',
                                        api.chromium.c.build_config_fs)


*I prefer the first.

[shenghuazhang, 2016/10/14 19:27:23]

I see. Will do with the first one. Second one is possible to have 80 characters
problem.

+
   api.chromium.run_mb(mastername=mastername,
                       buildername=buildername,
                       build_dir=linux_build,
@@ -77,7 +82,7 @@ def GenTests(api):
   sanitize = lambda s: ''.join(c if c.isalnum() else '_' for c in s)
 
   yield (
-      api.test('%s_test_pass' % sanitize('Blimp Android Tester')) +
+      api.test('%s_test_pass' % sanitize('Blimp Client Engine Integration')) +
       api.properties.generic(
         buildername='Blimp Client Engine Integration',
         mastername='chromium.fyi')