Address more TODO(GYP) in the GN build.

chrome/installer/util/BUILD.gn

Index: chrome/installer/util/BUILD.gn
diff --git a/chrome/installer/util/BUILD.gn b/chrome/installer/util/BUILD.gn
index 63a89d901157f999498223edde997d79f42a9e10..8adfc753e88731f0935a1087113619818ccec163 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",
@@ -343,6 +337,7 @@ if (is_win) {
       "//base",
       "//base:i18n",
       "//base/test:test_support",
+      "//build/win:default_exe_manifest",

[brucedawson, 2016/05/09 21:30:32]

When I build this with gn I see that it already has a manifest, which means that
this line is not necessary. I think that is because test() implies the default
manifest.

       "//chrome:other_version",
       "//chrome/installer/setup:lib",
       "//chrome/installer/test:alternate_version_generator_lib",
@@ -355,7 +350,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