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

Unified Diff: content/DEPS

Issue 1777503003: More strict restrictions for content -> content/shell dependencies. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Removed unnecessary namespace-qualification. Created 4 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | content/common/DEPS » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/DEPS
diff --git a/content/DEPS b/content/DEPS
index b7cda8593eaef5d064b454078771a96f7d616786..b0e052beb74bf96d5d2cf67634b29f3638bce8bb 100644
--- a/content/DEPS
+++ b/content/DEPS
@@ -10,7 +10,6 @@ include_rules = [
"+content/grit",
"+content/public/common",
"+content/public/test",
- "+content/shell", # for content_browsertests
"+content/test",
"+blink",
@@ -112,3 +111,12 @@ include_rules = [
# For generated JNI includes.
"+jni",
]
+
+# content -> content/shell dependency is not allowed, except for browser tests.
+specific_include_rules = {
+ ".*_browsertest[a-z_]*\.cc": [
+ "+content/shell/browser",
+ "+content/shell/common",
+ ],
+}
+
« no previous file with comments | « no previous file | content/common/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698