content/browser/DEPS - Issue 7044013: Drop url_constants dependency in content/.

Side by Side Diff: content/browser/DEPS

Issue 7044013: Drop url_constants dependency in content/. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: forgot null check Created 9 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 | Annotate | Revision Log

« no previous file with comments | « chrome/browser/chrome_content_browser_client.cc ('k') | content/browser/browsing_instance.cc » ('j') | content/browser/site_instance.cc » ('J')

OLD	NEW
1 include_rules = [	1 include_rules = [

2 "+content/gpu", # For gpu_info_collector.h. Needs to move to common\gpu.	2 "+content/gpu", # For gpu_info_collector.h. Needs to move to common\gpu.

3	3

4 # Listing of the remaining #chrome includes in content\browser. People are	4 # Listing of the remaining #chrome includes in content\browser. People are

5 # actively working on reducing this to 0. Do not add any more!	5 # actively working on reducing this to 0. Do not add any more!

6	6

7 "+chrome/browser/accessibility/browser_accessibility_state.h",	7 "+chrome/browser/accessibility/browser_accessibility_state.h",

8	8

9 "+chrome/browser/browser_about_handler.h",	9 "+chrome/browser/browser_about_handler.h",

10	10

(...skipping 83 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
94 "+chrome/browser/ui/browser_dialogs.h",	94 "+chrome/browser/ui/browser_dialogs.h",

95 "+chrome/browser/ui/crypto_module_password_dialog.h",	95 "+chrome/browser/ui/crypto_module_password_dialog.h",

96 "+chrome/browser/ui/find_bar/find_bar_controller.h",	96 "+chrome/browser/ui/find_bar/find_bar_controller.h",

97 "+chrome/browser/ui/login/login_prompt.h",	97 "+chrome/browser/ui/login/login_prompt.h",

98	98

99 "+chrome/common/chrome_constants.h",	99 "+chrome/common/chrome_constants.h",

100 "+chrome/common/chrome_switches.h",	100 "+chrome/common/chrome_switches.h",

101 "+chrome/common/chrome_paths.h",	101 "+chrome/common/chrome_paths.h",

102 "+chrome/common/logging_chrome.h",	102 "+chrome/common/logging_chrome.h",

103 "+chrome/common/render_messages.h",	103 "+chrome/common/render_messages.h",

104 "+chrome/common/url_constants.h",

105	104

106 "+chrome/common/net/url_fetcher.h",	105 "+chrome/common/net/url_fetcher.h",

107	106

108 "+chrome/common/sandbox_policy.h",	107 "+chrome/common/sandbox_policy.h",

109	108

110 "+chrome/common/security_style.h",	109 "+chrome/common/security_style.h",

111	110

112 # ONLY USED BY TESTS	111 # ONLY USED BY TESTS

113 "+chrome/browser/chromeos/cros/mock_network_library.h",	112 "+chrome/browser/chromeos/cros/mock_network_library.h",

114 "+chrome/browser/net/url_request_failed_dns_job.h",	113 "+chrome/browser/net/url_request_failed_dns_job.h",

115 "+chrome/browser/net/url_request_mock_http_job.h",	114 "+chrome/browser/net/url_request_mock_http_job.h",

116 "+chrome/browser/profiles/profile_manager.h",	115 "+chrome/browser/profiles/profile_manager.h",

117 "+chrome/browser/sessions/session_service.h",	116 "+chrome/browser/sessions/session_service.h",

118 "+chrome/browser/sessions/session_service_factory.h",	117 "+chrome/browser/sessions/session_service_factory.h",

119 "+chrome/browser/sessions/session_service_test_helper.h",	118 "+chrome/browser/sessions/session_service_test_helper.h",

120 "+chrome/browser/ui/browser.h",	119 "+chrome/browser/ui/browser.h",

121 "+chrome/browser/ui/cocoa/find_pasteboard.h",	120 "+chrome/browser/ui/cocoa/find_pasteboard.h",

122 "+chrome/browser/ui/tab_contents/tab_contents_wrapper.h",	121 "+chrome/browser/ui/tab_contents/tab_contents_wrapper.h",

123 "+chrome/common/net/test_url_fetcher_factory.h",	122 "+chrome/common/net/test_url_fetcher_factory.h",

	123 "+chrome/common/url_constants.h",
	jam 2011/05/18 22:12:29 are there many tests that use this? can we move t are there many tests that use this? can we move them? if we leave it here, people might add dependencies. Avi (use Gerrit) 2011/05/18 22:13:59 Already fixed. Already fixed.
124 ]	124 ]

OLD	NEW