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

Side by Side 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/chrome_common.gypi ('k') | chrome/chrome_syzygy.gyp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 { 4 {
5 'conditions': [ 5 'conditions': [
6 ['OS=="mac" or OS=="win"', { 6 ['OS=="mac" or OS=="win"', {
7 'targets': [ 7 'targets': [
8 { 8 {
9 'target_name': 'chrome_dll', 9 'target_name': 'chrome_dll',
10 'type': 'none', 10 'type': 'none',
11 'dependencies': [ 11 'dependencies': [
12 'chrome_main_dll', 12 'chrome_main_dll',
13 ], 13 ],
14 'conditions': [ 14 'conditions': [
15 ['OS=="mac" and component=="shared_library"', { 15 ['OS=="mac" and component=="shared_library"', {
16 'type': 'shared_library', 16 'type': 'shared_library',
17 'includes': [ 'chrome_dll_bundle.gypi' ], 17 'includes': [ 'chrome_dll_bundle.gypi' ],
18 'xcode_settings': { 18 'xcode_settings': {
19 'OTHER_LDFLAGS': [ 19 'OTHER_LDFLAGS': [
20 '-Wl,-reexport_library,<(PRODUCT_DIR)/libchrome_main_dll.dylib ', 20 '-Wl,-reexport_library,<(PRODUCT_DIR)/libchrome_main_dll.dylib ',
21 ], 21 ],
22 }, 22 },
23 }], # OS=="mac" 23 }], # OS=="mac"
24 ['chrome_multiple_dll==1', {
25 'dependencies': [
26 'chrome_child_dll',
27 ],
28 }],
24 ['incremental_chrome_dll==1', { 29 ['incremental_chrome_dll==1', {
25 # Linking to a different directory and then hardlinking back 30 # Linking to a different directory and then hardlinking back
26 # to OutDir is a workaround to avoid having the .ilk for 31 # to OutDir is a workaround to avoid having the .ilk for
27 # chrome.exe and chrome.dll conflicting. See crbug.com/92528 32 # chrome.exe and chrome.dll conflicting. See crbug.com/92528
28 # for more information. Done on the dll instead of the exe so 33 # for more information. Done on the dll instead of the exe so
29 # that people launching from VS don't need to modify 34 # that people launching from VS don't need to modify
30 # $(TargetPath) for the exe. 35 # $(TargetPath) for the exe.
31 'actions': [ 36 'actions': [
32 { 37 {
33 'action_name': 'hardlink_to_output', 38 'action_name': 'hardlink_to_output',
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 ] 73 ]
69 }, 74 },
70 { 75 {
71 'target_name': 'chrome_main_dll', 76 'target_name': 'chrome_main_dll',
72 'type': 'shared_library', 77 'type': 'shared_library',
73 'variables': { 78 'variables': {
74 'enable_wexit_time_destructors': 1, 79 'enable_wexit_time_destructors': 1,
75 }, 80 },
76 'dependencies': [ 81 'dependencies': [
77 '<@(chromium_browser_dependencies)', 82 '<@(chromium_browser_dependencies)',
78 '<@(chromium_child_dependencies)',
79 '../content/content.gyp:content_worker',
80 'app/policy/cloud_policy_codegen.gyp:policy', 83 'app/policy/cloud_policy_codegen.gyp:policy',
81 ], 84 ],
82 'conditions': [ 85 'conditions': [
83 ['use_aura==1', { 86 ['use_aura==1', {
84 'dependencies': [ 87 'dependencies': [
85 '../ui/compositor/compositor.gyp:compositor', 88 '../ui/compositor/compositor.gyp:compositor',
86 ], 89 ],
87 }], 90 }],
88 ['use_ash==1', { 91 ['use_ash==1', {
89 'sources': [ 92 'sources': [
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
219 'wininet.dll', 222 'wininet.dll',
220 'winspool.drv', 223 'winspool.drv',
221 'ws2_32.dll', 224 'ws2_32.dll',
222 'wsock32.dll', 225 'wsock32.dll',
223 ], 226 ],
224 }, 227 },
225 'VCManifestTool': { 228 'VCManifestTool': {
226 'AdditionalManifestFiles': '$(ProjectDir)\\app\\chrome.dll.man ifest', 229 'AdditionalManifestFiles': '$(ProjectDir)\\app\\chrome.dll.man ifest',
227 }, 230 },
228 }, 231 },
229 }], # OS=="win" 232 }],
233 ['chrome_multiple_dll==1', {
234 'defines': [
235 'CHROME_MULTIPLE_DLL_BROWSER',
236 ],
237 }, {
238 'dependencies': [
239 '<@(chromium_child_dependencies)',
240 '../content/content.gyp:content_worker',
241 ],
242 }],
230 ['OS=="mac" and component!="shared_library"', { 243 ['OS=="mac" and component!="shared_library"', {
231 'includes': [ 'chrome_dll_bundle.gypi' ], 244 'includes': [ 'chrome_dll_bundle.gypi' ],
232 }], 245 }],
233 ['OS=="mac" and component=="shared_library"', { 246 ['OS=="mac" and component=="shared_library"', {
234 'xcode_settings': { 'OTHER_LDFLAGS': [ '-Wl,-ObjC' ], }, 247 'xcode_settings': { 'OTHER_LDFLAGS': [ '-Wl,-ObjC' ], },
235 }], 248 }],
236 ['chrome_split_dll', {
237 'sources': [
238 # See comment in .cc for explanation.
239 'split_dll_fake_entry.cc',
240 ],
241 'msvs_settings': {
242 'VCLinkerTool': {
243 'AdditionalOptions': ['/splitlink'],
244 },
245 }
246 }],
247 ['OS=="mac"', { 249 ['OS=="mac"', {
248 'xcode_settings': { 250 'xcode_settings': {
249 # Define the order of symbols within the framework. This 251 # Define the order of symbols within the framework. This
250 # sets -order_file. 252 # sets -order_file.
251 'ORDER_FILE': 'app/framework.order', 253 'ORDER_FILE': 'app/framework.order',
252 }, 254 },
253 'sources': [ 255 'sources': [
254 'app/chrome_command_ids.h', 256 'app/chrome_command_ids.h',
255 'app/chrome_dll_resource.h', 257 'app/chrome_dll_resource.h',
256 'app/chrome_main.cc', 258 'app/chrome_main.cc',
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
328 # more details. 330 # more details.
329 'DebugInformationFormat': '3', 331 'DebugInformationFormat': '3',
330 'ProgramDataBaseFileName': '<(PRODUCT_DIR)/chrome.dll.pdb', 332 'ProgramDataBaseFileName': '<(PRODUCT_DIR)/chrome.dll.pdb',
331 }, 333 },
332 }, 334 },
333 }], 335 }],
334 ], 336 ],
335 }, 337 },
336 ], 338 ],
337 }], 339 }],
340 ['chrome_multiple_dll', {
341 'targets': [
342 {
343 'target_name': 'chrome_child_dll',
344 'type': 'shared_library',
345 'product_name': 'chrome_child',
346 'variables': {
347 'enable_wexit_time_destructors': 1,
348 },
349 'dependencies': [
350 '<@(chromium_child_dependencies)',
351 '../content/content.gyp:content_worker',
352 'policy_path_parser',
353 ],
354 'defines': [
355 'CHROME_MULTIPLE_DLL_CHILD',
356 ],
357 'sources': [
358 'app/chrome_main.cc',
359 'app/chrome_main_delegate.cc',
360 'app/chrome_main_delegate.h',
361 ],
362 }, # target chrome_child_dll
363 ],
364 }],
338 ], 365 ],
339 } 366 }
OLDNEW
« 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