Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(597)

Unified Diff: net/url_request/url_request_unittest.cc

Issue 154473002: Support redirectUrl at onHeadersReceived in WebRequest / DWR API (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 redirec
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);
« net/url_request/url_request_http_job.cc ('K') | « net/url_request/url_request_test_util.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698