Code coverage improvements....

build/common.gypi

 # Copyright (c) 2009 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.
 
 {
   'variables': {
     # .gyp files should set chromium_code to 1 if they build Chromium-specific
     # code, as opposed to external code.  This variable is used to control
     # such things as the set of warnings to enable, and whether warnings are
     # treated as errors.
@@ skipping 54 matching lines @@
               'GCC_GENERATE_TEST_COVERAGE_FILES': 'YES',
             },
             # Add -lgcov for executables, not for static_libraries.
             # This is a delayed conditional.
             'target_conditions': [
               ['_type=="executable"', {
                 'xcode_settings': { 'OTHER_LDFLAGS': [ '-lgcov' ] },
               }],
             ],
           }],
-          # TODO(jrg): complete this work once Linux transitions to gyp.
-          # This is untested (--> likely doesn't work).
+          # Linux gyp (into scons) doesn't like target_conditions?
+          # TODO(???): track down why 'target_conditions' doesn't work
+          # on Linux gyp into scons like it does on Mac gyp into xcodeproj.
           ['OS=="linux"', {
             'cflags': [ '-ftest-coverage',
                         '-fprofile-arcs' ],
-            'target_conditions': [
-              ['_type=="executable"', {
-                'link_settings': { 'libraries': [ '-lgcov' ] },
-              }],
-            ],
+            'link_settings': { 'libraries': [ '-lgcov' ] },
           }],
         ]},
       # TODO(jrg): options for code coverage on Windows
       ],
     ],
     'default_configuration': 'Debug',
     'configurations': {
       'Debug': {
         'conditions': [
           [ 'OS=="mac"', {
@@ skipping 338 matching lines @@
     # settings in target dicts.  SYMROOT is a special case, because many other
     # Xcode variables depend on it, including variables such as
     # PROJECT_DERIVED_FILE_DIR.  When a source group corresponding to something
     # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
     # files to appear (when present) in the UI as actual files and not red
     # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
     # and therefore SYMROOT, needs to be set at the project level.
     'SYMROOT': '<(DEPTH)/xcodebuild',
   },
 }