mustash: Restrict //ash/DEPS to services available in ash process

ash/DEPS

Index: ash/DEPS
diff --git a/ash/DEPS b/ash/DEPS
index 006de3c99041b5c6e1bf7a07bee7320518da1479..5942959cd10db0a698fae6beb09c3559a10aeda7 100644
--- a/ash/DEPS
+++ b/ash/DEPS
@@ -1,7 +1,6 @@
 include_rules = [
   "+device/bluetooth",
   "+cc/debug",
-  "+chromeos",
   "+components/quirks",
   "+components/signin/core/account_id",
   "+components/user_manager",
@@ -19,14 +18,26 @@ include_rules = [
   "-ash/host",
   "-content",
 
-  # Only allow D-Bus clients accessible to the ash process on mustash.
-  "-chromeos/dbus",
-  "+chromeos/dbus/dbus_client_types.h",
+  # Only allow Chrome OS services and D-Bus clients accessible to the mustash
+  # ash process.
+  "-chromeos",
+  "+chromeos/accelerometer",
+  "+chromeos/audio",
+  "+chromeos/chromeos_switches.h",
+  "+chromeos/chromeos_paths.h",
   "+chromeos/dbus/dbus_thread_manager.h",
   "+chromeos/dbus/fake_power_manager_client.h",
   "+chromeos/dbus/fake_session_manager_client.h",
+  "+chromeos/dbus/power_manager",
   "+chromeos/dbus/power_manager_client.h",
   "+chromeos/dbus/session_manager_client.h",
+  "+chromeos/dbus/system_clock_client.h",
+  "+chromeos/login",
+  # TODO(jamescook): Eliminate this. http://crbug.com/644355
+  "+chromeos/network",
+  # TODO(jamescook): Eliminate this. http://crbug.com/644361
+  "+chromeos/settings/timezone_settings.h",
+  "+chromeos/system",

[stevenjb, 2016/09/20 18:04:42]

Does it make sense to move some of these down to ash/system/chromeos ?

[James Cook, 2016/09/20 18:14:42]

I prefer to keep them here. Note that there are both //ash/system/chromeos and
//ash/common/system/chromeos directories (for classic ash vs. mustash code). The
former probably won't go away entirely until we turn off classic ash.

Also, usage isn't that concentrated in the system directories:

git grep "include \"chromeos" ash

I think it's easier to reason about the deps if they all live here.

[stevenjb, 2016/09/20 18:20:31]

OK

 ]
 
 specific_include_rules = {