Add MaybeInterceptRedirect/Response to URLRequestInterceptor

net/url_request/url_request_job_manager.cc

Index: net/url_request/url_request_job_manager.cc
diff --git a/net/url_request/url_request_job_manager.cc b/net/url_request/url_request_job_manager.cc
index 0fe557b14418b54ee437845aa9b481973f9e3adb..3376e199a51c7d8e2290ae477e5c1175eb224a56 100644
--- a/net/url_request/url_request_job_manager.cc
+++ b/net/url_request/url_request_job_manager.cc
@@ -128,7 +128,8 @@ URLRequestJob* URLRequestJobManager::MaybeInterceptRedirect(
     if (job)
       return job;
   }
-  return NULL;
+  return request->context()->job_factory()->MaybeInterceptRedirect(
+      request, network_delegate, location);

[mmenke, 2014/10/30 19:59:24]

Ideally, these should ignore the LOAD_DISABLE_INTERCEPT flag.  AppCache should
just track the URLRequests itself, probably by attaching UserData to them (And
it actually already does this, I believe).

Suppose that should probably be in its own CL, though, after AppCache has been
moved to the new API, so not really something you should worry about (Just don't
use the load flag in your new code).

[mmenke, 2014/10/30 19:59:24]

These should go before the old interceptors, for consistency.

[bengr, 2014/10/31 00:45:31]

Acknowledged.

[bengr, 2014/10/31 00:45:31]

Done.

[michaeln1, 2014/10/31 00:47:10]

Consistency with what?

The old URLRequest::Interceptor's MaybeIntercept method gets called prior to the
corresponding method of the new URLReqeustInterceptor.

 }
 
 URLRequestJob* URLRequestJobManager::MaybeInterceptResponse(
@@ -154,7 +155,8 @@ URLRequestJob* URLRequestJobManager::MaybeInterceptResponse(
     if (job)
       return job;
   }
-  return NULL;
+  return request->context()->job_factory()->MaybeInterceptResponse(
+      request, network_delegate);
 }
 
 // static