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

Side by Side Diff: content/DEPS

Issue 8316004: Update DEPS so that only browser directories can access content public API for browser. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 2 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 | Annotate | Revision Log
« no previous file with comments | « chrome/browser/DEPS ('k') | content/browser/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 from 1 # Do NOT add chrome to the list below. We shouldn't be including files from
2 # src/chrome in src/content. 2 # src/chrome in src/content.
3 include_rules = [ 3 include_rules = [
4 "+crypto", 4 "+crypto",
5 5
6 "-chrome", 6 "-chrome",
7 # Until we move most of our testing infrastructure in a way that content can 7 # Until we move most of our testing infrastructure in a way that content can
8 # make use of it. 8 # make use of it.
9 "+chrome/test", 9 "+chrome/test",
10 10
11 # The following directories have been refactored, so no new dependencies 11 # The following directories have been refactored, so no new dependencies
12 # should be added from these directories. 12 # should be added from these directories.
13 "-chrome/browser/printing", 13 "-chrome/browser/printing",
14 "-chrome/browser/extensions", 14 "-chrome/browser/extensions",
15 "-chrome/common/extensions", 15 "-chrome/common/extensions",
16 "-chrome/renderer/extensions", 16 "-chrome/renderer/extensions",
17 17
18 # The subdirectories in content/ will manually allow their own include 18 # The subdirectories in content/ will manually allow their own include
19 # directories in content/ so we disallow all of them. 19 # directories in content/ so we disallow all of them.
20 "-content", 20 "-content",
21 "+content/common", 21 "+content/common",
22 "+content/public/browser",
23 "+content/public/common", 22 "+content/public/common",
24 "+content/test", 23 "+content/test",
25 24
25 # TODO(ben): remove this
26 "+content/public/browser/native_web_keyboard_event.h",
27
26 "+dbus", 28 "+dbus",
27 "+grit", 29 "+grit",
28 "+gpu", 30 "+gpu",
29 "+net", 31 "+net",
30 "+ppapi", 32 "+ppapi",
31 "+printing", 33 "+printing",
32 "+sandbox", 34 "+sandbox",
33 "+skia", 35 "+skia",
34 36
35 # Don't allow inclusion of these other libs we shouldn't be calling directly. 37 # Don't allow inclusion of these other libs we shouldn't be calling directly.
(...skipping 23 matching lines...) Expand all
59 # Content shouldn't depend on views. While we technically don't need this 61 # Content shouldn't depend on views. While we technically don't need this
60 # line, since the top level DEPS doesn't allow it, we add it to make this 62 # line, since the top level DEPS doesn't allow it, we add it to make this
61 # explicit. 63 # explicit.
62 "-views", 64 "-views",
63 65
64 # Aura Shell consumes views, so it too is banned. 66 # Aura Shell consumes views, so it too is banned.
65 "-ui/aura_shell", 67 "-ui/aura_shell",
66 68
67 "+webkit", 69 "+webkit",
68 ] 70 ]
OLDNEW
« no previous file with comments | « chrome/browser/DEPS ('k') | content/browser/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698