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

Side by Side Diff: content/DEPS

Issue 1921553003: Move permission.mojom from WebKit/public/platform/ to components/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: gn check fix Created 4 years, 7 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
OLDNEW
1 # Do NOT add chrome to the list below. We shouldn't be including files 1 # Do NOT add chrome to the list below. We shouldn't be including files
2 # from src/chrome in src/content. Also, we may only depend on files in 2 # from src/chrome in src/content. Also, we may only depend on mojo interface
jam 2016/05/03 03:32:46 actually you can just remove the second sentence a
xhwang 2016/05/03 05:04:11 Done.
3 # src/components that are shared with the mojo html_viewer. 3 # files in src/components that are shared with other modules.
4 include_rules = [ 4 include_rules = [
5 # The subdirectories in content/ will manually allow their own include 5 # The subdirectories in content/ will manually allow their own include
6 # directories in content/ so we disallow all of them. 6 # directories in content/ so we disallow all of them.
7 "-content", 7 "-content",
8 "+content/app/resources/grit/content_resources.h", 8 "+content/app/resources/grit/content_resources.h",
9 "+content/common", 9 "+content/common",
10 "+content/grit", 10 "+content/grit",
11 "+content/public/common", 11 "+content/public/common",
12 "+content/public/test", 12 "+content/public/test",
13 "+content/test", 13 "+content/test",
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
103 # Content shouldn't depend on views. While we technically don't need this 103 # Content shouldn't depend on views. While we technically don't need this
104 # line, since the top level DEPS doesn't allow it, we add it to make this 104 # line, since the top level DEPS doesn't allow it, we add it to make this
105 # explicit. 105 # explicit.
106 "-ui/views", 106 "-ui/views",
107 107
108 "+storage/browser", 108 "+storage/browser",
109 "+storage/common", 109 "+storage/common",
110 110
111 # For generated JNI includes. 111 # For generated JNI includes.
112 "+jni", 112 "+jni",
113
114 # For permission mojo interfaces.
115 "+components/permissions",
jam 2016/05/03 03:32:46 put this up near the other components/ lines
xhwang 2016/05/03 05:04:11 Moved up. But I don't see other components/ lines.
113 ] 116 ]
114 117
115 # content -> content/shell dependency is not allowed, except for browser tests. 118 # content -> content/shell dependency is not allowed, except for browser tests.
116 specific_include_rules = { 119 specific_include_rules = {
117 ".*_browsertest[a-z_]*\.(cc|h)": [ 120 ".*_browsertest[a-z_]*\.(cc|h)": [
118 "+content/shell/browser", 121 "+content/shell/browser",
119 "+content/shell/common", 122 "+content/shell/common",
120 ], 123 ],
121 } 124 }
122 125
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698