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

Unified Diff: chrome/installer/util/BUILD.gn

Issue 1957683002: Address more TODO(GYP) in the GN build. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@todo
Patch Set: iOS Created 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/installer/test/BUILD.gn ('k') | chrome/test/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/installer/util/BUILD.gn
diff --git a/chrome/installer/util/BUILD.gn b/chrome/installer/util/BUILD.gn
index 63a89d901157f999498223edde997d79f42a9e10..003b20b7c9e9e4dc2402efab21268517b9fee139 100644
--- a/chrome/installer/util/BUILD.gn
+++ b/chrome/installer/util/BUILD.gn
@@ -52,11 +52,6 @@ static_library("with_no_strings") {
allow_circular_includes_from = [ "//chrome/common:constants" ]
if (is_win) {
- # TODO(GYP) installer_util on Windows.
- # This is a bit tricky. It seems that what's currently called
- # installer_util_nacl_win64 should be a different target with a different
- # name ("installer_util_minimal"?) since it's not strictly the 64-bit build
- # of that target.
sources = [
"chrome_binaries_operations.cc",
"chrome_binaries_operations.h",
@@ -134,7 +129,6 @@ static_library("with_no_strings") {
"//build/config/compiler:no_size_t_to_int_warning",
]
- # TODO(GYP) hook up corresponding version of installer_util_nacl_win64 in GN
sources += [
"advanced_firewall_manager_win.cc",
"advanced_firewall_manager_win.h",
@@ -355,7 +349,5 @@ if (is_win) {
"//chrome/test/data/extensions/",
"//chrome/test/data/installer/",
]
-
- # TODO(GYP) manifest: $(ProjectDir)\\installer\\mini_installer\\mini_installer.exe.manifest
}
} # is_win
« no previous file with comments | « chrome/installer/test/BUILD.gn ('k') | chrome/test/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698