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

Side by Side Diff: chrome/DEPS

Issue 10515007: Tighten up DEPS checking to enforce that embedders of content only include public test headers. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 6 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 | « no previous file | chrome/test/DEPS » ('j') | chrome/test/DEPS » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 include_rules = [ 1 include_rules = [
2 "+ash", 2 "+ash",
3 "+crypto", 3 "+crypto",
4 "+gpu", 4 "+gpu",
5 "+native_client", 5 "+native_client",
6 "+net", 6 "+net",
7 "+printing", 7 "+printing",
8 "+sql", 8 "+sql",
9 # Browser, renderer, common and tests access V8 for various purposes. 9 # Browser, renderer, common and tests access V8 for various purposes.
10 "-v8", 10 "-v8",
11 "+v8/include", 11 "+v8/include",
12 12
13 # The subdirectories in chrome/ will manually allow their own include 13 # The subdirectories in chrome/ will manually allow their own include
14 # directories in chrome/ so we disallow all of them. 14 # directories in chrome/ so we disallow all of them.
15 "-chrome", 15 "-chrome",
16 "+chrome/common", 16 "+chrome/common",
17 "+chrome/test", 17 "+chrome/test",
18 "-content",
19 # TODO(jam): remove me once chrome only consumes content/common through its 18 # TODO(jam): remove me once chrome only consumes content/common through its
20 # public headers. 19 # public headers.
21 "+content/common", 20 "+content/common",
22 "+content/public/common", 21 "+content/public/common",
23 "+content/public/test", 22 "+content/public/test",
24 "+content/test", 23 "+content/test/gpu",
24 "+content/test/net",
25 "+content/test/browser_test_base.h", # http://crbug.com/90448
25 26
26 # TODO(ben): remove this 27 # TODO(ben): remove this
27 "+content/public/browser/native_web_keyboard_event.h", 28 "+content/public/browser/native_web_keyboard_event.h",
28 29
29 # Don't allow inclusion of these other libs we shouldn't be calling directly. 30 # Don't allow inclusion of these other libs we shouldn't be calling directly.
30 "-webkit", 31 "-webkit",
31 "-tools", 32 "-tools",
32 33
33 "-crypto/third_party", 34 "-crypto/third_party",
34 35
(...skipping 12 matching lines...) Expand all
47 "+third_party/icon_family", # IconFamily for Mac. 48 "+third_party/icon_family", # IconFamily for Mac.
48 "+third_party/libxml", 49 "+third_party/libxml",
49 "+third_party/mozilla", # Mozilla interface headers. 50 "+third_party/mozilla", # Mozilla interface headers.
50 "+third_party/npapi", # NPAPI interface headers. 51 "+third_party/npapi", # NPAPI interface headers.
51 "+third_party/skia", 52 "+third_party/skia",
52 "+third_party/tcmalloc", # tcmallo 53 "+third_party/tcmalloc", # tcmallo
53 "+third_party/GTM", # Google Toolbox for Mac. 54 "+third_party/GTM", # Google Toolbox for Mac.
54 55
55 "+ui", 56 "+ui",
56 ] 57 ]
OLDNEW
« no previous file with comments | « no previous file | chrome/test/DEPS » ('j') | chrome/test/DEPS » ('J')

Powered by Google App Engine
This is Rietveld 408576698