diff options
author | rob@robwu.nl <rob@robwu.nl@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-26 10:41:16 +0000 |
---|---|---|
committer | rob@robwu.nl <rob@robwu.nl@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-03-26 10:41:16 +0000 |
commit | 5f71413b4cefc6db3f111b2da1681266e3c056ec (patch) | |
tree | f137d56cdc889d2d411c55bbe59a402abbff95b0 /net/base/network_delegate.h | |
parent | 1615f14112858597ccff27f6acab09be19dc40d7 (diff) | |
download | chromium_src-5f71413b4cefc6db3f111b2da1681266e3c056ec.zip chromium_src-5f71413b4cefc6db3f111b2da1681266e3c056ec.tar.gz chromium_src-5f71413b4cefc6db3f111b2da1681266e3c056ec.tar.bz2 |
Support redirectUrl at onHeadersReceived in WebRequest / DWR API
Add support for extension-initiated redirects via the (declarative) Web Request API.
The existing URL validation and conflict resolution logic of redirectUrl at onBeforeRequest is re-used for the implementation of redirectUrl at onHeadersReceived.
To make sure that redirects to data:// and chrome-extension:// URLs are not blocked, a new parameter has been added to the network delegate (allowed_unsafe_redirect_url).
BUG=280464,115940
TEST=browser_tests: ExtensionWebRequestApiTest.WebRequestBlocking:ExtensionWebRequestApiTest.WebRequestDeclarative1
unit_tests: ExtensionWebRequestTest.*:WebRequestActionWithThreadsTest.*
net_unittests: URLRequestTestHTTP.NetworkDelegateRedirectRequestOnHeadersReceived*: URLRequestTestHTTP.UnsafeRedirect*: URLRequestTestHTTP.*ReferenceFragment*
Review URL: https://codereview.chromium.org/154473002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259546 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/base/network_delegate.h')
-rw-r--r-- | net/base/network_delegate.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/net/base/network_delegate.h b/net/base/network_delegate.h index 882701c..4f930cf 100644 --- a/net/base/network_delegate.h +++ b/net/base/network_delegate.h @@ -69,7 +69,8 @@ class NET_EXPORT NetworkDelegate : public base::NonThreadSafe { URLRequest* request, const CompletionCallback& callback, const HttpResponseHeaders* original_response_headers, - scoped_refptr<HttpResponseHeaders>* override_response_headers); + scoped_refptr<HttpResponseHeaders>* override_response_headers, + GURL* allowed_unsafe_redirect_url); void NotifyBeforeRedirect(URLRequest* request, const GURL& new_location); void NotifyResponseStarted(URLRequest* request); @@ -140,7 +141,8 @@ class NET_EXPORT NetworkDelegate : public base::NonThreadSafe { URLRequest* request, const CompletionCallback& callback, const HttpResponseHeaders* original_response_headers, - scoped_refptr<HttpResponseHeaders>* override_response_headers); + scoped_refptr<HttpResponseHeaders>* override_response_headers, + GURL* allowed_unsafe_redirect_url); // Called right after a redirect response code was received. // |new_location| is only valid until OnURLRequestDestroyed is called for this |