Chromium Code Reviews Chromium Code Reviews
Issue 11573034:
Add DEPS rules to help enforce the general rule that the embedder should provide URLs and access to… (Closed)
Base URL: svn://svn.chromium.org/chrome/trunk/src| OLD | NEW |
|---|---|
| 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 "+cc", | 4 "+cc", |
| 5 | 5 |
| 6 "+crypto", | 6 "+crypto", |
| 7 | 7 |
| 8 # The subdirectories in content/ will manually allow their own include | 8 # The subdirectories in content/ will manually allow their own include |
| 9 # directories in content/ so we disallow all of them. | 9 # directories in content/ so we disallow all of them. |
| 10 "-content", | 10 "-content", |
| 11 "+content/common", | 11 "+content/common", |
| 12 "+content/port/common", | 12 "+content/port/common", |
| 13 "+content/public/common", | 13 "+content/public/common", |
| 14 "+content/public/test", | 14 "+content/public/test", |
| 15 "+content/shell", # for content_browsertests | 15 "+content/shell", # for content_browsertests |
| 16 "+content/test", | 16 "+content/test", |
| 17 | 17 |
| 18 "+google_apis", | |
| 19 | |
| 20 "+grit/content_resources.h", | 18 "+grit/content_resources.h", |
| 21 "+grit/ui_resources.h", | 19 "+grit/ui_resources.h", |
| 22 "+grit/webkit_chromium_resources.h", | 20 "+grit/webkit_chromium_resources.h", |
| 23 "+grit/webkit_resources.h", | 21 "+grit/webkit_resources.h", |
| 24 "+grit/webkit_strings.h", | 22 "+grit/webkit_strings.h", |
| 25 | 23 |
| 26 "+dbus", | 24 "+dbus", |
| 27 "+gpu", | 25 "+gpu", |
| 28 "+net", | 26 "+net", |
| 29 "+ppapi", | 27 "+ppapi", |
| 30 "+printing", | 28 "+printing", |
| 31 "+sandbox", | 29 "+sandbox", |
| 32 "+skia", | 30 "+skia", |
| 33 | 31 |
| 32 # In general, content/ should not rely on google_apis, since URLs | |
| 33 # and access tokens should usually be provided by the | |
| 34 # embedder. | |
| 35 # | |
| 36 # There are a couple of specific parts of content that are excepted | |
| 37 # from this rule, see content/browser/speech/DEPS and | |
| 38 # content/browser/geolocation/DEPS. Both of these are cases of | |
|
joth
2012/12/14 23:53:39
content/browser/geolocation/DEPS is missing from t
| |
| 39 # implementations that are strongly tied to Google servers, i.e. we | |
| 40 # don't expect alternate implementations to be provided by the | |
| 41 # embedder. | |
| 42 "-google_apis", | |
| 43 | |
| 34 # Don't allow inclusion of these other libs we shouldn't be calling directly. | 44 # Don't allow inclusion of these other libs we shouldn't be calling directly. |
| 35 "-v8", | 45 "-v8", |
| 36 "-tools", | 46 "-tools", |
| 37 | 47 |
| 38 # Allow inclusion of third-party code: | 48 # Allow inclusion of third-party code: |
| 39 "+third_party/angle", | 49 "+third_party/angle", |
| 40 "+third_party/flac", | 50 "+third_party/flac", |
| 41 "+third_party/gpsd", | 51 "+third_party/gpsd", |
| 42 "+third_party/mozilla", | 52 "+third_party/mozilla", |
| 43 "+third_party/npapi/bindings", | 53 "+third_party/npapi/bindings", |
| (...skipping 26 matching lines...) Expand all Loading... | |
| 70 # explicit. | 80 # explicit. |
| 71 "-ui/views", | 81 "-ui/views", |
| 72 | 82 |
| 73 "+webkit", | 83 "+webkit", |
| 74 "-webkit/dom_storage", | 84 "-webkit/dom_storage", |
| 75 "+webkit/dom_storage/dom_storage_types.h", | 85 "+webkit/dom_storage/dom_storage_types.h", |
| 76 | 86 |
| 77 # For generated JNI includes. | 87 # For generated JNI includes. |
| 78 "+jni", | 88 "+jni", |
| 79 ] | 89 ] |
| OLD | NEW |
