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

chrome/BUILD.gn

Index: chrome/BUILD.gn
diff --git a/chrome/BUILD.gn b/chrome/BUILD.gn
index 4505493152be345f94feebc57fcac810b4acff8e..bf736d929ed34f1e5935ef2066d23b70de952293 100644
--- a/chrome/BUILD.gn
+++ b/chrome/BUILD.gn
@@ -282,9 +282,6 @@ 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",
     ]
@@ -353,10 +350,13 @@ if (is_win) {
         "/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.
+        configs -= [ "//build/config/win:default_incremental_linking" ]
+        configs +=
+            [ "//build/config/win:default_large_module_incremental_linking" ]
+      }

[Nico, 2017/08/16 20:07:26]

This basically reverts
https://codereview.chromium.org/1930073002/diff/20001/chrome/BUILD.gn

I think the idea here was that default_large_module_incremental_linking does the
checking so that its clients don't have to do any itself. (But then it didn't do
it quite right, https://chromium-review.googlesource.com/c/617764/). Once
https://chromium-review.googlesource.com/c/617764/ is in, I think this should be
reverted -- does that sound right?

[scottmg, 2017/08/16 20:22:57]

Yeah.

     }
 
     if (use_aura) {