Re-commit r22981 after backout at r22992

chrome/chrome.gyp

 # 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': {
     'chromium_code': 1,
     # Define the common dependencies that contain all the actual
     # Chromium functionality.  This list gets pulled in below by
     # the link of the actual chrome (or chromium) executable on
@@ skipping 47 matching lines @@
     # TODO(jcampan): move these vars to views.gyp.
     'views_unit_tests_sources': [
       '../views/view_unittest.cc',
       '../views/focus/focus_manager_unittest.cc',
     ],
     'views_unit_tests_sources_win_specific': [
       # TODO(jcampan): make the following tests work on Linux.
       '../views/controls/label_unittest.cc',
       '../views/controls/table/table_view_unittest.cc',
       '../views/grid_layout_unittest.cc',
-    ]
+    ],
+    'conditions': [
+      ['OS=="mac"', {
+        'conditions': [
+          ['branding=="Chrome"', {
+            'bundle_id': 'com.google.Chrome',
+          }, {  # else: branding!="Chrome"
+            'bundle_id': 'org.chromium.Chromium',
+          }],  # branding
+        ],  # conditions
+      }],  # OS=="mac"
+    ],  # conditions
   },
   'includes': [
     '../build/common.gypi',
   ],
   'target_defaults': {
     'sources/': [
       ['exclude', '/(cocoa|gtk|win)/'],
       ['exclude', '_(cocoa|gtk|linux|mac|posix|skia|win|views|x)(_unittest)?\\.(cc|mm?)$'],
       ['exclude', '/(gtk|win|x11)_[^/]*\\.cc$'],
     ],
@@ skipping 2927 matching lines @@
           'dependencies': [
             '../views/views.gyp:views',
           ],
         }],
         ['OS=="mac"', {
           # 'branding' is a variable defined in common.gypi
           # (e.g. "Chromium", "Chrome")
           'conditions': [
             ['branding=="Chrome"', {
               'mac_bundle_resources': ['app/theme/google_chrome/app.icns'],
-              'variables': {
-                'bundle_id': 'com.google.Chrome',
-              },
               'copies': [
                 {
-                  'destination': '<(PRODUCT_DIR)/<(mac_product_name).app/Contents/MacOS/',
-                  'files': ['../third_party/ffmpeg/binaries/chrome/libavcodec.52.dylib',
-                            '../third_party/ffmpeg/binaries/chrome/libavformat.52.dylib',
-                            '../third_party/ffmpeg/binaries/chrome/libavutil.50.dylib',
-                            '<(PRODUCT_DIR)/plugin_carbon_interpose.dylib'],
+                  'destination': '<(PRODUCT_DIR)/<(mac_product_name).app/Contents/MacOS',
+                  'files': [
+                    '../third_party/ffmpeg/binaries/chrome/libavcodec.52.dylib',
+                    '../third_party/ffmpeg/binaries/chrome/libavformat.52.dylib',
+                    '../third_party/ffmpeg/binaries/chrome/libavutil.50.dylib',
+                  ],
                 },
               ],
             }, {  # else: 'branding!="Chrome"
               'mac_bundle_resources': ['app/theme/chromium/app.icns'],
-              'variables': {
-                'bundle_id': 'org.chromium.Chromium',
-              },
               'copies': [
                 {
-                  'destination': '<(PRODUCT_DIR)/<(mac_product_name).app/Contents/MacOS/',
-                  'files': ['../third_party/ffmpeg/binaries/chromium/libavcodec.52.dylib',
-                            '../third_party/ffmpeg/binaries/chromium/libavformat.52.dylib',
-                            '../third_party/ffmpeg/binaries/chromium/libavutil.50.dylib',
-                            '<(PRODUCT_DIR)/plugin_carbon_interpose.dylib'],
+                  'destination': '<(PRODUCT_DIR)/<(mac_product_name).app/Contents/MacOS',
+                  'files': [
+                    '../third_party/ffmpeg/binaries/chromium/libavcodec.52.dylib',
+                    '../third_party/ffmpeg/binaries/chromium/libavformat.52.dylib',
+                    '../third_party/ffmpeg/binaries/chromium/libavutil.50.dylib',
+                  ],
                 },
               ],
             }],
             ['mac_breakpad==1', {
               'variables': {
                 # A real .dSYM is needed for dump_syms to operate on.
                 'mac_real_dsym': 1,
               },
               'dependencies': [
                 '../breakpad/breakpad.gyp:dump_syms',
@@ skipping 25 matching lines @@
           'product_name': '<(mac_product_name)',
           'xcode_settings': {
             # chrome/app/app-Info.plist has:
             #   CFBundleIdentifier of CHROMIUM_BUNDLE_ID
             #   CFBundleName of CHROMIUM_SHORT_NAME
             # Xcode then replaces these values with the branded values we set
             # as settings on the target.
             'CHROMIUM_BUNDLE_ID': '<(bundle_id)',
             'CHROMIUM_SHORT_NAME': '<(branding)',
           },
-          # Bring in pdfsqueeze and run it on all pdfs
+          'mac_bundle_resources': [
+            '<(PRODUCT_DIR)/<(mac_product_name) Helper.app',
+          ],
           'dependencies': [
+            'helper_app',
+            # Bring in pdfsqueeze and run it on all pdfs
             '../build/temp_gyp/pdfsqueeze.gyp:pdfsqueeze',
-            'interpose_dependency_shim',
           ],
           'rules': [
             {
               'rule_name': 'pdfsqueeze',
               'extension': 'pdf',
               'inputs': [
                 '<(PRODUCT_DIR)/pdfsqueeze',
               ],
               'outputs': [
                 '<(INTERMEDIATE_DIR)/pdfsqueeze/<(RULE_INPUT_ROOT).pdf',
               ],
               'action': ['<(PRODUCT_DIR)/pdfsqueeze', '<(RULE_INPUT_PATH)', '<@(_outputs)'],
               'message': 'Running pdfsqueeze on <(RULE_INPUT_PATH)',
             },
           ],
           'copies': [
             {
               'destination': '<(PRODUCT_DIR)/<(mac_product_name).app/Contents/Frameworks',
               'files': ['<(PRODUCT_DIR)/<(mac_product_name) Framework.framework'],
             },
+            {
+              # Copy web inspector resources to the Contents/Resources folder.
+              'destination': '<(PRODUCT_DIR)/<(mac_product_name).app/Contents/Resources',
+              'files': ['<(PRODUCT_DIR)/resources/inspector/'],
+            },
+          ],
+          'postbuilds': [
+            {
+              # Modify the Info.plist as needed.  The script explains why this
+              # is needed.  This is also done in the helper_app target.
+              'postbuild_name': 'Tweak Info.plist',
+              'action': ['<(DEPTH)/build/mac/tweak_app_infoplist',
+                         '-b<(mac_breakpad)',
+                         '-k<(mac_keystone)',
+                         '-s1',  # Include Subversion information
+                         '<(branding)'],
+            },
           ],
         }, { # else: OS != "mac"
           'conditions': [
             ['branding=="Chrome"', {
               'product_name': 'chrome'
             }, {  # else: Branding!="Chrome"
               # TODO:  change to:
               #   'product_name': 'chromium'
               # whenever we convert the rest of the infrastructure
               # (buildbots etc.) to use "gyp -Dbranding=Chrome".
               # NOTE: chrome/app/theme/chromium/BRANDING and
               # chrome/app/theme/google_chrome/BRANDING have the short names,
               # etc.; should we try to extract from there instead?
               'product_name': 'chrome'
             }],
           ],
         }],
-        ['OS=="mac"', {
-          'actions': [
-            {
-              # Mac adds an action to modify the Info.plist to meet our needs
-              # (see the script for why this is done).
-              'action_name': 'tweak_app_infoplist',
-              # We don't list any inputs or outputs because we always want
-              # the script to run.  Why?  Because it does thinks like record
-              # the svn revision into the info.plist, so there is no file to
-              # depend on that will change when ever that changes.
-              'inputs': [],
-              'outputs': [],
-              'action': ['<(DEPTH)/build/mac/tweak_app_infoplist',
-                         '-b<(mac_breakpad)',
-                         '-k<(mac_keystone)',
-                         '<(branding)'],
-            },
-          ],
-        }],
-        ['OS=="mac"', {
-          # Copy web inspector resources to the Contents/Resources folder.
-          'copies': [
-            {
-              'destination': '<(PRODUCT_DIR)/<(mac_product_name).app/Contents/Resources',
-              'files': ['<(PRODUCT_DIR)/resources/inspector/'],
-            },
-          ],
-        }],
         ['OS=="linux"', {
           'conditions': [
             ['branding=="Chrome"', {
               'dependencies': [
                 'installer/installer.gyp:installer_util',
               ],
             }],
           ],
         }],
         ['OS=="mac" or OS=="win"', {
@@ skipping 1229 matching lines @@
               ],
               'dependencies': [
                 '../build/util/support/support.gyp:*',
               ],
 
               # For now, don't put any resources into the framework.  Exclude
               # them all and push them into the bundle resources of the sole
               # app bundle, the only dependent of this target.
               # TODO(mark): Fix.
               'mac_bundle_resources/': [
-                ['exclude', ''],
+                ['exclude', '.*'],
               ],
               'direct_dependent_settings': {
                 'mac_bundle_resources': [
                   '../third_party/WebKit/WebCore/Resources/aliasCursor.png',
                   '../third_party/WebKit/WebCore/Resources/cellCursor.png',
                   '../third_party/WebKit/WebCore/Resources/contextMenuCursor.png',
                   '../third_party/WebKit/WebCore/Resources/copyCursor.png',
                   '../third_party/WebKit/WebCore/Resources/crossHairCursor.png',
                   '../third_party/WebKit/WebCore/Resources/eastResizeCursor.png',
                   '../third_party/WebKit/WebCore/Resources/eastWestResizeCursor.png',
@@ skipping 58 matching lines @@
         }, {
           'variables': {
             'linux_breakpad%': 0,
           },
         }],
       ],
     }],
     ['OS=="mac"',
       { 'targets': [
         {
+          'target_name': 'helper_app',
+          'type': 'executable',
+          'product_name': '<(mac_product_name) Helper',
+          'mac_bundle': 1,
+          'dependencies': [
+            'chrome_dll',
+            'interpose_dependency_shim',
+          ],
+          'sources': [
+            # chrome_exe_main.mm's main() is the entry point for the "chrome"
+            # (browser app) target.  All it does is jump to chrome_dll's
+            # ChromeMain.  This is appropriate for helper processes too,
+            # because the logic to discriminate between process types at run
+            # time is actually directed by the --type command line argument
+            # processed by ChromeMain.  Sharing chrome_exe_main.mm with the
+            # browser app will suffice for now.
+            'app/chrome_exe_main.mm',
+            'app/helper-Info.plist',
+          ],
+          # TODO(mark): Come up with a fancier way to do this.  It should only
+          # be necessary to list app-Info.plist once, not the three times it is
+          # listed here.
+          'mac_bundle_resources!': [
+            'app/helper-Info.plist',
+          ],
+          # TODO(mark): For now, don't put any resources into this app.  Its
+          # resources directory will be a symbolic link to the browser app's
+          # resources directory.
+          'mac_bundle_resources/': [
+            ['exclude', '.*'],
+          ],
+          'xcode_settings': {
+            'CHROMIUM_BUNDLE_ID': '<(bundle_id)',
+            'CHROMIUM_SHORT_NAME': '<(branding)',
+            'INFOPLIST_FILE': 'app/helper-Info.plist',
+          },
+          'copies': [
+            {
+              'destination': '<(PRODUCT_DIR)/<(mac_product_name) Helper.app/Contents/MacOS',
+              'files': [
+                '<(PRODUCT_DIR)/plugin_carbon_interpose.dylib',
+              ],
+            },
+          ],
+          'postbuilds': [
+            {
+              'postbuild_name': 'Make Symbolic Links',
+              'action': ['app/make_mac_app_symlinks'],
+            },
+            {
+              # Modify the Info.plist as needed.  The script explains why this
+              # is needed.  This is also done in the chrome target.  In
+              # this case, -k0 is always used because Keystone never runs
+              # within the helper app.  -s0 is used to avoid placing Subversion
+              # data in the helper app's Info.plist.  It will be present in
+              # the main app's Info.plist, which is sufficient.
+              'postbuild_name': 'Tweak Info.plist',
+              'action': ['<(DEPTH)/build/mac/tweak_app_infoplist',
+                         '-b<(mac_breakpad)',
+                         '-k0',
+                         '-s0',
+                         '<(branding)'],
+            },
+          ],
+          'conditions': [
+            ['mac_breakpad==1', {
+              'variables': {
+                # A real .dSYM is needed for dump_syms to operate on.
+                'mac_real_dsym': 1,
+              },
+            }],
+          ],
+        },
+        {
           # Convenience target to build a disk image.
           'target_name': 'build_app_dmg',
           # Don't place this in the 'all' list; most won't want it.
           # In GYP, booleans are 0/1, not True/False.
           'suppress_wildcard': 1,
           'type': 'none',
           'dependencies': [
             'chrome',
           ],
           'variables': {
@@ skipping 48 matching lines @@
             'browser/plugin_carbon_interpose_mac.cc',
           ],
           'include_dirs': [
             '..',
           ],
           'link_settings': {
             'libraries': [
               '$(SDKROOT)/System/Library/Frameworks/Carbon.framework',
             ],
           },
+          'xcode_settings': {
+            'DYLIB_INSTALL_NAME_BASE': '@executable_path',
+          },
         },
       ]
     }, { # else: OS != "mac"
       'targets': [
         {
           'target_name': 'convert_dict',
           'type': 'executable',
           'msvs_guid': '42ECD5EC-722F-41DE-B6B8-83764C8016DF',
           'dependencies': [
             '../base/base.gyp:base',
@@ skipping 841 matching lines @@
               # Use outputs of this action as inputs for the main target build.
               # Seems as a misnomer but makes this happy on Linux (scons).
               'process_outputs_as_sources': 1,
             },
           ],  # 'actions'
         },
       ]
     }],
   ],  # 'conditions'
 }