| Index: android_webview/browser/net/aw_url_request_job_factory.cc
|
| diff --git a/android_webview/browser/net/aw_url_request_job_factory.cc b/android_webview/browser/net/aw_url_request_job_factory.cc
|
| index 0d8ddecff1c11e981f5752df85d339c92007f3db..ff3b9b8b0031b1972a2c95b01f23d625f488b3a2 100644
|
| --- a/android_webview/browser/net/aw_url_request_job_factory.cc
|
| +++ b/android_webview/browser/net/aw_url_request_job_factory.cc
|
| @@ -18,20 +18,14 @@ using net::URLRequestJob;
|
| namespace android_webview {
|
|
|
| AwURLRequestJobFactory::AwURLRequestJobFactory()
|
| - : next_factory_(new net::URLRequestJobFactoryImpl()) {
|
| -}
|
| + : next_factory_(
|
| + net::URLRequestJobFactoryImpl::CreateWithHttpProtocolHandlers()) {}
|
|
|
| AwURLRequestJobFactory::~AwURLRequestJobFactory() {
|
| }
|
|
|
| bool AwURLRequestJobFactory::IsHandledProtocol(
|
| const std::string& scheme) const {
|
| - // This introduces a dependency on the URLRequestJobManager
|
| - // implementation. The assumption is that if true is returned from this
|
| - // method it is still valid to return NULL from the
|
| - // MaybeCreateJobWithProtocolHandler method and in that case the
|
| - // URLRequestJobManager will try and create the URLRequestJob by using the
|
| - // set of built in handlers.
|
| return true;
|
| }
|
|
|
| @@ -49,15 +43,6 @@ URLRequestJob* AwURLRequestJobFactory::MaybeCreateJobWithProtocolHandler(
|
| if (job)
|
| return job;
|
|
|
| - // If a URLRequestJobManager built-in factory supports the scheme, NULL should
|
| - // be returned from this method.
|
| - // In that case the built in handlers will then be used to create the job.
|
| - // TODO(mgersh): remove this check once HttpProtocolHandler exists and is
|
| - // added to the relevant URLRequestJobFactory.
|
| - // NOTE(joth): See the assumption in IsHandledProtocol above.
|
| - if (next_factory_->IsHandledProtocol(scheme))
|
| - return NULL;
|
| -
|
| return new net::URLRequestErrorJob(
|
| request, network_delegate, net::ERR_UNKNOWN_URL_SCHEME);
|
| }
|
|
|