Make gyp target_arch setting native to chromium/config.py

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 101 matching lines @@
 
   if c.HOST_BITS < c.TARGET_BITS:
     raise BadConf('host bits < targ bits')
 
   c.build_config_fs = c.BUILD_CONFIG
   if c.HOST_PLATFORM == 'win':
     if c.TARGET_BITS == 64:
       # Windows requires 64-bit builds to be in <dir>_x64.
       c.build_config_fs = c.BUILD_CONFIG + '_x64'
       c.gyp_env.GYP_MSVS_VERSION = '2012'
-      c.gyp_env.GYP_DEFINES['target_arch'] = 'x64'
     else:
       c.gyp_env.GYP_MSVS_VERSION = '2010'
 
+  gyp_arch = {
+    ('intel', 32): 'ia32',
+    ('intel', 64): 'x64',
+    ('arm',   32): 'arm',
+    ('arm',   64): 'arm',
+    ('mips',  32): 'mips',

[Nico, 2014/02/19 02:53:31]

This should be "mipsel", not "mips", right?

+    ('mips',  64): 'mips',

[Nico, 2014/02/19 02:53:31]

(and this shouldn't do anything yet as it's currently not used – but "mips" is
fairly likely not what we want here once it's going to be used.)

+  }.get((c.TARGET_ARCH, c.TARGET_BITS))
+  if gyp_arch:
+    c.gyp_env.GYP_DEFINES['target_arch'] = gyp_arch
+
   if c.BUILD_CONFIG == 'Release':
     static_library(c, final=False)
   elif c.BUILD_CONFIG == 'Debug':
     shared_library(c, final=False)
   else:  # pragma: no cover
     raise BadConf('Unknown build config "%s"' % c.BUILD_CONFIG)
 
 @config_ctx(group='builder')
 def ninja(c):
   c.gyp_env.GYP_GENERATORS.add('ninja')
@@ skipping 76 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']