Update gn_all and gyp_remaining to reflect reality on the win config.

BUILD.gn

 # Copyright (c) 2013 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.
 
 # This is the root build file for GN. GN will start processing by loading this
 # file, and recursively load all dependencies until all dependencies are either
 # resolved or known not to exist (which will cause the build to fail). So if
 # you add a new build file, there must be some path of dependencies from this
 # file to your new one or GN won't know about it.
 
+import("//build/config/crypto.gni")

[brettw, 2015/04/13 21:31:45]

Long-term, this structure is unfortunate since it means all of these files have
to be loaded in series. This is fine for now but we should try to have a better
system in the future that avoids making this list longer.

 import("//build/config/features.gni")
 import("//build/config/ui.gni")
+import("//build/module_args/v8.gni")
 import("//remoting/remoting_host.gni")
 
 if (is_android) {
   import("//build/config/android/config.gni")
 }
 
 declare_args() {
   # A list of extra dependencies to add to the root target. This allows a
   # checkout to add additional targets without explicitly changing any checked-
   # in files.
@@ skipping 88 matching lines @@
     "//third_party/mojo/src/mojo/edk/system:mojo_system_unittests",
     "//third_party/mojo/src/mojo/edk/test:mojo_public_bindings_unittests",
     "//third_party/mojo/src/mojo/edk/test:mojo_public_environment_unittests",
     "//third_party/mojo/src/mojo/edk/test:mojo_public_system_unittests",
     "//third_party/mojo/src/mojo/edk/test:mojo_public_utility_unittests",
     "//third_party/pdfium/samples:pdfium_test",
     "//third_party/smhasher:pmurhash",
     "//tools/imagediff($host_toolchain)",
     "//tools/gn",
     "//tools/gn:gn_unittests",
+    "//tools/gn:generate_test_gn_data",
     "//tools/telemetry:bitmaptools($host_toolchain)",
     "//ui/accessibility:accessibility_unittests",
     "//ui/app_list:app_list_unittests",
     "//ui/base:ui_base_unittests",
     "//ui/display:display_unittests",
     "//ui/events:events_unittests",
     "//ui/gfx:gfx_unittests",
     "//ui/touch_selection:ui_touch_selection_unittests",
     "//url:url_unittests",
   ]
 
   deps += root_extra_deps
 
-  # TODO(GYP): Get this working on the mac?
   if (enable_extensions && !is_mac) {
+    # TODO(GYP): Get this working on the mac?
     deps += [ "//extensions/shell:app_shell_unittests" ]
   }
 
+  if (enable_me2me_host) {
+    deps += [ "//remoting/host:remoting_me2me_host" ]
+  }
+
+  if (enable_media_router) {
+    deps += [
+      "//chrome/browser/media/router/",
+      "//chrome/browser/media/router:unit_tests",
+    ]
+  }
+
   if (enable_remoting_host) {
     deps += [
       "//remoting:remoting_unittests",
       "//remoting:remoting_perftests",
       "//remoting/host",
       "//remoting/host:remoting_start_host",
       "//remoting/host/it2me:remote_assistance_host",
     ]
   }
 
-  if (enable_me2me_host) {
-    deps += [ "//remoting/host:remoting_me2me_host" ]
-  }
-
-  if (!is_win) {
-    deps += [ "//breakpad:symupload" ]
-  }
-
-  if (use_x11) {
-    deps += [ "//tools/xdisplaycheck" ]
-  }
-
   if (toolkit_views) {
     deps += [ "//ui/views:views_unittests" ]
   }
 
   if (use_aura) {
     deps += [ "//ui/wm:wm_unittests" ]
   }
 
   if (use_ozone) {
     deps += [ "//ui/ozone" ]
   }
 
