Rename ProtocolInterceptJobFactory and make it not use ProtocolHandlers.
ProtocolHandlers are intended to handle all requests for a particular
scheme, while UrlRequestInterceptors are intended to sit in front of
ProtocolHandlers, and optionally take over handling of any request,
regardless of scheme.
Separating the classes removes some ambiguity and weirdness,
particularly when passing a bunch of interceptors all over the place
from content/ to chrome/ during initialization.
BUG=373800
Review URL: https://codereview.chromium.org/282103004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273874 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/content/shell/browser/shell_browser_context.cc b/content/shell/browser/shell_browser_context.cc
index 91806ba..39d70a5 100644
--- a/content/shell/browser/shell_browser_context.cc
+++ b/content/shell/browser/shell_browser_context.cc
@@ -141,7 +141,7 @@
net::URLRequestContextGetter* ShellBrowserContext::CreateRequestContext(
ProtocolHandlerMap* protocol_handlers,
- ProtocolHandlerScopedVector protocol_interceptors) {
+ URLRequestInterceptorScopedVector request_interceptors) {
DCHECK(!url_request_getter_.get());
url_request_getter_ = new ShellURLRequestContextGetter(
ignore_certificate_errors_,
@@ -149,7 +149,7 @@
BrowserThread::UnsafeGetMessageLoopForThread(BrowserThread::IO),
BrowserThread::UnsafeGetMessageLoopForThread(BrowserThread::FILE),
protocol_handlers,
- protocol_interceptors.Pass(),
+ request_interceptors.Pass(),
net_log_);
resource_context_->set_url_request_context_getter(url_request_getter_.get());
return url_request_getter_.get();
@@ -222,7 +222,7 @@
const base::FilePath& partition_path,
bool in_memory,
ProtocolHandlerMap* protocol_handlers,
- ProtocolHandlerScopedVector protocol_interceptors) {
+ URLRequestInterceptorScopedVector request_interceptors) {
return NULL;
}
