Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 mojo interface |
|
jam
2016/05/03 03:32:46
actually you can just remove the second sentence a
xhwang
2016/05/03 05:04:11
Done.
| |
| 3 # src/components that are shared with the mojo html_viewer. | 3 # files in src/components that are shared with other modules. |
| 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", |
| (...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 103 # Content shouldn't depend on views. While we technically don't need this | 103 # Content shouldn't depend on views. While we technically don't need this |
| 104 # 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 |
| 105 # explicit. | 105 # explicit. |
| 106 "-ui/views", | 106 "-ui/views", |
| 107 | 107 |
| 108 "+storage/browser", | 108 "+storage/browser", |
| 109 "+storage/common", | 109 "+storage/common", |
| 110 | 110 |
| 111 # For generated JNI includes. | 111 # For generated JNI includes. |
| 112 "+jni", | 112 "+jni", |
| 113 | |
| 114 # For permission mojo interfaces. | |
| 115 "+components/permissions", | |
|
jam
2016/05/03 03:32:46
put this up near the other components/ lines
xhwang
2016/05/03 05:04:11
Moved up. But I don't see other components/ lines.
| |
| 113 ] | 116 ] |
| 114 | 117 |
| 115 # content -> content/shell dependency is not allowed, except for browser tests. | 118 # content -> content/shell dependency is not allowed, except for browser tests. |
| 116 specific_include_rules = { | 119 specific_include_rules = { |
| 117 ".*_browsertest[a-z_]*\.(cc|h)": [ | 120 ".*_browsertest[a-z_]*\.(cc|h)": [ |
| 118 "+content/shell/browser", | 121 "+content/shell/browser", |
| 119 "+content/shell/common", | 122 "+content/shell/common", |
| 120 ], | 123 ], |
| 121 } | 124 } |
| 122 | 125 |
| OLD | NEW |