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

Side by Side Diff: chrome/browser/favicon/DEPS

Issue 16072002: chrome/browser: Remove unnecessary entries from DEPS files. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 7 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
OLDNEW
1 include_rules = [ 1 include_rules = [
2 # Favicon is being made into a Browser Component, so we have these 2 # Favicon is being made into a Browser Component, so we have these
3 # three basic rules followed by temporary exceptions. Please don't 3 # three basic rules followed by temporary exceptions. Please don't
4 # add to the list of exceptions! 4 # add to the list of exceptions!
5 "-chrome/browser", 5 "-chrome/browser",
6 "+chrome/browser/api",
7 "+chrome/browser/common", 6 "+chrome/browser/common",
8 "+chrome/browser/favicon", 7 "+chrome/browser/favicon",
9 8
10 # Permanently-allowed DEPS beyond the standard Browser 9 # Permanently-allowed DEPS beyond the standard Browser
11 # Components-like DEPS above go here. 10 # Components-like DEPS above go here.
12 "+chrome/browser/search/search.h", 11 "+chrome/browser/search/search.h",
13 "+third_party/skia/include/core/SkBitmap.h", 12 "+third_party/skia/include/core/SkBitmap.h",
14 13
15 # TODO(caitkp): Bring this list to zero. 14 # TODO(caitkp): Bring this list to zero.
16 # 15 #
17 # Do not add to the list of temporarily-allowed dependencies below, 16 # Do not add to the list of temporarily-allowed dependencies below,
18 # and please do not introduce more #includes of these files. 17 # and please do not introduce more #includes of these files.
19 "!chrome/browser/bookmarks/bookmark_service.h", 18 "!chrome/browser/bookmarks/bookmark_service.h",
20 "!chrome/browser/history/history_backend.h", 19 "!chrome/browser/history/history_backend.h",
21 "!chrome/browser/history/history_service.h", 20 "!chrome/browser/history/history_service.h",
22 "!chrome/browser/history/history_service_factory.h", 21 "!chrome/browser/history/history_service_factory.h",
23 "!chrome/browser/history/select_favicon_frames.h", 22 "!chrome/browser/history/select_favicon_frames.h",
24 "!chrome/browser/profiles/profile.h", 23 "!chrome/browser/profiles/profile.h",
25 "!chrome/browser/profiles/profile_dependency_manager.h",
26 "!chrome/browser/profiles/profile_keyed_service.h",
27 "!chrome/browser/profiles/profile_keyed_service_factory.h",
28 "!chrome/browser/ui/webui/chrome_web_ui_controller_factory.h", 24 "!chrome/browser/ui/webui/chrome_web_ui_controller_factory.h",
29 # Do not add to the list of temporarily-allowed dependencies above, 25 # Do not add to the list of temporarily-allowed dependencies above,
30 # and please do not introduce more #includes of these files. 26 # and please do not introduce more #includes of these files.
31 ] 27 ]
32 28
33 specific_include_rules = { 29 specific_include_rules = {
34 # TODO(caitkp): Look into removing history/ dependencies. 30 # TODO(caitkp): Look into removing history/ dependencies.
35 'favicon_util\.(h|cc)': [ 31 'favicon_util\.(h|cc)': [
36 "-chrome", 32 "-chrome",
37 "-chrome/browser", 33 "-chrome/browser",
38 "+chrome/browser/favicon/favicon_types.h", 34 "+chrome/browser/favicon/favicon_types.h",
39 "+chrome/browser/favicon/favicon_util.h", 35 "+chrome/browser/favicon/favicon_util.h",
40 "+chrome/common/icon_messages.h", 36 "+chrome/common/icon_messages.h",
41 "!chrome/browser/history/history_types.h", 37 "!chrome/browser/history/history_types.h",
42 "!chrome/browser/history/select_favicon_frames.h", 38 "!chrome/browser/history/select_favicon_frames.h",
43 ] 39 ]
44 } 40 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698