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

Unified Diff: net/url_request/url_request_unittest.cc

Issue 10855209: Refactoring: ProtocolHandler::MaybeCreateJob takes NetworkDelegate as argument (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 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 8ff168c1b0118a959b93ea9db1bff63d74f8d36d..b4a955acb9acf9a90d14a032a8a00b516b182a4e 100644
--- a/net/url_request/url_request_unittest.cc
+++ b/net/url_request/url_request_unittest.cc
@@ -3638,9 +3638,10 @@ class TestInterceptor : URLRequest::Interceptor {
intercept_main_request_ = false;
did_intercept_main_ = true;
return new URLRequestTestJob(request,
- main_headers_,
- main_data_,
- true);
+ request->context()->network_delegate(),
+ main_headers_,
+ main_data_,
+ true);
}
virtual URLRequestJob* MaybeInterceptRedirect(URLRequest* request,
@@ -3655,9 +3656,10 @@ class TestInterceptor : URLRequest::Interceptor {
intercept_redirect_ = false;
did_intercept_redirect_ = true;
return new URLRequestTestJob(request,
- redirect_headers_,
- redirect_data_,
- true);
+ request->context()->network_delegate(),
+ redirect_headers_,
+ redirect_data_,
+ true);
}
virtual URLRequestJob* MaybeInterceptResponse(URLRequest* request) {
@@ -3671,9 +3673,10 @@ class TestInterceptor : URLRequest::Interceptor {
intercept_final_response_ = false;
did_intercept_final_ = true;
return new URLRequestTestJob(request,
- final_headers_,
- final_data_,
- true);
+ request->context()->network_delegate(),
+ final_headers_,
+ final_data_,
+ true);
}
// Whether to intercept the main request, and if so the response to return.
@@ -4117,13 +4120,11 @@ TEST_F(URLRequestTestFTP, UnsafePort) {
GURL url("ftp://127.0.0.1:7");
FtpProtocolHandler ftp_protocol_handler(
- default_context_.network_delegate(),
default_context_.ftp_transaction_factory(),
default_context_.ftp_auth_cache());
job_factory.SetProtocolHandler(
"ftp",
- new FtpProtocolHandler(default_context_.network_delegate(),
- default_context_.ftp_transaction_factory(),
+ new FtpProtocolHandler(default_context_.ftp_transaction_factory(),
default_context_.ftp_auth_cache()));
default_context_.set_job_factory(&job_factory);

Powered by Google App Engine
This is Rietveld 408576698