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

Side by Side Diff: content/DEPS

Issue 1965273005: Drop unnecessary media deps from content (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Eliminate empty lists from DEPS 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
« no previous file with comments | « no previous file | content/app/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/test", 13 "+content/test",
14 14
15 "+blink", 15 "+blink",
16 16
17 "+cc", 17 "+cc",
18 "-cc/blink", 18 "-cc/blink",
19 # 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.
20 "-cc/base/scoped_ptr_algorithm.h", 20 "-cc/base/scoped_ptr_algorithm.h",
21 "-cc/base/scoped_ptr_deque.h", 21 "-cc/base/scoped_ptr_deque.h",
22 "-cc/base/scoped_ptr_vector.h", 22 "-cc/base/scoped_ptr_vector.h",
23 23
24 "+crypto", 24 "+crypto",
25 "+grit/blink_resources.h", 25 "+grit/blink_resources.h",
26 "+grit/content_strings.h", 26 "+grit/content_strings.h",
27 27
28 "+dbus", 28 "+dbus",
29 "+gpu", 29 "+gpu",
30 "+media",
jam 2016/05/11 20:39:56 why are you removing this one? what I had meant th
Mark Dittmer 2016/05/12 15:00:37 Done.
31 "+mojo/common", 30 "+mojo/common",
32 "+mojo/edk/embedder", 31 "+mojo/edk/embedder",
33 "+mojo/edk/js", 32 "+mojo/edk/js",
34 "+mojo/edk/test", 33 "+mojo/edk/test",
35 "+mojo/public", 34 "+mojo/public",
36 "+net", 35 "+net",
37 "+ppapi", 36 "+ppapi",
38 "+printing", 37 "+printing",
39 "+sandbox", 38 "+sandbox",
40 "+skia", 39 "+skia",
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 "+jni", 111 "+jni",
113 ] 112 ]
114 113
115 # content -> content/shell dependency is not allowed, except for browser tests. 114 # content -> content/shell dependency is not allowed, except for browser tests.
116 specific_include_rules = { 115 specific_include_rules = {
117 ".*_browsertest[a-z_]*\.(cc|h)": [ 116 ".*_browsertest[a-z_]*\.(cc|h)": [
118 "+content/shell/browser", 117 "+content/shell/browser",
119 "+content/shell/common", 118 "+content/shell/common",
120 ], 119 ],
121 } 120 }
OLDNEW
« no previous file with comments | « no previous file | content/app/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698