Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3748)

Unified Diff: chrome/chrome.gyp

Issue 9416012: Mac: Generate App Mode Loader bundle + cleanup (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased against trunk Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/web_applications/web_app_mac_unittest.mm ('k') | chrome/chrome_dll.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome.gyp
diff --git a/chrome/chrome.gyp b/chrome/chrome.gyp
index ccac7c47f8b1b95889ed48d322e1b5b580c97031..f07d5545719f1b5c8845d13af264021899db75e9 100644
--- a/chrome/chrome.gyp
+++ b/chrome/chrome.gyp
@@ -781,7 +781,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)',
@@ -841,17 +841,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',
],
'include_dirs': [
'..',
@@ -862,6 +866,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': '<(mac_bundle_id).app.@APP_MODE_SHORTCUT_ID@',
+ },
+ '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 app mode loader.
+ '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.
« no previous file with comments | « chrome/browser/web_applications/web_app_mac_unittest.mm ('k') | chrome/chrome_dll.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698