WebRTC Android APK trybot recipe

scripts/slave/recipe_modules/chromium/config.py

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 from slave.recipe_config import config_item_context, ConfigGroup
 from slave.recipe_config import Dict, Single, Static, Set, BadConf
 from slave.recipe_config_types import Path
 
 # Because of the way that we use decorators, pylint can't figure out the proper
 # type signature of functions annotated with the @config_ctx decorator.
@@ skipping 201 matching lines @@
 def chromium_clang(c):
   c.compile_py.default_targets = ['All', 'chromium_builder_tests']
 
 @config_ctx(includes=['chromium'])
 def blink(c):
   c.compile_py.default_targets = ['blink_tests']
 
 @config_ctx(includes=['chromium_clang'])
 def blink_clang(c):
   c.compile_py.default_targets = ['blink_tests']
+
+@config_ctx(includes=['ninja', 'static_library', 'default_compiler', 'goma'])
+def android(c):
+  c.gyp_env.GYP_CROSSCOMPILE = 1

[iannucci, 2013/11/21 00:33:59]

Should we just set this in BASE() any time HOST_PLATFORM != TARGET_PLATFORM?

[kjellander_chromium, 2013/11/22 12:49:09]

That is an excellent idea. I'm not 100% sure it's always needed since I don't
know where GYP_CROSSCOMPILE is used, but let's do it like that for now.
Looking at the updated config files, there were many different configurations
that now got this flag set. I'm not sure that's desired?

+  gyp_defs = c.gyp_env.GYP_DEFINES
+  gyp_defs['fastbuild'] = 1
+  gyp_defs['OS'] = c.TARGET_PLATFORM
+  gyp_defs['host_os'] = c.HOST_PLATFORM
+  gyp_defs['gcc_version'] = 46
+  gyp_defs['order_text_section'] = Path(
+    '[CHECKOUT]', 'orderfiles', 'orderfile.out')
+  gyp_defs['target_arch'] = c.TARGET_ARCH

[iannucci, 2013/11/21 00:33:59]

I think target_arch is now automatically handled for everything. Try rebasing on
HEAD.

[kjellander_chromium, 2013/11/22 12:49:09]

Ah, that's great.