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

Unified Diff: chrome/chrome_dll.gypi

Issue 17619005: Create top-level separate targets for browser and child dlls (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: android Created 7 years, 5 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/chrome_common.gypi ('k') | chrome/chrome_syzygy.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_dll.gypi
diff --git a/chrome/chrome_dll.gypi b/chrome/chrome_dll.gypi
index 090a3d2dd2063566ef4d281d9dc604d8572e8a90..f5808a782ec004b7643895356fc904d9a90e2720 100644
--- a/chrome/chrome_dll.gypi
+++ b/chrome/chrome_dll.gypi
@@ -21,6 +21,11 @@
],
},
}], # OS=="mac"
+ ['chrome_multiple_dll==1', {
+ 'dependencies': [
+ 'chrome_child_dll',
+ ],
+ }],
['incremental_chrome_dll==1', {
# Linking to a different directory and then hardlinking back
# to OutDir is a workaround to avoid having the .ilk for
@@ -75,8 +80,6 @@
},
'dependencies': [
'<@(chromium_browser_dependencies)',
- '<@(chromium_child_dependencies)',
- '../content/content.gyp:content_worker',
'app/policy/cloud_policy_codegen.gyp:policy',
],
'conditions': [
@@ -226,24 +229,23 @@
'AdditionalManifestFiles': '$(ProjectDir)\\app\\chrome.dll.manifest',
},
},
- }], # OS=="win"
+ }],
+ ['chrome_multiple_dll==1', {
+ 'defines': [
+ 'CHROME_MULTIPLE_DLL_BROWSER',
+ ],
+ }, {
+ 'dependencies': [
+ '<@(chromium_child_dependencies)',
+ '../content/content.gyp:content_worker',
+ ],
+ }],
['OS=="mac" and component!="shared_library"', {
'includes': [ 'chrome_dll_bundle.gypi' ],
}],
['OS=="mac" and component=="shared_library"', {
'xcode_settings': { 'OTHER_LDFLAGS': [ '-Wl,-ObjC' ], },
}],
- ['chrome_split_dll', {
- 'sources': [
- # See comment in .cc for explanation.
- 'split_dll_fake_entry.cc',
- ],
- 'msvs_settings': {
- 'VCLinkerTool': {
- 'AdditionalOptions': ['/splitlink'],
- },
- }
- }],
['OS=="mac"', {
'xcode_settings': {
# Define the order of symbols within the framework. This
@@ -335,5 +337,30 @@
},
],
}],
+ ['chrome_multiple_dll', {
+ 'targets': [
+ {
+ 'target_name': 'chrome_child_dll',
+ 'type': 'shared_library',
+ 'product_name': 'chrome_child',
+ 'variables': {
+ 'enable_wexit_time_destructors': 1,
+ },
+ 'dependencies': [
+ '<@(chromium_child_dependencies)',
+ '../content/content.gyp:content_worker',
+ 'policy_path_parser',
+ ],
+ 'defines': [
+ 'CHROME_MULTIPLE_DLL_CHILD',
+ ],
+ 'sources': [
+ 'app/chrome_main.cc',
+ 'app/chrome_main_delegate.cc',
+ 'app/chrome_main_delegate.h',
+ ],
+ }, # target chrome_child_dll
+ ],
+ }],
],
}
« no previous file with comments | « chrome/chrome_common.gypi ('k') | chrome/chrome_syzygy.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698