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

Unified Diff: net/http/http_stream_factory_impl_request_unittest.cc

Issue 1326503003: Added a net::BidirectionalStream to expose a bidirectional streaming interface (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address Misha's comments Created 5 years 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/http/http_stream_factory_impl_request_unittest.cc
diff --git a/net/http/http_stream_factory_impl_request_unittest.cc b/net/http/http_stream_factory_impl_request_unittest.cc
index 8324d6141dba002778d31bca1402a273f4dca219..0704ebf1e48de00f286b9b5a23af9771918ef0bd 100644
--- a/net/http/http_stream_factory_impl_request_unittest.cc
+++ b/net/http/http_stream_factory_impl_request_unittest.cc
@@ -35,6 +35,11 @@ class DoNothingRequestDelegate : public HttpStreamRequest::Delegate {
void OnStreamReady(const SSLConfig& used_ssl_config,
const ProxyInfo& used_proxy_info,
HttpStream* stream) override {}
+ void OnBidirectionalStreamJobReady(
+ const SSLConfig& used_ssl_config,
+ const ProxyInfo& used_proxy_info,
+ BidirectionalStreamJob* stream_job) override {}
+
void OnWebSocketHandshakeStreamReady(
const SSLConfig& used_ssl_config,
const ProxyInfo& used_proxy_info,
@@ -72,7 +77,8 @@ TEST_P(HttpStreamFactoryImplRequestTest, SetPriority) {
DoNothingRequestDelegate request_delegate;
HttpStreamFactoryImpl::Request request(
- GURL(), factory, &request_delegate, NULL, BoundNetLog());
+ GURL(), factory, &request_delegate, NULL, BoundNetLog(),
+ HttpStreamFactoryImpl::Request::HTTP_STREAM);
HttpStreamFactoryImpl::Job* job =
new HttpStreamFactoryImpl::Job(factory,

Powered by Google App Engine
This is Rietveld 408576698