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

Unified Diff: net/http/http_stream_factory_impl_job_controller_unittest.cc

Issue 2297263002: Revert "Only allow HTTP/0.9 support on default ports." (Closed)
Patch Set: Created 4 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
« no previous file with comments | « net/http/http_stream_factory_impl_job.cc ('k') | net/http/http_stream_parser.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_stream_factory_impl_job_controller_unittest.cc
diff --git a/net/http/http_stream_factory_impl_job_controller_unittest.cc b/net/http/http_stream_factory_impl_job_controller_unittest.cc
index ccbcc268e4a6ff5c994e82ad8500ba092795f98b..dad4a1add61b761e421953a9127869bd9ca9465f 100644
--- a/net/http/http_stream_factory_impl_job_controller_unittest.cc
+++ b/net/http/http_stream_factory_impl_job_controller_unittest.cc
@@ -218,10 +218,10 @@ TEST_F(HttpStreamFactoryImplJobControllerTest,
BoundNetLog(), HttpStreamRequest::HTTP_STREAM,
DEFAULT_PRIORITY, SSLConfig(), SSLConfig()));
- // There's no other alternative job. Thus when a stream is ready, it should
+ // There's no other alternative job. Thus when stream is ready, it should
// notify Request.
HttpStream* http_stream =
- new HttpBasicStream(base::MakeUnique<ClientSocketHandle>(), false, false);
+ new HttpBasicStream(base::MakeUnique<ClientSocketHandle>(), false);
job_factory_.main_job()->SetStream(http_stream);
EXPECT_CALL(request_delegate_, OnStreamReady(_, _, http_stream))
@@ -339,7 +339,7 @@ TEST_F(HttpStreamFactoryImplJobControllerTest,
// to Request. The alternative job will mark the main job complete and gets
// orphaned.
HttpStream* http_stream =
- new HttpBasicStream(base::MakeUnique<ClientSocketHandle>(), false, false);
+ new HttpBasicStream(base::MakeUnique<ClientSocketHandle>(), false);
job_factory_.main_job()->SetStream(http_stream);
EXPECT_CALL(request_delegate_, OnStreamReady(_, _, http_stream))
@@ -398,7 +398,7 @@ TEST_F(HttpStreamFactoryImplJobControllerTest,
// |alternative_job| succeeds and should report status to Request.
HttpStream* http_stream =
- new HttpBasicStream(base::MakeUnique<ClientSocketHandle>(), false, false);
+ new HttpBasicStream(base::MakeUnique<ClientSocketHandle>(), false);
job_factory_.alternative_job()->SetStream(http_stream);
EXPECT_CALL(request_delegate_, OnStreamReady(_, _, http_stream))
@@ -449,7 +449,7 @@ TEST_F(HttpStreamFactoryImplJobControllerTest, GetLoadStateAfterMainJobFailed) {
// |alternative_job| succeeds and should report status to Request.
HttpStream* http_stream =
- new HttpBasicStream(base::MakeUnique<ClientSocketHandle>(), false, false);
+ new HttpBasicStream(base::MakeUnique<ClientSocketHandle>(), false);
job_factory_.alternative_job()->SetStream(http_stream);
EXPECT_CALL(request_delegate_, OnStreamReady(_, _, http_stream))
« no previous file with comments | « net/http/http_stream_factory_impl_job.cc ('k') | net/http/http_stream_parser.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698