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

Side by Side Diff: extensions/DEPS

Issue 702513002: Refine //extensions DEPS on //ui (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 1 month 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
OLDNEW
1 include_rules = [ 1 include_rules = [
2 # Do not add Chrome dependencies. Much work went into removing them. 2 # Do not add Chrome dependencies. Much work went into removing them.
3 "+components/crx_file", 3 "+components/crx_file",
4 "+components/url_matcher", 4 "+components/url_matcher",
5 "-content", 5 "-content",
6 "+content/grit/content_resources.h", 6 "+content/grit/content_resources.h",
7 "+content/public/common", 7 "+content/public/common",
8 "+content/public/test", 8 "+content/public/test",
9 "+crypto", 9 "+crypto",
10 "-extensions/components", 10 "-extensions/components",
11 "+extensions/test", 11 "+extensions/test",
12 "+grit/extensions_renderer_resources.h", 12 "+grit/extensions_renderer_resources.h",
13 "+grit/extensions_resources.h", 13 "+grit/extensions_resources.h",
14 "+mojo/public", 14 "+mojo/public",
15 "+testing", 15 "+testing",
16 "+ui", 16 "-ui",
James Cook 2014/11/03 21:35:20 Add a comment with the rationale for excluding all
Ken Rockot(use gerrit already) 2014/11/04 16:50:24 Done.
17 "+ui/base",
18 "+ui/gfx",
19 "+ui/events",
17 20
18 # NOTE: Please do not add includes without talking to the app shell team; 21 # NOTE: Please do not add includes without talking to the app shell team;
19 # see OWNERS for this directory. 22 # see OWNERS for this directory.
20 ] 23 ]
21 24
22 specific_include_rules = { 25 specific_include_rules = {
23 ".*(test|test_util)\.(cc|h)$": [ 26 ".*(test|test_util)\.(cc|h)$": [
24 "+content/public/test", 27 "+content/public/test",
25 28
26 # Temporarily allowed testing includes. See above. 29 # Temporarily allowed testing includes. See above.
(...skipping 10 matching lines...) Expand all
37 "+chrome/browser/extensions/test_extension_dir.h", 40 "+chrome/browser/extensions/test_extension_dir.h",
38 "+chrome/browser/extensions/test_extension_prefs.h", 41 "+chrome/browser/extensions/test_extension_prefs.h",
39 "+chrome/browser/extensions/test_extension_system.h", 42 "+chrome/browser/extensions/test_extension_system.h",
40 "+chrome/browser/ui/browser.h", 43 "+chrome/browser/ui/browser.h",
41 "+chrome/common/chrome_paths.h", 44 "+chrome/common/chrome_paths.h",
42 "+chrome/common/extensions/features/feature_channel.h", 45 "+chrome/common/extensions/features/feature_channel.h",
43 "+chrome/test/base/testing_profile.h", 46 "+chrome/test/base/testing_profile.h",
44 "+chrome/test/base/ui_test_utils.h", 47 "+chrome/test/base/ui_test_utils.h",
45 ], 48 ],
46 } 49 }
OLDNEW
« no previous file with comments | « no previous file | extensions/components/native_app_window/DEPS » ('j') | extensions/shell/browser/DEPS » ('J')

Powered by Google App Engine
This is Rietveld 408576698