Remove ash_unittests/mash_unittests from Windows build targets

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.
 
@@ skipping 329 matching lines @@
   }
 
   if (toolkit_views) {
     deps += [
       "//ui/views:views_unittests",
       "//ui/views/examples:views_examples_exe",
       "//ui/views/examples:views_examples_with_content_exe",
     ]
   }
 
+  if (use_ash) {

[James Cook, 2016/12/01 18:29:53]

use_ash is currently is_chromeos && !is_chromecast

+    deps += [
+      "//ash:ash_content_unittests",
+      "//ash:ash_shell_with_content",
+      "//ash:ash_unittests",
+    ]
+  }
+
   if (use_aura) {
     deps += [
-      "//ash:ash_content_unittests",
-      "//ash:ash_shell_with_content",
-      "//ash:ash_unittests",
       "//ui/app_list:app_list_unittests",
       "//ui/app_list/presenter:app_list_presenter_unittests",
       "//ui/aura:aura_unittests",
       "//ui/aura:demo",
       "//ui/wm:wm_unittests",
     ]
   }
 
   if (is_linux && !is_chromeos && !is_chromecast && !use_ozone) {
     # TODO(thomasanderson): Remove this once we build using
@@ skipping 104 matching lines @@
       "//breakpad:breakpad_unittests",
       "//breakpad:core-2-minidump",
       "//breakpad:generate_test_dump",
       "//breakpad:minidump-2-core",
     ]
   }
 
   if (is_chromeos) {
     deps += [
       "//chromeos:chromeos_unittests",
+      "//components/session_manager/core",

[James Cook, 2016/12/01 18:29:53]

This is just cleanup.

       "//ui/arc:ui_arc_unittests",
       "//ui/chromeos:ui_chromeos_unittests",
     ]
   }
 
   if (is_chromeos || is_mac || is_win) {
     deps += [
       "//rlz:rlz_id",
       "//rlz:rlz_lib",
       "//rlz:rlz_unittests",
@@ skipping 12 matching lines @@
       "//sandbox/linux:sandbox_linux_unittests",
     ]
 
     if (use_dbus) {
       deps += [
         "//dbus:dbus_test_server",
         "//dbus:dbus_unittests",
       ]
     }
 
-    if (is_chromeos || use_ash) {
-      deps += [ "//components/session_manager/core" ]
-    }
-
     if (is_chrome_branded && is_official_build) {
       # TODO(dpranke): add the linux_dump_symbols flag?
       deps += [ "//chrome:linux_symbols" ]
     }
   }
 
   if (is_ios || is_win || (is_linux && !is_chromeos)) {
     deps += [
       "//base:base_i18n_perftests",
       "//base:base_perftests",
@@ skipping 572 matching lines @@
   assert(target_name != "")  # Mark as used.
   sources = invoker.actual_sources
   assert(
       sources == invoker.actual_sources,
       "Do not use a platform name in your output directory (found \"$root_build_dir\"). http://crbug.com/548283")
 }
 
 assert_valid_out_dir("_unused") {
   actual_sources = [ "$root_build_dir/foo" ]
 }