Mac: Generate App Mode Loader bundle + cleanup

chrome/chrome.gyp

Index: chrome/chrome.gyp
diff --git a/chrome/chrome.gyp b/chrome/chrome.gyp
index c39eb245d404c4fec92cc1733c0ea4186a1212f7..bc376919bc45e3021fc7164fe2422d9aea156831 100644
--- a/chrome/chrome.gyp
+++ b/chrome/chrome.gyp
@@ -86,11 +86,13 @@
           ['branding=="Chrome"', {
             'mac_bundle_id': 'com.google.Chrome',
             'mac_creator': 'rimZ',
+            'app_mode_app_bundle_id': 'com.google.Chrome.app.@APP_MODE_SHORTCUT_ID@',

[Mark Mentovai, 2012/02/22 16:43:22]

I think you can just move this out one level to be a sibling to 'conditions' and
'tweak_info_plist_path' and write:

'app_mode_app_bundle_id': '<(mac_bundle_id).@APP_MODE_SHORTCUT_ID@',

Give it a try.

[jeremy, 2012/02/23 14:21:13]

Done.

             # The policy .grd file also needs the bundle id.
             'grit_defines': ['-D', 'mac_bundle_id=com.google.Chrome'],
           }, {  # else: branding!="Chrome"
             'mac_bundle_id': 'org.chromium.Chromium',
             'mac_creator': 'Cr24',
+            'app_mode_app_bundle_id': 'com.google.Chromium.app.@APP_MODE_SHORTCUT_ID@',
             # The policy .grd file also needs the bundle id.
             'grit_defines': ['-D', 'mac_bundle_id=org.chromium.Chromium'],
           }],  # branding
@@ -786,7 +788,7 @@
               # Modify the Info.plist as needed.  The script explains why this
               # is needed.  This is also done in the chrome and chrome_dll
               # targets.  In this case, --breakpad=0, -k0, and -s0 are used
-              # because Breakpad, Keystone, and Subersion keys are never
+              # because Breakpad, Keystone, and Subversion keys are never
               # placed into the helper.
               'postbuild_name': 'Tweak Info.plist',
               'action': ['<(tweak_info_plist_path)',
@@ -846,17 +848,21 @@
           ],
         },  # target app_mode_app_support
         {
-          # This produces the app mode loader, but not as a bundle. Chromium
-          # itself is responsible for producing bundles.
+          # This produces the template for app mode loader bundles. It's a
+          # template in the sense that parts of it need to be "filled in" by
+          # Chrome before it can be executed.
           'target_name': 'app_mode_app',
           'type': 'executable',
+          'mac_bundle' : 1,
           'variables': { 'enable_wexit_time_destructors': 1, },
-          'product_name': '<(mac_product_name) App Mode Loader',
+          'product_name': 'app_mode_loader',
           'dependencies': [
             'app_mode_app_support',
+            'infoplist_strings_tool',
           ],
           'sources': [
             'app/app_mode_loader_mac.mm',
+            'app/app_mode-Info.plist',

[Mark Mentovai, 2012/02/22 16:43:22]

Is this needed here?

           ],
           'include_dirs': [
             '..',
@@ -867,6 +873,32 @@
               '$(SDKROOT)/System/Library/Frameworks/Foundation.framework',
             ],
           },
+          'mac_bundle_resources!': [
+            'app/app_mode-Info.plist',
+          ],
+          'mac_bundle_resources/': [
+            ['exclude', '.*'],
+          ],
+          'xcode_settings': {
+            'INFOPLIST_FILE': 'app/app_mode-Info.plist',
+            'APP_MODE_APP_BUNDLE_ID': '<(app_mode_app_bundle_id)',

[Mark Mentovai, 2012/02/22 16:43:22]

It would also be reasonable to just write
'<(mac_bundle_id).@APP_MODE_SHORTCUT_ID@' here and not have a separate variable
for it, since it’s only used in one location.

[jeremy, 2012/02/23 14:21:13]

Done.

+          },
+          'postbuilds' : [
+            {
+              # Modify the Info.plist as needed.  The script explains why this
+              # is needed.  This is also done in the chrome and chrome_dll
+              # targets.  In this case, --breakpad=0, -k0, and -s0 are used
+              # because Breakpad, Keystone, and Subversion keys are never
+              # placed into the helper.

[Mark Mentovai, 2012/02/22 16:43:22]

Not helper, but app mode loader.

[jeremy, 2012/02/23 14:21:13]

Done.

+              'postbuild_name': 'Tweak Info.plist',
+              'action': ['<(tweak_info_plist_path)',
+                         '--breakpad=0',
+                         '-k0',
+                         '-s0',
+                         '<(branding)',
+                         '<(mac_bundle_id)'],
+            },
+          ],
         },  # target app_mode_app
         {
           # Convenience target to build a disk image.