-  if (enable_media_router) {
-    deps += [
-      "//chrome/browser/media/router/",
-      "//chrome/browser/media/router:unit_tests",
-    ]
+  if (use_x11) {
+    deps += [ "//tools/xdisplaycheck" ]
   }
 
-  if (is_win || is_mac || is_chromeos) {
-    # RLZ works on these platforms.
-    # TODO(GYP): Is this target needed, or pulled in automatically?
-    deps += [ "//rlz:rlz_lib" ]
+  if (v8_use_external_startup_data) {
+    deps += [ "//gin:gin_v8_snapshot_fingerprint" ]
   }
 
   if (is_android) {
     deps += [
       "//base/android/linker:chromium_android_linker",
       "//build/android/gyp/test:hello_world",
       "//build/android/rezip",
       "//chrome/android:chrome_shell_apk",
       "//chrome/test/chromedriver/test/webview_shell:chromedriver_webview_shell_apk",
       "//tools/imagediff($host_toolchain)",
@@ skipping 20 matching lines @@
       "//third_party/openmax_dl/dl",
       "//third_party/speex",
       "//ui/android:ui_java",
 
       # TODO(GYP): Are these needed?
       "//chrome/test:test_support_unit",
       "//third_party/smhasher:murmurhash3",
       "//ui/message_center:test_support",
     ]
     deps -= [
-      "//breakpad:symupload",  # TODO(GYP) ??
       "//chrome",  # TODO(GYP) ??
       "//chrome/test:browser_tests",  # TODO(GYP) ??
       "//chrome/test:interactive_ui_tests",  # TODO(GYP) ??
       "//chrome/test:sync_integration_tests",  # TODO(GYP) ??
       "//chrome/test:unit_tests",  # TODO(GYP)
 
       # Chromedriver shouldn't be compiled on Android.
       "//chrome/test/chromedriver:chromedriver_unittests",
       "//extensions:extensions_browsertests",
       "//extensions:extensions_unittests",
@@ skipping 28 matching lines @@
       "//ppapi/examples/scripting",
       "//ppapi/examples/stub",
       "//ppapi/examples/threading",
       "//ppapi/examples/url_loader",
       "//ppapi/examples/video_capture",
       "//ppapi/examples/video_decode",
       "//ppapi/examples/video_effects",
       "//ppapi/examples/video_encode",
       "//third_party/pdfium/samples:pdfium_test",
       "//tools/gn",
+      "//tools/gn:generate_test_gn_data",
       "//tools/gn:gn_unittests",
       "//ui/app_list:app_list_unittests",
       "//url:url_unittests",
     ]
 
     if (has_chrome_android_internal) {
       deps += [ "//clank" ]  # TODO(GYP) ??
     }
   }
 
+  if (is_linux) {  # TODO(GYP): || is_android || is_bsd?
+    deps += [
+      "//breakpad:core-2-minidump",
+      "//breakpad:minidump-2-core",
+    ]
+  }
+
+  if (is_chromeos || is_mac || is_win) {
+    deps += [
+      "//rlz:rlz_id",
+      "//rlz:rlz_lib",
+      "//rlz:rlz_unittests",
+    ]
+  }
+
   if (is_linux) {
     # The following are definitely linux-only.
     deps += [
       "//breakpad:breakpad_unittests",
       "//breakpad:generate_test_dump",
       "//dbus:dbus_test_server",
       "//dbus:dbus_unittests",
       "//net:disk_cache_memory_test",
       "//net:flip_in_mem_edsm_server",
       "//net:flip_in_mem_edsm_server_unittests",
       "//net:quic_client",
       "//net:quic_server",
       "//sandbox/linux:chrome_sandbox",
       "//sandbox/linux:sandbox_linux_unittests",
       "//sandbox/linux:sandbox_linux_jni_unittests",
     ]
 
     if (is_chromeos || use_ash) {
       deps += [ "//components/session_manager/core" ]
     }
   }
 
-  if (is_linux && !is_chromeos) {
+  if (is_win || (is_linux && !is_chromeos)) {
+    # TODO(GYP): Figure out which of these should (and can) build
+    # for android/chromeos/mac/ios.
     deps += [
-      # TODO(GYP): Figure out which of these should (and can) build
-      # under which other conditions.
       "//base:base_perftests",
       "//base:base_i18n_perftests",
       "//base:check_example",
       "//base:protect_file_posix",
       "//base:build_utf8_validator_tables",
-      "//breakpad:core-2-minidump",
-      "//breakpad:minidump-2-core",
-      "//build/sanitizers:copy_llvm_symbolizer",
+      "//cc:cc_perftests",
       "//cc/blink:cc_blink_unittests",
-      "//cc:cc_perftests",
-      "//chrome/test:chrome_app_unittests",
       "//chrome/test:load_library_perf_tests",
       "//chrome/test:performance_browser_tests",
       "//chrome/test:sync_performance_tests",
       "//chrome/test/chromedriver:chromedriver",
       "//chrome/test/chromedriver:chromedriver_tests",
       "//chrome/tools/profile_reset:jtl_compiler",
-      "//cloud_print:cloud_print_unittests",
       "//components:components_perftests",
-      "//components/network_hints/browser",
-      "//components/webui_generator",
-      "//content/public/app:browser",
-      "//content/public/app:child",
       "//content/test:content_gl_tests",
       "//content/test:content_gl_benchmark",
       "//courgette:courgette",
       "//courgette:courgette_fuzz",
       "//courgette:courgette_minimal_tool",
       "//courgette:courgette_unittests",
       "//device:device_unittests",
       "//gin:gin_shell",
-      "//gin:gin_v8_snapshot_fingerprint",
       "//gin:gin_unittests",
       "//google_apis:google_apis_unittests",
       "//google_apis/gcm:mcs_probe",
       "//gpu:angle_unittests",
+      "//gpu:gpu_perftests",
       "//gpu:gl_tests",
+      "//ipc:ipc_perftests",
+      "//media:ffmpeg_regression_tests",  # TODO(GYP) this should be conditional on media_use_ffmpeg
+      "//media:media_perftests",
+      "//media/cast:generate_barcode_video",
+      "//media/cast:generate_timecode_audio",
+      "//net:crash_cache",
+      "//net:crl_set_dump",
+      "//net:dns_fuzz_stub",
+      "//net:dump_cache",
+      "//net:gdig",
+      "//net:get_server_time",
+      "//net:net_watcher",  # TODO(GYP): This should be conditional on use_v8_in_net
+      "//net:run_testserver",
+      "//net:stress_cache",
+      "//net:tld_cleanup",
+      "//ppapi:pepper_hash_for_uma",
+      "//ppapi:ppapi_perftests",
+      "//sync:run_sync_testserver",
+      "//sync:sync_endtoend_tests",
+      "//third_party/codesighs:maptsvdifftool",
+      "//third_party/leveldatabase:env_chromium_unittests",
+      "//third_party/libaddressinput:libaddressinput_unittests",
+      "//third_party/libphonenumber:libphonenumber_unittests",
+      "//ui/compositor:compositor_unittests",
+    ]
+
+    if (enable_extensions) {
+      deps += [ "//extensions/shell:app_shell" ]
+    }
+
+    if (enable_nacl) {
+      deps += [ "//components/nacl:nacl_loader_unittests" ]
+    }
+
+    if (enable_nacl && enable_remoting) {
+      deps += [ "//remoting:remoting_key_tester" ]
+    }
+
+    if (use_ash) {
+      deps += [
+        "//ash:ash_shell",
+        "//ash:ash_shell_unittests",
+        "//ash:ash_unittests",
+      ]
+    }
+
+    if (use_aura) {
+      deps += [
+        "//ui/aura:aura_unittests",
+        "//ui/aura:bench",
+        "//ui/aura:demo",
+      ]
+    }
+  }
+
+  if (is_linux && !is_chromeos) {
+    deps += [
+      # TODO(GYP): Figure out which of these should (and can) build
+      # under which other conditions.
+      "//build/sanitizers:copy_llvm_symbolizer",
+      "//chrome/test:chrome_app_unittests",
+      "//cloud_print:cloud_print_unittests",
+      "//components/network_hints/browser",
+      "//components/webui_generator",
+      "//content/public/app:browser",
+      "//content/public/app:child",
 
       # TODO(GYP): Remove this when the gles2 tests work
       "//gpu/command_buffer/client:gles2_implementation_no_check",
 
-      "//gpu:gpu_perftests",
       "//gpu/khronos_glcts_support:khronos_glcts_test",  # TODO(GYP) crbug.com/471903 to make this complete.
-      "//ipc:ipc_perftests",
-      "//media:ffmpeg_regression_tests",  # TODO(GYP) this should be conditional on media_use_ffmpeg
-      "//media:media_perftests",
       "//media/cast:cast_benchmarks",
-      "//media/cast:generate_barcode_video",
-      "//media/cast:generate_timecode_audio",
       "//media/cast:tap_proxy",
       "//mojo/application",
-      "//net:crash_cache",
-      "//net:crl_set_dump",
-      "//net:dns_fuzz_stub",
-      "//net:gdig",
-      "//net:get_server_time",
-      "//net:net_watcher",  # TODO(GYP): This should be conditional on use_v8_in_net
-      "//net:stress_cache",
-      "//net:tld_cleanup",
-      "//net:run_testserver",
-      "//net:dump_cache",
-      "//ppapi:pepper_hash_for_uma",
-      "//ppapi:ppapi_perftests",  # TODO(GYP): Are there other ppapi_* test targets?
       "//skia:filter_fuzz_stub",
       "//skia:image_operations_bench",
-      "//sync:run_sync_testserver",
-      "//sync:sync_endtoend_tests",
       "//sync/tools:sync_client",
       "//sync/tools:sync_listen_notifications",
       "//testing/gmock:gmock_main",
-      "//third_party/codesighs:maptsvdifftool",
-      "//third_party/libphonenumber:libphonenumber_unittests",
       "//third_party/mojo/src/mojo/edk/test:mojo_public_system_perftests",
-      "//tools/gn:generate_test_gn_data",
       "//tools/perf/clear_system_cache",
       "//ui/keyboard:keyboard_unittests",
       "//ui/message_center:message_center_unittests",
       "//ui/snapshot:snapshot_unittests",
       "//ui/views/examples:views_examples_with_content_exe",
 
       # "//v8:v8_snapshot",  # TODO(GYP): visibility?
       # "//v8:postmortem-metadata",  # TODO(GYP): visibility?
 
       "//third_party/codesighs:nm2tsv",
-      "//third_party/leveldatabase:env_chromium_unittests",
-      "//third_party/libaddressinput:libaddressinput_unittests",
       "//third_party/sqlite:sqlite_shell",
-      "//ui/compositor:compositor_unittests",
     ]
 
     if (current_toolchain == host_toolchain) {
       # Do not build the breakpad utilities in cross-compiles.
       deps += [
         "//breakpad:dump_syms",
         "//breakpad:microdump_stackwalk",
         "//breakpad:minidump_dump",
         "//breakpad:minidump_stackwalk",
       ]
     }
 
-    if (enable_extensions) {
-      deps += [ "//extensions/shell:app_shell" ]
-    }
-
-    if (enable_nacl) {
-      deps += [
-        "//components/nacl:nacl_loader_unittests",
-        "//remoting:remoting_key_tester",
-      ]
-    }
-
     if (!is_debug && !is_component_build) {
       deps += [ "//chrome/tools/service_discovery_sniffer" ]
     }
 
     if (toolkit_views) {
       deps += [ "//ui/app_list:app_list_demo" ]
     }
 
-    if (use_ash) {
-      deps += [
-        "//ash:ash_shell",
-        "//ash:ash_shell_unittests",
-        "//ash:ash_unittests",
-      ]
-    }
-
-    if (use_aura) {
-      deps += [
-        "//ui/aura:aura_unittests",
-        "//ui/aura:bench",
-        "//ui/aura:demo",
-      ]
-    }
-
     if (use_x11) {
       deps += [ "//media:player_x11" ]
       if (target_cpu != "arm") {
         deps += [ "//gpu/tools/compositor_model_bench" ]
       }
     }
   }
 
   if (is_mac) {
     deps += [
       "//breakpad:crash_inspector",
       "//breakpad:dump_syms",
-      "//breakpad:symupload",
       "//third_party/apple_sample_code",
       "//third_party/molokocacao",
     ]
 
     # TODO(GYP): Remove these when the targets below work and these
     # are pulled in automatically.
     deps += [
       "//cc/blink",
       "//components/ui/zoom:ui_zoom",
       "//content",
@@ skipping 32 matching lines @@
       "//components:components_unittests",  # TODO(GYP)
       "//content/test:content_browsertests",  # TODO(GYP)
       "//content/test:content_perftests",  # TODO(GYP)
       "//content/test:content_unittests",  # TODO(GYP)
       "//extensions:extensions_browsertests",  # TODO(GYP)
       "//extensions:extensions_unittests",  # TODO(GYP)
       "//net:net_unittests",  # TODO(GYP)
       "//ui/app_list:app_list_unittests",  # TODO(GYP)
       "//ui/gfx:gfx_unittests",  # TODO(GYP)
     ]
-  } else if (is_win) {
-    deps += [ "//ui/metro_viewer" ]
+  }
+
+  if (is_win) {
+    deps += [
+      "//base:pe_image_test",
+      "//chrome_elf:chrome_elf_unittests",
+      "//chrome_elf:dll_hash_main",
+
+      # "//components/crash/tools:crash_service", TODO(GYP) - doesn't fully build yet.
+      "//components/wifi:wifi_test",
+      "//net:quic_client",
+      "//net:quic_server",
+      "//sandbox/win:pocdll",
+      "//sandbox/win:sandbox_poc",
+      "//sandbox/win:sbox_integration_tests",
+      "//sandbox/win:sbox_unittests",
+      "//sandbox/win:sbox_validation_tests",
+      "//testing/gtest:gtest_main",
+      "//third_party/codesighs:msmap2tsv",
+      "//third_party/pdfium/samples:pdfium_diff",
+      "//ui/metro_viewer",
+    ]
     deps -= [
       "//crypto:crypto_unittests",  # TODO(GYP)
       "//net:net_unittests",  # TODO(GYP)
     ]
+  } else {
+    if (!is_android) {
+      # TODO(GYP): Make this work on android also.
+      deps += [ "//breakpad:symupload" ]
+    }
   }
 }
 
 group("gn_only") {
   testonly = true
 
   if (is_linux && !is_chromeos) {
     # TODO(GYP): Figure out if any of these should be in gn_all
     # and figure out how cross-platform they are
     deps = [
@@ skipping 204 matching lines @@
 
       # Note:
       # (*) Fails but failures match GYP build at time of testing.
     ]
 
     if (enable_nacl) {
       deps += [ "//components/nacl:nacl_loader_unittests" ]  # PASSES 3/28/2015
     }
   }
 }