Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(310)

Side by Side Diff: ash/DEPS

Issue 2352973003: mustash: Restrict //ash/DEPS to services available in ash process (Closed)
Patch Set: remove some +chromeos, restructure Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | ash/common/system/chromeos/DEPS » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 include_rules = [ 1 include_rules = [
2 "+device/bluetooth", 2 "+device/bluetooth",
3 "+cc/debug", 3 "+cc/debug",
4 "+chromeos",
5 "+components/quirks", 4 "+components/quirks",
6 "+components/signin/core/account_id", 5 "+components/signin/core/account_id",
7 "+components/user_manager", 6 "+components/user_manager",
8 "+components/wallpaper", 7 "+components/wallpaper",
9 "+gpu/config", 8 "+gpu/config",
10 "+grit/ash_resources.h", 9 "+grit/ash_resources.h",
11 "+grit/ash_strings.h", 10 "+grit/ash_strings.h",
12 "+media", 11 "+media",
13 "+skia/ext", 12 "+skia/ext",
14 "+third_party/cros_system_api", 13 "+third_party/cros_system_api",
15 "+third_party/skia", 14 "+third_party/skia",
16 "+net", 15 "+net",
17 "+ui", 16 "+ui",
18 "+win8", 17 "+win8",
19 "-ash/host", 18 "-ash/host",
20 "-content", 19 "-content",
21 20
22 # Only allow D-Bus clients accessible to the ash process on mustash. 21 # Only allow Chrome OS services and D-Bus clients accessible to the mustash
23 "-chromeos/dbus", 22 # ash process.
24 "+chromeos/dbus/dbus_client_types.h", 23 "-chromeos",
24 "+chromeos/accelerometer",
25 "+chromeos/audio",
26 "+chromeos/chromeos_switches.h",
27 "+chromeos/chromeos_paths.h",
25 "+chromeos/dbus/dbus_thread_manager.h", 28 "+chromeos/dbus/dbus_thread_manager.h",
26 "+chromeos/dbus/fake_power_manager_client.h", 29 "+chromeos/dbus/fake_power_manager_client.h",
27 "+chromeos/dbus/fake_session_manager_client.h", 30 "+chromeos/dbus/fake_session_manager_client.h",
31 "+chromeos/dbus/power_manager",
28 "+chromeos/dbus/power_manager_client.h", 32 "+chromeos/dbus/power_manager_client.h",
29 "+chromeos/dbus/session_manager_client.h", 33 "+chromeos/dbus/session_manager_client.h",
34 "+chromeos/dbus/system_clock_client.h",
35 "+chromeos/login",
36 # TODO(jamescook): Eliminate this. http://crbug.com/644355
37 "+chromeos/network",
38 # TODO(jamescook): Eliminate this. http://crbug.com/644361
39 "+chromeos/settings/timezone_settings.h",
40 "+chromeos/system",
stevenjb 2016/09/20 18:04:42 Does it make sense to move some of these down to a
James Cook 2016/09/20 18:14:42 I prefer to keep them here. Note that there are bo
stevenjb 2016/09/20 18:20:31 OK
30 ] 41 ]
31 42
32 specific_include_rules = { 43 specific_include_rules = {
33 "root_window_controller\.*": [ 44 "root_window_controller\.*": [
34 "+ash/host" 45 "+ash/host"
35 ], 46 ],
36 "shell.cc": [ 47 "shell.cc": [
37 "+ash/host/ash_window_tree_host_init_params.h" 48 "+ash/host/ash_window_tree_host_init_params.h"
38 ], 49 ],
39 "touch_transformer_controller\.*": [ 50 "touch_transformer_controller\.*": [
40 "+ash/host" 51 "+ash/host"
41 ], 52 ],
42 } 53 }
OLDNEW
« no previous file with comments | « no previous file | ash/common/system/chromeos/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698