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

Side by Side Diff: chrome/BUILD.gn

Issue 2576573002: Move enable_pdf to a buildflag header. (Closed)
Patch Set: No Chromecast Created 4 years 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/config/features.gni ('k') | chrome/app/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/buildflag_header.gni") 5 import("//build/buildflag_header.gni")
6 import("//build/config/chrome_build.gni") 6 import("//build/config/chrome_build.gni")
7 import("//build/config/compiler/compiler.gni") 7 import("//build/config/compiler/compiler.gni")
8 import("//build/config/compiler/pgo/pgo.gni") 8 import("//build/config/compiler/pgo/pgo.gni")
9 import("//build/config/features.gni") 9 import("//build/config/features.gni")
10 import("//build/config/locales.gni") 10 import("//build/config/locales.gni")
(...skipping 221 matching lines...) Expand 10 before | Expand all | Expand 10 after
232 deps += [ 232 deps += [
233 # On Linux, link the dependencies (libraries) that make up actual 233 # On Linux, link the dependencies (libraries) that make up actual
234 # Chromium functionality directly into the executable. 234 # Chromium functionality directly into the executable.
235 ":browser_dependencies", 235 ":browser_dependencies",
236 ":child_dependencies", 236 ":child_dependencies",
237 ":manpage", 237 ":manpage",
238 238
239 # Needed to use the master_preferences functions 239 # Needed to use the master_preferences functions
240 "//chrome/installer/util:with_no_strings", 240 "//chrome/installer/util:with_no_strings",
241 "//content/public/app:both", 241 "//content/public/app:both",
242 "//pdf",
242 243
243 # For headless mode. 244 # For headless mode.
244 "//headless:headless_shell_lib", 245 "//headless:headless_shell_lib",
245 ] 246 ]
246 if (enable_plugins && enable_pdf) {
247 deps += [ "//pdf" ]
248 }
249 247
250 public_deps = [ 248 public_deps = [
251 ":xdg_mime", # Needs to be public for installer to consume files. 249 ":xdg_mime", # Needs to be public for installer to consume files.
252 "//chrome/common:features", 250 "//chrome/common:features",
253 ] 251 ]
254 252
255 ldflags = [ "-pie" ] 253 ldflags = [ "-pie" ]
256 254
257 if (use_pango || use_cairo) { 255 if (use_pango || use_cairo) {
258 # Needed for chrome_main.cc initialization of libraries. 256 # Needed for chrome_main.cc initialization of libraries.
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
371 deps += [ "//ui/compositor" ] 369 deps += [ "//ui/compositor" ]
372 } 370 }
373 371
374 if (is_multi_dll_chrome) { 372 if (is_multi_dll_chrome) {
375 defines += [ "CHROME_MULTIPLE_DLL_BROWSER" ] 373 defines += [ "CHROME_MULTIPLE_DLL_BROWSER" ]
376 deps += [ "//content/public/app:browser" ] 374 deps += [ "//content/public/app:browser" ]
377 } else { 375 } else {
378 deps += [ 376 deps += [
379 ":child_dependencies", 377 ":child_dependencies",
380 "//content/public/app:both", 378 "//content/public/app:both",
379 "//pdf",
381 ] 380 ]
382 } 381 }
383 382
384 if (enable_plugins && enable_pdf && !is_multi_dll_chrome) {
385 deps += [ "//pdf" ]
386 }
387
388 if (enable_package_mash_services) { 383 if (enable_package_mash_services) {
389 deps += [ "//chrome/app/mash" ] 384 deps += [ "//chrome/app/mash" ]
390 } 385 }
391 } 386 }
392 387
393 if (is_multi_dll_chrome) { 388 if (is_multi_dll_chrome) {
394 # This manifest matches what GYP produced. It may not even be necessary. 389 # This manifest matches what GYP produced. It may not even be necessary.
395 windows_manifest("chrome_child_manifest") { 390 windows_manifest("chrome_child_manifest") {
396 sources = [ 391 sources = [
397 as_invoker_manifest, 392 as_invoker_manifest,
(...skipping 15 matching lines...) Expand all
413 ":child_dependencies", 408 ":child_dependencies",
414 ":chrome_child_manifest", 409 ":chrome_child_manifest",
415 ":chrome_dll_version", 410 ":chrome_dll_version",
416 "//build/config/sanitizers:deps", 411 "//build/config/sanitizers:deps",
417 "//chrome/browser/policy:path_parser", 412 "//chrome/browser/policy:path_parser",
418 "//chrome/common:features", 413 "//chrome/common:features",
419 "//chrome/install_static:install_static_util", 414 "//chrome/install_static:install_static_util",
420 "//components/browser_watcher:browser_watcher_client", 415 "//components/browser_watcher:browser_watcher_client",
421 "//components/crash/content/app", 416 "//components/crash/content/app",
422 "//content/public/app:child", 417 "//content/public/app:child",
418 "//pdf",
423 ] 419 ]
424 420
425 ldflags = [ 421 ldflags = [
426 "/DELAYLOAD:d3d11.dll", 422 "/DELAYLOAD:d3d11.dll",
427 "/DELAYLOAD:d3d9.dll", 423 "/DELAYLOAD:d3d9.dll",
428 "/DELAYLOAD:dwmapi.dll", 424 "/DELAYLOAD:dwmapi.dll",
429 "/DELAYLOAD:dxva2.dll", 425 "/DELAYLOAD:dxva2.dll",
430 "/DELAYLOAD:esent.dll", 426 "/DELAYLOAD:esent.dll",
431 "/DELAYLOAD:wininet.dll", 427 "/DELAYLOAD:wininet.dll",
432 ] 428 ]
433 429
434 if (symbol_level == 2) { 430 if (symbol_level == 2) {
435 # Incremental linking doesn't work on this target in debug mode with 431 # Incremental linking doesn't work on this target in debug mode with
436 # full symbols, but does work in other cases, including minimal 432 # full symbols, but does work in other cases, including minimal
437 # symbols. 433 # symbols.
438 configs -= [ "//build/config/win:default_incremental_linking" ] 434 configs -= [ "//build/config/win:default_incremental_linking" ]
439 configs += [ "//build/config/win:no_incremental_linking" ] 435 configs += [ "//build/config/win:no_incremental_linking" ]
440 } 436 }
441 configs += [ "//build/config/compiler/pgo:default_pgo_flags" ] 437 configs += [ "//build/config/compiler/pgo:default_pgo_flags" ]
442
443 if (enable_plugins && enable_pdf) {
444 deps += [ "//pdf" ]
445 }
446 } 438 }
447 } 439 }
448 copy("copy_first_run") { 440 copy("copy_first_run") {
449 sources = [ 441 sources = [
450 "app/firstRun", 442 "app/firstRun",
451 ] 443 ]
452 outputs = [ 444 outputs = [
453 "$root_out_dir/First Run", 445 "$root_out_dir/First Run",
454 ] 446 ]
455 } 447 }
(...skipping 517 matching lines...) Expand 10 before | Expand all | Expand 10 after
973 ] 965 ]
974 966
975 deps = [ 967 deps = [
976 ":browser_dependencies", 968 ":browser_dependencies",
977 ":child_dependencies", 969 ":child_dependencies",
978 "//chrome/app:command_ids", 970 "//chrome/app:command_ids",
979 "//chrome/common:features", 971 "//chrome/common:features",
980 "//components/crash/content/app", 972 "//components/crash/content/app",
981 "//components/policy:generated", 973 "//components/policy:generated",
982 "//content/public/app:both", 974 "//content/public/app:both",
975 "//pdf",
983 "//third_party/cld", 976 "//third_party/cld",
984 ] 977 ]
985 978
986 if (is_component_build) { 979 if (is_component_build) {
987 libs = [ "Carbon.framework" ] 980 libs = [ "Carbon.framework" ]
988 } 981 }
989 982
990 ldflags = [ 983 ldflags = [
991 "-Wl,-order_file", 984 "-Wl,-order_file",
992 "-Wl," + rebase_path("app/framework.order", root_build_dir), 985 "-Wl," + rebase_path("app/framework.order", root_build_dir),
993 "-ObjC", 986 "-ObjC",
994 ] 987 ]
995 988
996 if (enable_plugins && enable_pdf) {
997 deps += [ "//pdf" ]
998 }
999
1000 if (enable_package_mash_services) { 989 if (enable_package_mash_services) {
1001 deps += [ "//chrome/app/mash" ] 990 deps += [ "//chrome/app/mash" ]
1002 } 991 }
1003 992
1004 configs += [ "//build/config/compiler:wexit_time_destructors" ] 993 configs += [ "//build/config/compiler:wexit_time_destructors" ]
1005 } 994 }
1006 995
1007 mac_framework_bundle("chrome_framework") { 996 mac_framework_bundle("chrome_framework") {
1008 output_name = chrome_framework_name 997 output_name = chrome_framework_name
1009 998
(...skipping 594 matching lines...) Expand 10 before | Expand all | Expand 10 after
1604 "//chrome/app/theme/$branding_path_component/product_logo_48.png", 1593 "//chrome/app/theme/$branding_path_component/product_logo_48.png",
1605 "//chrome/tools/build/linux/chrome-wrapper", 1594 "//chrome/tools/build/linux/chrome-wrapper",
1606 "//third_party/xdg-utils/scripts/xdg-mime", 1595 "//third_party/xdg-utils/scripts/xdg-mime",
1607 "//third_party/xdg-utils/scripts/xdg-settings", 1596 "//third_party/xdg-utils/scripts/xdg-settings",
1608 ] 1597 ]
1609 outputs = [ 1598 outputs = [
1610 "$root_out_dir/{{source_file_part}}", 1599 "$root_out_dir/{{source_file_part}}",
1611 ] 1600 ]
1612 } 1601 }
1613 } 1602 }
OLDNEW
« no previous file with comments | « build/config/features.gni ('k') | chrome/app/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698