Revert "Move installer.cc file to updater/win directory."

This reverts commit e1f49aca.

Reason for revert: Breaks build - linker error (see bug)

Bug: 1165994

Original change's description:
> Move installer.cc file to updater/win directory.
>
> The installer API code is Windows-specific and it would be desirable
> to isolate //chrome/updater from its implementation details.
>
> This change uses public_deps and visibility to make
> updater:lib and updater/win:lib act as a unit, to resolve the issue
> of the circular includes as suggested by the GN documentation.
>
> Bug: 1014630
>
> Change-Id: Ib47dc9041af08651549d0feaf35790f336dfe193
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2588508
> Commit-Queue: Sorin Jianu <sorin@chromium.org>
> Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#842765}

TBR=sorin@chromium.org,waffles@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com

Change-Id: I0acf2fce533e34424465c4ce65202a53881feb4d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1014630
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2626351Reviewed-by: Dana Fried <dfried@google.com>
Commit-Queue: Dana Fried <dfried@google.com>
Cr-Commit-Position: refs/heads/master@{#842780}

chrome/updater/BUILD.gn

@@ -166,6 +166,7 @@ if (is_win || is_mac) {
         "app/server/win/service_main.cc",
         "app/server/win/service_main.h",
         "external_constants_win.cc",
+        "installer_win.cc",
         "lib_util_win.cc",
         "prefs_win.cc",
         "service_factory_win.cc",
@@ -180,6 +181,7 @@ if (is_win || is_mac) {
       ":version_header",
       "//base",
       "//base:i18n",
+      "//chrome/updater/device_management",
       "//components/crash/core/common:crash_key",
       "//components/crx_file:crx_file",
       "//components/prefs",
@@ -190,17 +192,13 @@ if (is_win || is_mac) {
     ]
 
     if (is_win) {
-      # //chrome/updater:lib and //chrome/updater/win:lib behave as one unit
-      # for linking purposes. Targets depending on the latter must use
-      # the former as a dependency.
-      allow_circular_includes_from = [ "//chrome/updater/win:lib" ]
       deps += [
         "//chrome/updater/app/server/win:updater_idl_idl",
         "//chrome/updater/app/server/win:updater_internal_idl_idl",
         "//chrome/updater/app/server/win:updater_legacy_idl_idl",
         "//chrome/updater/win:constants",
+        "//chrome/updater/win:lib",
       ]
-      public_deps = [ "//chrome/updater/win:lib" ]
     }
 
     if (is_mac) {
@@ -372,6 +370,7 @@ if (is_win || is_mac) {
         "//chrome/updater/app/server/win:updater_internal_idl_idl",
         "//chrome/updater/app/server/win:updater_legacy_idl_idl",
         "//chrome/updater/win:constants",
+        "//chrome/updater/win:lib",
         "//chrome/updater/win:updater_tests",
       ]
 

chrome/updater/device_management/BUILD.gn

@@ -45,8 +45,8 @@ source_set("device_management") {
   if (is_win) {
     sources += [ "dm_storage_win.cc" ]
     deps += [
-      "//chrome/updater:lib",
       "//chrome/updater/win:constants",
+      "//chrome/updater/win:lib",
     ]
   }
 }
@@ -81,6 +81,6 @@ source_set("unittest") {
   }
 
   if (is_win) {
-    deps += [ "//chrome/updater:lib" ]
+    deps += [ "//chrome/updater/win:lib" ]
   }
 }

chrome/updater/test/test_app/BUILD.gn

@@ -87,8 +87,8 @@ source_set("app_sources") {
       "update_client_win.h",
     ]
     deps += [
-      "//chrome/updater:lib",
       "//chrome/updater/app/server/win:updater_idl_idl",
+      "//chrome/updater/win:lib",
     ]
   }
 }

chrome/updater/win/BUILD.gn

@@ -34,6 +34,7 @@ executable("updater") {
 
   deps = [
     ":app_install_controller",
+    ":lib",
     ":version_resources",
     "//build/win:default_exe_manifest",
     "//chrome/updater:lib",
@@ -73,15 +74,10 @@ source_set("constants") {
 }
 
 source_set("lib") {
-  # This build target and "//chrome/updater:lib" act as a unit. Depending
-  # targets must use "//chrome/updater:lib" as a dependency.
-  visibility = [ "//chrome/updater:lib" ]
-
   sources = [
     "action_handler.cc",
     "group_policy_manager.cc",
     "group_policy_manager.h",
-    "installer.cc",
     "net/net_util.cc",
     "net/net_util.h",
     "net/network.h",
@@ -157,22 +153,25 @@ source_set("install_progress_observer") {
 # compilation units outside the UI itself.
 # TODO(sorin): https://crbug.com/1014311
 source_set("app_install_controller") {
-  visibility = [ "//chrome/updater/win/*" ]
-
-  cflags_cc = [ "-Wno-missing-braces" ]
-
-  sources = [ "app_install_controller.cc" ]
-
-  allow_circular_includes_from = [ "//chrome/updater:lib" ]
-  deps = [
-    ":install_progress_observer",
-    "//base",
-    "//base:i18n",
-    "//chrome/updater:base",
-    "//chrome/updater:lib",
-    "//chrome/updater:version_header",
-    "//chrome/updater/win/ui",
-  ]
+  if (is_win) {
+    visibility = [ "//chrome/updater/win/*" ]
+    allow_circular_includes_from = [ "//chrome/updater:lib" ]
+
+    cflags_cc = [ "-Wno-missing-braces" ]
+
+    sources = [ "app_install_controller.cc" ]
+
+    deps = [
+      ":install_progress_observer",
+      ":lib",
+      "//base",
+      "//base:i18n",
+      "//chrome/updater:base",
+      "//chrome/updater:lib",
+      "//chrome/updater:version_header",
+      "//chrome/updater/win/ui",
+    ]
+  }
 }
 
 source_set("tag_extractor") {
@@ -209,9 +208,9 @@ source_set("updater_tests") {
   deps = [
     ":app_install_controller",
     ":constants",
+    ":lib",
     ":tag_extractor",
     "//base/test:test_support",
-    "//chrome/updater:lib",
     "//testing/gtest",
     "//url:url",
   ]
@@ -242,10 +241,10 @@ test("updater_unittests") {
 
   deps = [
     ":app_install_controller",
+    ":lib",
     "//base",
     "//base/test:test_support",
     "//chrome/updater:branding_header",
-    "//chrome/updater:lib",
     "//chrome/updater/win/test:test_executables",
     "//chrome/updater/win/test:test_strings",
     "//testing/gtest",

chrome/updater/win/test/BUILD.gn

@@ -26,8 +26,8 @@ source_set("test_common") {
   deps = [
     "//base",
     "//chrome/updater:base",
-    "//chrome/updater:lib",
     "//chrome/updater/win:app_install_controller",
+    "//chrome/updater/win:lib",
   ]
 }
 

chrome/updater/win/ui/BUILD.gn

@@ -41,8 +41,8 @@ source_set("ui") {
     "//base:i18n",
     "//chrome/updater:base",
     "//chrome/updater:branding_header",
-    "//chrome/updater:lib",
     "//chrome/updater/win:install_progress_observer",
+    "//chrome/updater/win:lib",
     "//third_party/wtl",
   ]