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

Unified 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, 8 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 | « build/secondary/tools/grit/grit_rule.gni ('k') | chrome/app/resources/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/BUILD.gn
diff --git a/chrome/BUILD.gn b/chrome/BUILD.gn
index 833dcb197a0d358cbd768ba25a6995c46bdd774c..0ab9496a535a5e658eda5f4827c6304b3852a986 100644
--- a/chrome/BUILD.gn
+++ b/chrome/BUILD.gn
@@ -2,7 +2,6 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
-import("//build/config/chrome_build.gni")
import("//build/config/features.gni")
import("//build/config/locales.gni")
import("//build/config/ui.gni")
@@ -25,7 +24,7 @@
]
defines = []
deps = []
- data_deps = []
+ datadeps = []
# TODO(GYP) mac_bundle_resources, xcode_settings
@@ -33,6 +32,7 @@
if (is_win) {
sources += [
+ "$target_gen_dir/chrome_exe_version.rc",
"//content/app/startup_helper_win.cc",
"//content/public/common/content_switches.cc",
"app/chrome_crash_reporter_client.cc",
@@ -145,22 +145,17 @@
"//third_party/adobe/flash:flapper_binaries",
"//third_party/widevine/cdm:widevinecdmadapter",
]
+ # TODO(GYP) some stuff from GYP including chrome_multiple_dll.
}
if (is_linux && enable_plugins) {
deps += [ "//pdf" ]
}
-
- if (is_multi_dll_chrome) {
- defines += [ "CHROME_MULTIPLE_DLL" ]
- data_deps += [ ":chrome_child" ]
- }
}
} # !is_android
shared_library("main_dll") {
configs += [ "//build/config/compiler:wexit_time_destructors" ]
- defines = []
deps = [
":browser_dependencies",
@@ -219,7 +214,8 @@
deps += [ "//ui/compositor" ]
}
- if (is_multi_dll_chrome) {
+ #TODO(GYP) add chrome_multiple_dll support
+ if (false) { #chrome_multiple_dll) {
defines += [ "CHROME_MULTIPLE_DLL_BROWSER" ]
deps += [ "//content/public/app:browser" ]
} else {
@@ -240,52 +236,8 @@
# TODO(GYP) Lots of other stuff in the OS=="mac" block.
}
- if (enable_plugins && !is_multi_dll_chrome) {
+ if (enable_plugins) {
deps += [ "//pdf" ]
- }
-}
-
-if (is_multi_dll_chrome) {
- shared_library("chrome_child") {
- sources = [
- "app/chrome_main.cc",
- "app/chrome_main_delegate.cc",
- "app/chrome_main_delegate.h",
- "app/close_handle_hook_win.cc",
- "app/close_handle_hook_win.h",
- ]
-
- configs += [ "//build/config/compiler:wexit_time_destructors" ]
- defines = [ "CHROME_MULTIPLE_DLL_CHILD" ]
-
- deps = [
- ":child_dependencies",
- ":chrome_dll_version",
- "//chrome/browser/policy:path_parser",
- "//content/public/app:child",
- ]
-
- if (is_win) {
- deps += [ "//chrome_elf" ]
- # TODO(GYP): PGO on Windows.
- # ['chrome_pgo_phase==1', {
- # 'msvs_settings': {
- # 'VCLinkerTool': {
- # 'LinkTimeCodeGeneration': '2',
- # },
- # },
- # }],
- # ['chrome_pgo_phase==2', {
- # 'msvs_settings': {
- # 'VCLinkerTool': {
- # 'LinkTimeCodeGeneration': '3',
- # },
- # },
- # }],
- }
- if (enable_plugins) {
- deps += [ "//pdf" ]
- }
}
}
« 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