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

Side by Side Diff: content/DEPS

Issue 1777503003: More strict restrictions for content -> content/shell dependencies. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed unnecessary namespace-qualification. Created 4 years, 9 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 | content/common/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 # 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 files in
3 # src/components that are shared with the mojo html_viewer. 3 # src/components that are shared with the mojo html_viewer.
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/shell", # for content_browsertests
14 "+content/test", 13 "+content/test",
15 14
16 "+blink", 15 "+blink",
17 16
18 "+cc", 17 "+cc",
19 "-cc/blink", 18 "-cc/blink",
20 # If you want to use any of these files, move them to src/base first. 19 # If you want to use any of these files, move them to src/base first.
21 "-cc/base/scoped_ptr_algorithm.h", 20 "-cc/base/scoped_ptr_algorithm.h",
22 "-cc/base/scoped_ptr_deque.h", 21 "-cc/base/scoped_ptr_deque.h",
23 "-cc/base/scoped_ptr_vector.h", 22 "-cc/base/scoped_ptr_vector.h",
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 # 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
106 # explicit. 105 # explicit.
107 "-ui/views", 106 "-ui/views",
108 107
109 "+storage/browser", 108 "+storage/browser",
110 "+storage/common", 109 "+storage/common",
111 110
112 # For generated JNI includes. 111 # For generated JNI includes.
113 "+jni", 112 "+jni",
114 ] 113 ]
114
115 # content -> content/shell dependency is not allowed, except for browser tests.
116 specific_include_rules = {
117 ".*_browsertest[a-z_]*\.cc": [
118 "+content/shell/browser",
119 "+content/shell/common",
120 ],
121 }
122
OLDNEW
« no previous file with comments | « no previous file | content/common/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698