Add support for component=shared_library for base_unittests_run.

build/isolate.gypi

 # Copyright (c) 2012 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.
 
 # This file is meant to be included into a target to provide a rule
 # to "build" .isolate files into a .isolated file.
 #
 # To use this, create a gyp target with the following form:
 # 'conditions': [
 #   ['test_isolation_mode != "noop"', {
@@ skipping 51 matching lines @@
         # the switch-over to running tests on Swarm is completed.
         #'<@(isolate_dependency_tracked)',
       ],
       'outputs': [
         '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).isolated',
       ],
       'action': [
         'python',
         '<(DEPTH)/tools/swarming_client/isolate.py',
         '<(test_isolation_mode)',
+        '--result', '<@(_outputs)',
+        '--isolate', '<(RULE_INPUT_PATH)',
+
         # Variables should use the -V FOO=<(FOO) form so frequent values,

[M-A Ruel, 2014/01/21 17:53:56]

Try to increase the maintainability of this file by adding more explanations and
separating the configurations per purpose.

-        # like '0' or '1', aren't stripped out by GYP.
-        # This list needs to be kept in sync with the cmd line options
-        # in src/build/android/pylib/gtest/setup.py.
+        # like '0' or '1', aren't stripped out by GYP. Run 'isolate.py help' for
+        # more details.
+        #
+        # This list needs to be kept in sync with the cmd line options
+        # in src/build/android/pylib/gtest/setup.py.
+
+        # Path variables are used to replace file paths when loading a .isolate
+        # file
         '--path-variable', 'PRODUCT_DIR', '<(PRODUCT_DIR) ',
+
+        # Extra variables are replaced on the 'command' entry and on paths in
+        # the .isolate file but are not considered relative paths.
+        '--extra-variable', 'version_full=<(version_full)',
+
         '--config-variable', 'OS=<(OS)',
+        '--config-variable', 'component=<(component)',
         # TODO(kbr): move this to chrome_tests.gypi:gles2_conform_tests_run
         # once support for user-defined config variables is added.
         '--config-variable',
-        'internal_gles2_conform_tests=<(internal_gles2_conform_tests)',
+          'internal_gles2_conform_tests=<(internal_gles2_conform_tests)',
         '--config-variable', 'icu_use_data_file_flag=<(icu_use_data_file_flag)',
-        '--extra-variable', 'version_full=<(version_full)',
-        '--result', '<@(_outputs)',
-        '--isolate', '<(RULE_INPUT_PATH)',
+        '--config-variable', 'use_openssl=<(use_openssl)',
+        # TODO(maruel): This variable is defined in build/linux/system.gyp which
+        # is not available on other OSes.
+        #'--config-variable', 'use_system_ssl=<(use_system_ssl)',

[Ryan Sleevi, 2014/01/21 19:43:50]

Filed http://crbug.com/336511 - this isn't something we really want to support
:)

       ],
       'conditions': [
         # Note: When gyp merges lists, it appends them to the old value.
         ['OS=="mac"', {
           # <(mac_product_name) can contain a space, so don't use FOO=<(FOO)
           # form.
           'action': [
             '--extra-variable', 'mac_product_name', '<(mac_product_name)',
           ],
         }],
@@ skipping 11 matching lines @@
         ['test_isolation_fail_on_missing == 0', {
             'action': ['--ignore_broken_items'],
           },
         ],
       ],
 
       'msvs_cygwin_shell': 0,
     },
   ],
 }