optimize branding - use branding_path_component variable for defining pranding specific paths

chrome/chrome_installer.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.
 
 {
   'variables': {
     'lastchange_path': '../build/util/LASTCHANGE',
     'libpeer_target_type%': 'static_library',
-    # 'branding_dir' is set in the 'conditions' section at the bottom.
+    'branding_dir': 'app/theme/<(branding_path_component)',
+    'branding_dir_100': 'app/theme/default_100_percent/<(branding_path_component)',
   },
   'conditions': [
     ['OS=="win"', {
       'targets': [
         {
           'target_name': 'gcapi_dll',
           'type': 'loadable_module',
           'dependencies': [
             'gcapi_lib',
           ],
@@ skipping 131 matching lines @@
         },
         {
           # GN version: //chrome/installer/util:strings
           'target_name': 'installer_util_strings',
           'type': 'none',
           'actions': [
             {
               'action_name': 'installer_util_strings',
               'variables': {
                 'create_string_rc_py': 'installer/util/prebuild/create_string_rc.py',
+                'brand_strings': '<(branding_path_component)_strings',
               },
-              'conditions': [
-                ['branding=="Chrome"', {
-                  'variables': {
-                    'brand_strings': 'google_chrome_strings',
-                  },
-                }, {
-                  'variables': {
-                    'brand_strings': 'chromium_strings',
-                  },
-                }],
-              ],
+
               'inputs': [
                 '<(create_string_rc_py)',
                 'app/<(brand_strings).grd',
               ],
               'outputs': [
                 '<(SHARED_INTERMEDIATE_DIR)/installer_util_strings/installer_util_strings.h',
                 '<(SHARED_INTERMEDIATE_DIR)/installer_util_strings/installer_util_strings.rc',
               ],
               'action': ['python',
                          '<(create_string_rc_py)',
@@ skipping 110 matching lines @@
                 '-f', '<(lastchange_path)',
                 '-f', '<(branding_dir)/BRANDING',
                 '<(template_input_path)',
                 '<@(_outputs)',
               ],
               'process_outputs_as_sources': 1,
               'message': 'Generating version information'
             },
           ],
           'conditions': [
-            # TODO(mark):  <(branding_dir) should be defined by the
-            # global condition block at the bottom of the file, but
-            # this doesn't work due to the following issue:
-            #
-            #   http://code.google.com/p/gyp/issues/detail?id=22
-            #
-            # Remove this block once the above issue is fixed.
-            [ 'branding == "Chrome"', {
-              'variables': {
-                 'branding_dir': 'app/theme/google_chrome',
-                 'branding_dir_100': 'app/theme/default_100_percent/google_chrome',
-              },
-            }, { # else branding!="Chrome"
-              'variables': {
-                 'branding_dir': 'app/theme/chromium',
-                 'branding_dir_100': 'app/theme/default_100_percent/chromium',
-              },
-            }],
             ['target_arch=="ia32"', {
               'msvs_settings': {
                 'VCCLCompilerTool': {
                   'EnableEnhancedInstructionSet': '4',  # NoExtensions
                 },
               },
             }],
           ],
         },
         {
@@ skipping 753 matching lines @@
           ],
           'include_dirs': [
             '..',
           ],
           'sources': [
             'installer/gcapi_mac/gcapi_example_client.mm',
           ],
         },
       ],  # targets
     }],  # OS=="mac"
-    [ 'branding == "Chrome"', {
-      'variables': {
-         'branding_dir': 'app/theme/google_chrome',
-         'branding_dir_100': 'app/theme/default_100_percent/google_chrome',
-      },
-    }, { # else branding!="Chrome"
-      'variables': {
-         'branding_dir': 'app/theme/chromium',
-         'branding_dir_100': 'app/theme/default_100_percent/chromium',
-      },
-    }],
   ],
 }