Remove APIs to override URLRequestJobs on redirect and response received
Neither of these were in use. They were both added for AppCache, and
possibly used by the Data Reduction Proxy, neither of which hooks
into URLRequests any more.
[email protected]
Bug: 934009
Change-Id: If1ceb324d8c31b367147b9c2d44c19b9ac9b0722
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1790491
Commit-Queue: Matt Menke <[email protected]>
Reviewed-by: Eric Roman <[email protected]>
Cr-Commit-Position: refs/heads/master@{#695458}
diff --git a/net/url_request/url_request_interceptor.h b/net/url_request/url_request_interceptor.h
index f7d5275..9ed1670 100644
--- a/net/url_request/url_request_interceptor.h
+++ b/net/url_request/url_request_interceptor.h
@@ -8,17 +8,18 @@
#include "base/macros.h"
#include "net/base/net_export.h"
-class GURL;
-
namespace net {
class URLRequest;
class URLRequestJob;
class NetworkDelegate;
-// A URLRequestInterceptor is given a chance to create a URLRequestJob to
-// handle URLRequests before they're handed off to the ProtocolHandler for
+// In tests, URLRequestFilter lets URLRequestInterceptors create URLRequestJobs
+// to handle URLRequests before they're handed off to the ProtocolHandler for
// the request's scheme.
+//
+// TODO(mmenke): Only include this file in test targets. Also consider using
+// callbacks instead, or even removing URLRequestFilter.
class NET_EXPORT URLRequestInterceptor {
public:
URLRequestInterceptor();
@@ -30,20 +31,6 @@
virtual URLRequestJob* MaybeInterceptRequest(
URLRequest* request, NetworkDelegate* network_delegate) const = 0;
- // Returns a URLRequestJob to handle |request|, if the interceptor wants to
- // take over the handling of the request after a redirect is received,
- // instead of using the default ProtocolHandler. Otherwise, returns NULL.
- virtual URLRequestJob* MaybeInterceptRedirect(
- URLRequest* request,
- NetworkDelegate* network_delegate,
- const GURL& location) const;
-
- // Returns a URLRequestJob to handle |request, if the interceptor wants to
- // take over the handling of the request after a response has started,
- // instead of using the default ProtocolHandler. Otherwise, returns NULL.
- virtual URLRequestJob* MaybeInterceptResponse(
- URLRequest* request, NetworkDelegate* network_delegate) const;
-
private:
DISALLOW_COPY_AND_ASSIGN(URLRequestInterceptor);
};
