Chromium Code Reviews| Index: net/url_request/url_request_job_factory.h |
| diff --git a/net/url_request/url_request_job_factory.h b/net/url_request/url_request_job_factory.h |
| index bbd1d09f09ccf1bfbfafb4a633645a4ec45c3232..ec848492bde0add5bfe551f218516ea9019e4f0d 100644 |
| --- a/net/url_request/url_request_job_factory.h |
| +++ b/net/url_request/url_request_job_factory.h |
| @@ -30,6 +30,13 @@ class NET_EXPORT URLRequestJobFactory |
| virtual URLRequestJob* MaybeCreateJob( |
| URLRequest* request, NetworkDelegate* network_delegate) const = 0; |
| + |
| + // Indicates if it should be safe to redirect to |location|. Should handle |
| + // protocols handled by MaybeCreateJob(). Only called when registered with |
| + // URLRequestJobFactoryImpl::SetProtocolHandler(); not called when used with |
|
mmenke
2013/02/20 15:48:58
nit: Don't think we need the semi-colon.
|
| + // ProtocolInterceptJobFactory. |
| + // NOTE(pauljensen): Default implementation returns true. |
| + virtual bool IsSafeRedirectTarget(const GURL& location) const; |
| }; |
| URLRequestJobFactory(); |
| @@ -44,6 +51,8 @@ class NET_EXPORT URLRequestJobFactory |
| virtual bool IsHandledURL(const GURL& url) const = 0; |
| + virtual bool IsSafeRedirectTarget(const GURL& location) const = 0; |
| + |
| private: |
| DISALLOW_COPY_AND_ASSIGN(URLRequestJobFactory); |
| }; |