Remove obsolete app_list Linux code.

chrome/test/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/config/chrome_build.gni")
 import("//build/config/compiler/compiler.gni")
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//chrome/common/features.gni")
@@ skipping 2533 matching lines @@
         # Tests for non mobile and non CrOS (includes Linux, Win, Mac).
         "../browser/metrics/desktop_session_duration/audible_contents_tracker_browsertest.cc",
         "../browser/metrics/desktop_session_duration/chrome_visibility_observer_browsertest.cc",
         "../browser/profiles/profile_statistics_browsertest.cc",
       ]
     }
     if (enable_app_list) {
       sources += [
         "../browser/apps/drive/drive_app_converter_browsertest.cc",
         "../browser/apps/drive/drive_app_provider_browsertest.cc",
         "../browser/ui/app_list/app_list_controller_browsertest.cc",

[tapted, 2016/12/16 05:11:41]

There are some #if (OS_CHROMEOS) in here

[Matt Giuca, 2016/12/21 06:18:03]

Done.

         "../browser/ui/app_list/app_list_service_impl_browsertest.cc",

[tapted, 2016/12/16 05:11:41]

There are lots (more)  #if (OS_CHROMEOS) in this file

[Matt Giuca, 2016/12/21 06:18:03]

Done.

-        "../browser/ui/app_list/app_list_service_views_browsertest.cc",
         "../browser/ui/app_list/search/webstore/webstore_provider_browsertest.cc",
         "../browser/ui/app_list/speech_recognizer_browsertest.cc",
       ]
-      if (is_mac) {
-        # This assumes the AppList is views-based, but Mac only links
-        # browser parts for the Cocoa implementation.
-        sources -=
-            [ "../browser/ui/app_list/app_list_service_views_browsertest.cc" ]
-      }
     }
     if (enable_service_discovery) {
       sources += [ "../browser/extensions/api/mdns/mdns_apitest.cc" ]
     }
     if (enable_supervised_users) {
       sources += [
         "../browser/supervised_user/supervised_user_browsertest.cc",
         "../browser/supervised_user/supervised_user_resource_throttle_browsertest.cc",
         "../browser/supervised_user/supervised_user_service_browsertest.cc",
       ]
@@ skipping 2249 matching lines @@
       "../browser/ui/webui/signin/signin_supervised_user_import_handler_unittest.cc",
     ]
   }
   if (safe_browsing_mode == 1 && enable_extensions) {
     sources += [ "../browser/extensions/blacklist_unittest.cc" ]
   }
   if (enable_app_list) {
     sources += [
       "../browser/apps/drive/drive_app_mapping_unittest.cc",
       "../browser/ui/app_list/app_context_menu_unittest.cc",
-      "../browser/ui/app_list/app_list_positioner_unittest.cc",
       "../browser/ui/app_list/app_list_service_unittest.cc",

[tapted, 2016/12/16 05:11:41]

This has some profile switching stuff that will probably come out in the wash
with ProfileLoader being purged

[Matt Giuca, 2016/12/21 06:18:03]

Acknowledged.

       "../browser/ui/app_list/app_list_test_util.cc",
       "../browser/ui/app_list/app_list_test_util.h",
       "../browser/ui/app_list/extension_app_model_builder_unittest.cc",
       "../browser/ui/app_list/profile_loader_unittest.cc",
       "../browser/ui/app_list/search/app_search_provider_unittest.cc",
       "../browser/ui/app_list/search/history_unittest.cc",
       "../browser/ui/app_list/search/omnibox_result_unittest.cc",
       "../browser/ui/app_list/search/suggestions/suggestions_search_provider_unittest.cc",
       "../browser/ui/app_list/speech_auth_helper_unittest.cc",
       "../browser/ui/app_list/test/fake_profile.cc",
       "../browser/ui/app_list/test/fake_profile.h",
       "../browser/ui/app_list/test/fake_profile_store.cc",
       "../browser/ui/app_list/test/fake_profile_store.h",

[tapted, 2016/12/16 05:11:41]

All the fake_profile goop probably isn't needed either

[Matt Giuca, 2016/12/21 06:18:03]

Acknowledged.

-      "../browser/ui/app_list/test/fast_show_pickler_unittest.cc",
-      "../browser/ui/views/app_list/linux/app_list_linux_unittest.cc",
     ]
     deps += [ "//ui/app_list:test_support" ]
   }
   if (enable_app_list && is_chromeos) {

[tapted, 2016/12/16 05:11:41]

I guess this can be simplified in a follow-up?

[Matt Giuca, 2016/12/21 06:18:03]

Done here.

     sources += [
       "../browser/ui/app_list/arc/arc_app_test.cc",
       "../browser/ui/app_list/arc/arc_app_test.h",
       "../browser/ui/app_list/arc/arc_app_unittest.cc",
       "../browser/ui/app_list/search/launcher_search/launcher_search_icon_image_loader_unittest.cc",
       "../browser/ui/views/apps/app_info_dialog/app_info_dialog_ash_unittest.cc",
     ]
-    sources -=
-        [ "../browser/ui/views/app_list/linux/app_list_linux_unittest.cc" ]
     deps += [ "//components/arc:arc_test_support" ]
   }
   if (enable_plugins && !enable_plugin_installation) {
     sources -= [ "../browser/plugins/plugin_installer_unittest.cc" ]
   }
 }
 
 if (safe_browsing_mode == 1 && is_mac) {
   action("mac_safe_browsing_test_data") {
     script = "//build/gn_run_binary.py"
@@ skipping 220 matching lines @@
 }
 
 if (is_win) {
   loadable_module("conflicts_dll") {
     testonly = true
     sources = [
       "conflicts/conflicts_dll.cc",
     ]
   }
 }