Don't disable incremental linking for chrome.dll in component

chrome/BUILD.gn

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//build/buildflag_header.gni")
 import("//build/config/chrome_build.gni")
 import("//build/config/compiler/compiler.gni")
 import("//build/config/features.gni")
 import("//build/config/locales.gni")
 import("//build/config/sanitizers/sanitizers.gni")
@@ skipping 264 matching lines @@
       defines += [ "CHROME_MULTIPLE_DLL" ]
       data_deps += [ ":chrome_child" ]
     }
   }
 }  # !is_android && !is_mac
 
 if (is_win) {
   # This target is a forwarding target to compile the necessary DLLs used
   # by Chrome.
   group("chrome_dll") {
-    # TODO(GYP) support incremental_chrome_dll on Windows for faster links in
-    # developer component builds. When that's supported, this target will need
-    # to become more complicated.
     data_deps = [
       ":main_dll",
     ]
     if (is_multi_dll_chrome) {
       data_deps += [ ":chrome_child" ]
     }
   }
 
   shared_library("main_dll") {
     configs += [ "//build/config/compiler:wexit_time_destructors" ]
@@ skipping 48 matching lines @@
         "/DELAYLOAD:iphlpapi.dll",
         "/DELAYLOAD:setupapi.dll",
         "/DELAYLOAD:urlmon.dll",
         "/DELAYLOAD:winhttp.dll",
         "/DELAYLOAD:wininet.dll",
         "/DELAYLOAD:winspool.drv",
         "/DELAYLOAD:ws2_32.dll",
         "/DELAYLOAD:wsock32.dll",
       ]
 
-      # This is a large module that can't do incremental linking in some cases.
-      configs -= [ "//build/config/win:default_incremental_linking" ]
-      configs +=
-          [ "//build/config/win:default_large_module_incremental_linking" ]
+      if (!is_component_build) {
+        # This is a large module that can't do incremental linking in some cases.

[brettw, 2016/08/10 23:29:48]

80 col rapping.

[scottmg, 2016/08/10 23:40:39]

I really need to fix that in gn format some day. Some penalty doesn't quite work
or something, I forget.

+        configs -= [ "//build/config/win:default_incremental_linking" ]
+        configs +=
+            [ "//build/config/win:default_large_module_incremental_linking" ]
+      }
     }
 
     if (use_aura) {
       deps += [ "//ui/compositor" ]
     }
 
     if (is_multi_dll_chrome) {
       defines += [ "CHROME_MULTIPLE_DLL_BROWSER" ]
       deps += [ "//content/public/app:browser" ]
     } else {
@@ skipping 1397 matching lines @@
       "//chrome/app/theme/$branding_path_component/product_logo_48.png",
       "//chrome/tools/build/linux/chrome-wrapper",
       "//third_party/xdg-utils/scripts/xdg-mime",
       "//third_party/xdg-utils/scripts/xdg-settings",
     ]
     outputs = [
       "$root_out_dir/{{source_file_part}}",
     ]
   }
 }