Support redirectUrl at onHeadersReceived in WebRequest / DWR API

net/url_request/url_request_unittest.cc

Index: net/url_request/url_request_unittest.cc
diff --git a/net/url_request/url_request_unittest.cc b/net/url_request/url_request_unittest.cc
index 63e7c757244ddf6c31f28adcd369cd86c2a0ef48..b3d745d10726d4000e121b1b7b05b10ec117090a 100644
--- a/net/url_request/url_request_unittest.cc
+++ b/net/url_request/url_request_unittest.cc
@@ -366,6 +366,7 @@ class BlockingNetworkDelegate : public TestNetworkDelegate {
   virtual int OnHeadersReceived(
       URLRequest* request,
       const CompletionCallback& callback,
+      GURL* new_url,
       const HttpResponseHeaders* original_response_headers,
       scoped_refptr<HttpResponseHeaders>* override_response_headers) OVERRIDE;
 
@@ -481,10 +482,11 @@ int BlockingNetworkDelegate::OnBeforeSendHeaders(
 int BlockingNetworkDelegate::OnHeadersReceived(

[battre, 2014/02/12 11:54:48]

I think this should have similar logic for redirects as
BlockingNetworkDelegate::OnBeforeURLRequest.

Then there should be a test similar to TEST_F(URLRequestTestHTTP,
NetworkDelegateRedirectRequest). Also you probably need to update
TestNetworkDelegate::OnBeforeRedirect.

     URLRequest* request,
     const CompletionCallback& callback,
+    GURL* new_url,
     const HttpResponseHeaders* original_response_headers,
     scoped_refptr<HttpResponseHeaders>* override_response_headers) {
   TestNetworkDelegate::OnHeadersReceived(
-      request, callback, original_response_headers,
+      request, callback, new_url, original_response_headers,
       override_response_headers);
 
   return MaybeBlockStage(ON_HEADERS_RECEIVED, callback);
@@ -2415,6 +2417,7 @@ class FixedDateNetworkDelegate : public TestNetworkDelegate {
   virtual int OnHeadersReceived(
       net::URLRequest* request,
       const net::CompletionCallback& callback,
+      GURL* new_url,
       const net::HttpResponseHeaders* original_response_headers,
       scoped_refptr<net::HttpResponseHeaders>* override_response_headers)
       OVERRIDE;
@@ -2428,6 +2431,7 @@ class FixedDateNetworkDelegate : public TestNetworkDelegate {
 int FixedDateNetworkDelegate::OnHeadersReceived(
     net::URLRequest* request,
     const net::CompletionCallback& callback,
+    GURL* new_url,
     const net::HttpResponseHeaders* original_response_headers,
     scoped_refptr<net::HttpResponseHeaders>* override_response_headers) {
   net::HttpResponseHeaders* new_response_headers =
@@ -2439,6 +2443,7 @@ int FixedDateNetworkDelegate::OnHeadersReceived(
   *override_response_headers = new_response_headers;
   return TestNetworkDelegate::OnHeadersReceived(request,
                                                 callback,
+                                                new_url,
                                                 original_response_headers,
                                                 override_response_headers);
 }
@@ -3918,9 +3923,10 @@ class AsyncLoggingNetworkDelegate : public TestNetworkDelegate {
   virtual int OnHeadersReceived(
       URLRequest* request,
       const CompletionCallback& callback,
+      GURL* new_url,
       const HttpResponseHeaders* original_response_headers,
       scoped_refptr<HttpResponseHeaders>* override_response_headers) OVERRIDE {
-    TestNetworkDelegate::OnHeadersReceived(request, callback,
+    TestNetworkDelegate::OnHeadersReceived(request, callback, new_url,
                                            original_response_headers,
                                            override_response_headers);
     return RunCallbackAsynchronously(request, callback);