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

Side by Side Diff: content/browser/loader/DEPS

Issue 2251643003: Remove the BeginSaveFile and BeginDownload methods from ResourceDispatcherHostImpl (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address next round of review comments and keep the download test in resource_dispatcher_host_unitteā€¦ Created 4 years, 3 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 # The core loading code in this directory is being transitioned into a separate 1 # The core loading code in this directory is being transitioned into a separate
2 # Mojo networking service. See http://crbug.com/598073. The first stage is to 2 # Mojo networking service. See http://crbug.com/598073. The first stage is to
3 # decouple the code that will be moved from the rest of content. These rules 3 # decouple the code that will be moved from the rest of content. These rules
4 # help ensure that we don't add more includes, and can progressively reduce the 4 # help ensure that we don't add more includes, and can progressively reduce the
5 # code's dependencies. 5 # code's dependencies.
6 # 6 #
7 # For each file, the first section shows includes that are allowed (i.e. because 7 # For each file, the first section shows includes that are allowed (i.e. because
8 # they are for files that will be moving as well). The second section, separated 8 # they are for files that will be moving as well). The second section, separated
9 # by a comment, lists files that we will have to decouple. 9 # by a comment, lists files that we will have to decouple.
10 specific_include_rules = { 10 specific_include_rules = {
(...skipping 143 matching lines...) Expand 10 before | Expand all | Expand 10 after
154 "+content/public/browser/resource_dispatcher_host.h", 154 "+content/public/browser/resource_dispatcher_host.h",
155 "+content/public/browser/resource_dispatcher_host_delegate.h", 155 "+content/public/browser/resource_dispatcher_host_delegate.h",
156 "+content/public/browser/resource_throttle.h", 156 "+content/public/browser/resource_throttle.h",
157 157
158 # TODO: These all have to be removed. 158 # TODO: These all have to be removed.
159 "+content/browser/appcache/appcache_interceptor.h", 159 "+content/browser/appcache/appcache_interceptor.h",
160 "+content/browser/appcache/chrome_appcache_service.h", 160 "+content/browser/appcache/chrome_appcache_service.h",
161 "+content/browser/bad_message.h", 161 "+content/browser/bad_message.h",
162 "+content/browser/blob_storage/chrome_blob_storage_context.h", 162 "+content/browser/blob_storage/chrome_blob_storage_context.h",
163 "+content/browser/child_process_security_policy_impl.h", 163 "+content/browser/child_process_security_policy_impl.h",
164 "+content/browser/download/download_resource_handler.h",
165 "+content/browser/download/save_file_resource_handler.h",
166 "+content/browser/download/save_types.h", 164 "+content/browser/download/save_types.h",
167 "+content/browser/frame_host/frame_tree.h", 165 "+content/browser/frame_host/frame_tree.h",
168 "+content/browser/frame_host/navigation_request_info.h", 166 "+content/browser/frame_host/navigation_request_info.h",
169 "+content/browser/frame_host/navigator.h", 167 "+content/browser/frame_host/navigator.h",
170 "+content/browser/loader/cross_site_resource_handler.h", 168 "+content/browser/loader/cross_site_resource_handler.h",
171 "+content/browser/loader/detachable_resource_handler.h", 169 "+content/browser/loader/detachable_resource_handler.h",
172 "+content/browser/loader/mime_type_resource_handler.h", 170 "+content/browser/loader/mime_type_resource_handler.h",
173 "+content/browser/loader/navigation_resource_handler.h", 171 "+content/browser/loader/navigation_resource_handler.h",
174 "+content/browser/loader/navigation_resource_throttle.h", 172 "+content/browser/loader/navigation_resource_throttle.h",
175 "+content/browser/loader/navigation_url_loader_impl_core.h", 173 "+content/browser/loader/navigation_url_loader_impl_core.h",
(...skipping 158 matching lines...) Expand 10 before | Expand all | Expand 10 after
334 "+content/common/resource_request_completion_status.h", 332 "+content/common/resource_request_completion_status.h",
335 "+content/common/url_loader.mojom.h", 333 "+content/common/url_loader.mojom.h",
336 "+content/common/url_loader_factory.mojom.h", 334 "+content/common/url_loader_factory.mojom.h",
337 "+content/public/browser/resource_context.h", 335 "+content/public/browser/resource_context.h",
338 "+content/public/browser/resource_dispatcher_host_delegate.h", 336 "+content/public/browser/resource_dispatcher_host_delegate.h",
339 "+content/public/common/content_paths.h", 337 "+content/public/common/content_paths.h",
340 "+content/public/test/test_browser_context.h", 338 "+content/public/test/test_browser_context.h",
341 "+content/public/test/test_browser_thread_bundle.h", 339 "+content/public/test/test_browser_thread_bundle.h",
342 ], 340 ],
343 } 341 }
OLDNEW
« no previous file with comments | « content/browser/download/save_file_manager.cc ('k') | content/browser/loader/resource_dispatcher_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698