ash: removes unnecessary includes and dep

ash/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/features.gni")
 import("//build/config/ui.gni")
 import("//testing/test.gni")
 import("//ui/base/ui_features.gni")
 
 # Historical note: Ash shipped on Windows for a couple years to support
@@ skipping 838 matching lines @@
   ]
 
   configs += [ "//build/config:precompiled_headers" ]
   defines = [ "ASH_IMPLEMENTATION" ]
 
   public_deps = [
     "//ash/common/strings",
     "//ash/resources",
     "//ash/resources/vector_icons",
     "//ash/shared:app_types",
-
-    # This is a tiny dependency that just defines some service name strings & is
-    # not equivalent to depending on content in a more substantial manner.
-    "//content/public/common:service_names",

[James Cook, 2017/01/23 17:25:05]

Per offline discussion, can you ping ben and see if we still need this to be a
separate target? My understanding was that the reason it's separated out is to
allow ash to connect to things it shouldn't (e.g. content_browser).

[sky, 2017/01/23 17:27:41]

Will do.

   ]
   deps = [
     "//ash/autoclick/common:autoclick",
     "//ash/public/cpp",
     "//ash/public/interfaces",
     "//ash/touch_hud",
     "//base",
     "//base:i18n",
     "//base/third_party/dynamic_annotations",
     "//cc",
@@ skipping 554 matching lines @@
     "//content/public/app:both",
     "//device/bluetooth",
   ]
 
   data_deps = [
     "//ash/common/strings:ash_test_strings",
     "//ash/resources:ash_test_resources_with_content_100_percent",
     "//ash/resources:ash_test_resources_200_percent",
   ]
 }