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

Side by Side Diff: chrome/BUILD.gn

Issue 1117063004: Revert of Add a GN flag for the Windows multi-dll build. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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
« no previous file with comments | « build/secondary/tools/grit/grit_rule.gni ('k') | chrome/app/resources/BUILD.gn » ('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 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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 import("//build/config/chrome_build.gni")
6 import("//build/config/features.gni") 5 import("//build/config/features.gni")
7 import("//build/config/locales.gni") 6 import("//build/config/locales.gni")
8 import("//build/config/ui.gni") 7 import("//build/config/ui.gni")
9 import("//chrome/chrome_repack_locales.gni") 8 import("//chrome/chrome_repack_locales.gni")
10 import("//chrome/version.gni") 9 import("//chrome/version.gni")
11 10
12 if (is_android) { 11 if (is_android) {
13 import("//build/config/android/rules.gni") 12 import("//build/config/android/rules.gni")
14 } 13 }
15 14
16 if (!is_android) { 15 if (!is_android) {
17 # TODO(GYP) for Windows need to the the reorder-imports step which probably 16 # TODO(GYP) for Windows need to the the reorder-imports step which probably
18 # means adding another target and renaming this to chrome_initial like in GYP. 17 # means adding another target and renaming this to chrome_initial like in GYP.
19 executable("chrome") { 18 executable("chrome") {
20 # Because the sources list varies so significantly per-platform, generally 19 # Because the sources list varies so significantly per-platform, generally
21 # each platform lists its own files rather than relying on filtering or 20 # each platform lists its own files rather than relying on filtering or
22 # removing unused files. 21 # removing unused files.
23 sources = [ 22 sources = [
24 "app/chrome_exe_resource.h", 23 "app/chrome_exe_resource.h",
25 ] 24 ]
26 defines = [] 25 defines = []
27 deps = [] 26 deps = []
28 data_deps = [] 27 datadeps = []
29 28
30 # TODO(GYP) mac_bundle_resources, xcode_settings 29 # TODO(GYP) mac_bundle_resources, xcode_settings
31 30
32 # TODO(GYP) order_profiling, order_text_section 31 # TODO(GYP) order_profiling, order_text_section
33 32
34 if (is_win) { 33 if (is_win) {
35 sources += [ 34 sources += [
35 "$target_gen_dir/chrome_exe_version.rc",
36 "//content/app/startup_helper_win.cc", 36 "//content/app/startup_helper_win.cc",
37 "//content/public/common/content_switches.cc", 37 "//content/public/common/content_switches.cc",
38 "app/chrome_crash_reporter_client.cc", 38 "app/chrome_crash_reporter_client.cc",
39 "app/chrome_crash_reporter_client.h", 39 "app/chrome_crash_reporter_client.h",
40 "app/chrome_exe.rc", 40 "app/chrome_exe.rc",
41 "app/chrome_exe_main_win.cc", 41 "app/chrome_exe_main_win.cc",
42 "app/chrome_watcher_client_win.cc", 42 "app/chrome_watcher_client_win.cc",
43 "app/chrome_watcher_client_win.h", 43 "app/chrome_watcher_client_win.h",
44 "app/chrome_watcher_command_line_win.cc", 44 "app/chrome_watcher_command_line_win.cc",
45 "app/chrome_watcher_command_line_win.h", 45 "app/chrome_watcher_command_line_win.h",
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 ":packed_extra_resources", 138 ":packed_extra_resources",
139 ":packed_resources", 139 ":packed_resources",
140 "//components/startup_metric_utils", 140 "//components/startup_metric_utils",
141 141
142 # Precompiled plugins that need to get copied to the output directory. 142 # Precompiled plugins that need to get copied to the output directory.
143 # On Mac, internal plugins go inside the framework, so these 143 # On Mac, internal plugins go inside the framework, so these
144 # dependencies are on chrome.dll. 144 # dependencies are on chrome.dll.
145 "//third_party/adobe/flash:flapper_binaries", 145 "//third_party/adobe/flash:flapper_binaries",
146 "//third_party/widevine/cdm:widevinecdmadapter", 146 "//third_party/widevine/cdm:widevinecdmadapter",
147 ] 147 ]
148 # TODO(GYP) some stuff from GYP including chrome_multiple_dll.
148 } 149 }
149 150
150 if (is_linux && enable_plugins) { 151 if (is_linux && enable_plugins) {
151 deps += [ "//pdf" ] 152 deps += [ "//pdf" ]
152 } 153 }
153
154 if (is_multi_dll_chrome) {
155 defines += [ "CHROME_MULTIPLE_DLL" ]
156 data_deps += [ ":chrome_child" ]
157 }
158 } 154 }
159 } # !is_android 155 } # !is_android
160 156
161 shared_library("main_dll") { 157 shared_library("main_dll") {
162 configs += [ "//build/config/compiler:wexit_time_destructors" ] 158 configs += [ "//build/config/compiler:wexit_time_destructors" ]
163 defines = []
164 159
165 deps = [ 160 deps = [
166 ":browser_dependencies", 161 ":browser_dependencies",
167 "//base/allocator", 162 "//base/allocator",
168 ] 163 ]
169 if (is_win) { 164 if (is_win) {
170 output_name = "chrome" 165 output_name = "chrome"
171 166
172 sources = [ 167 sources = [
173 "$root_gen_dir/base/trace_event/etw_manifest/chrome_events_win.rc", 168 "$root_gen_dir/base/trace_event/etw_manifest/chrome_events_win.rc",
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
212 207
213 # TODO(GYP) Lots of VCLinkerTool stuff on Windows. 208 # TODO(GYP) Lots of VCLinkerTool stuff on Windows.
214 209
215 # TODO(GYP) chrome_pgo_phase on Windows. 210 # TODO(GYP) chrome_pgo_phase on Windows.
216 } 211 }
217 212
218 if (use_aura) { 213 if (use_aura) {
219 deps += [ "//ui/compositor" ] 214 deps += [ "//ui/compositor" ]
220 } 215 }
221 216
222 if (is_multi_dll_chrome) { 217 #TODO(GYP) add chrome_multiple_dll support
218 if (false) { #chrome_multiple_dll) {
223 defines += [ "CHROME_MULTIPLE_DLL_BROWSER" ] 219 defines += [ "CHROME_MULTIPLE_DLL_BROWSER" ]
224 deps += [ "//content/public/app:browser" ] 220 deps += [ "//content/public/app:browser" ]
225 } else { 221 } else {
226 deps += [ 222 deps += [
227 ":child_dependencies", 223 ":child_dependencies",
228 "//content/public/app:both", 224 "//content/public/app:both",
229 ] 225 ]
230 } 226 }
231 227
232 if (cld_version == 0 || cld_version == 2) { 228 if (cld_version == 0 || cld_version == 2) {
233 deps += [ "//third_party/cld_2" ] 229 deps += [ "//third_party/cld_2" ]
234 } 230 }
235 231
236 if (is_mac) { 232 if (is_mac) {
237 #['OS=="mac" and component!="shared_library"', { TODO(GYP) 233 #['OS=="mac" and component!="shared_library"', { TODO(GYP)
238 # 'includes': [ 'chrome_dll_bundle.gypi' ], 234 # 'includes': [ 'chrome_dll_bundle.gypi' ],
239 #}], 235 #}],
240 # TODO(GYP) Lots of other stuff in the OS=="mac" block. 236 # TODO(GYP) Lots of other stuff in the OS=="mac" block.
241 } 237 }
242 238
243 if (enable_plugins && !is_multi_dll_chrome) { 239 if (enable_plugins) {
244 deps += [ "//pdf" ] 240 deps += [ "//pdf" ]
245 } 241 }
246 } 242 }
247 243
248 if (is_multi_dll_chrome) {
249 shared_library("chrome_child") {
250 sources = [
251 "app/chrome_main.cc",
252 "app/chrome_main_delegate.cc",
253 "app/chrome_main_delegate.h",
254 "app/close_handle_hook_win.cc",
255 "app/close_handle_hook_win.h",
256 ]
257
258 configs += [ "//build/config/compiler:wexit_time_destructors" ]
259 defines = [ "CHROME_MULTIPLE_DLL_CHILD" ]
260
261 deps = [
262 ":child_dependencies",
263 ":chrome_dll_version",
264 "//chrome/browser/policy:path_parser",
265 "//content/public/app:child",
266 ]
267
268 if (is_win) {
269 deps += [ "//chrome_elf" ]
270 # TODO(GYP): PGO on Windows.
271 # ['chrome_pgo_phase==1', {
272 # 'msvs_settings': {
273 # 'VCLinkerTool': {
274 # 'LinkTimeCodeGeneration': '2',
275 # },
276 # },
277 # }],
278 # ['chrome_pgo_phase==2', {
279 # 'msvs_settings': {
280 # 'VCLinkerTool': {
281 # 'LinkTimeCodeGeneration': '3',
282 # },
283 # },
284 # }],
285 }
286 if (enable_plugins) {
287 deps += [ "//pdf" ]
288 }
289 }
290 }
291
292 # GYP version: chromium_browser_dependencies variable in chrome.gyp 244 # GYP version: chromium_browser_dependencies variable in chrome.gyp
293 group("browser_dependencies") { 245 group("browser_dependencies") {
294 deps = [ 246 deps = [
295 "//chrome/browser", 247 "//chrome/browser",
296 "//chrome/common", 248 "//chrome/common",
297 "//sync", 249 "//sync",
298 ] 250 ]
299 if (!is_ios) { 251 if (!is_ios) {
300 deps += [ "//ppapi/host" ] 252 deps += [ "//ppapi/host" ]
301 } 253 }
(...skipping 490 matching lines...) Expand 10 before | Expand all | Expand 10 after
792 744
793 args = [ 745 args = [
794 rebase_path(infile, root_build_dir), 746 rebase_path(infile, root_build_dir),
795 rebase_path(outfile, root_build_dir), 747 rebase_path(outfile, root_build_dir),
796 "-e s/@@NAME@@/$name/", 748 "-e s/@@NAME@@/$name/",
797 "-e s/@@FILENAME@@/$filename/", 749 "-e s/@@FILENAME@@/$filename/",
798 "-e s/@@CONFDIR@@/$confdir/", 750 "-e s/@@CONFDIR@@/$confdir/",
799 ] 751 ]
800 } 752 }
801 } 753 }
OLDNEW
« no previous file with comments | « build/secondary/tools/grit/grit_rule.gni ('k') | chrome/app/resources/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698