Prepare for building with enable_app_list=0 on Desktop

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 172 matching lines @@
       "//printing:printing_unittests",
       "//third_party/cacheinvalidation:cacheinvalidation_unittests",
       "//third_party/codesighs",
       "//third_party/pdfium/samples:pdfium_test",
       "//tools/battor_agent",
       "//tools/battor_agent:battor_agent_unittests",
       "//tools/gn",
       "//tools/gn:gn_unittests",
       "//tools/perf/clear_system_cache",
       "//ui/accessibility:accessibility_unittests",
-      "//ui/app_list:app_list_unittests",
     ]
   }
 
   if (!is_ios) {
     # TODO(GYP): Figure out which of these should actually build on iOS,
     # and whether there should be other targets that are iOS-only and missing.
     deps += [
       "//cc:cc_unittests",
       "//chrome/test:unit_tests",
       "//components:components_browsertests",
@@ skipping 57 matching lines @@
   }
 
   if (enable_remoting) {
     deps += [ "//remoting:remoting_all" ]
   }
 
   if (toolkit_views) {
     deps += [ "//ui/views:views_unittests" ]
   }
 
+  if (enable_app_list && !is_chromecast) {

[Devlin, 2016/05/05 16:01:36]

What's the link between app list and chromecast?

[Dirk Pranke, 2016/05/05 17:13:26]

It seems unlikely to me that cast actually wants or uses app list, so maybe we
should update the default in build/config/features.gni .

[tapted, 2016/05/06 02:36:56]

Done.

+    deps += [ "//ui/app_list:app_list_unittests" ]
+  }
+
+  if (enable_app_list && use_aura) {
+    deps += [
+      # TODO(tapted): Move this into enable_app_list when enable_app_list is not
+      # true on Mac.
+      "//ui/app_list/presenter:app_list_presenter_unittests",
+    ]
+  }
+
   if (use_aura) {
     deps += [
       "//ash:ash_shell_with_content",
       "//ash:ash_unittests",
-      "//ui/app_list/presenter:app_list_presenter_unittests",
       "//ui/aura:aura_unittests",
       "//ui/aura:demo",
       "//ui/wm:wm_unittests",
     ]
   }
 
   if (use_ozone) {
     deps += [ "//ui/ozone" ]
   }
 
@@ skipping 255 matching lines @@
         "//ui/message_center:message_center_unittests",
         "//ui/snapshot:snapshot_unittests",
         "//ui/views/examples:views_examples_with_content_exe",
       ]
 
       if (!is_debug && !is_component_build) {
         deps += [ "//chrome/tools/service_discovery_sniffer" ]
       }
     }
 
-    if (toolkit_views) {
+    if (toolkit_views && enable_app_list) {
       deps += [ "//ui/app_list:app_list_demo" ]
     }
 
     if (use_x11) {
       if (target_cpu != "arm") {
         deps += [ "//gpu/tools/compositor_model_bench" ]
       }
     }
   }
 
@@ skipping 389 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" ]
 }