Remove no-op |direct| argument and |direct_| member.
Remove |direct| argument from CreateSpdyHeadersFromHttpRequest(), where
it does not do anything.
Also remove |direct| argument from the following methods where it is now
unused:
HttpStreamFactoryImpl::JobController::OnNewSpdySessionReady()
HttpStreamFactoryImpl::Job::SetSpdyHttpStreamOrBidirectionalStreamImpl()
SpdySessionPool::OnNewSpdySessionReady()
SpdyHttpStream::SpdyHttpStream()
Also remove SpdyHttpStream::direct_ member.
Turns out SpdyHttpUtilsTest.CreateSpdyHeadersFromHttpRequestHTTP2 and
SpdyHttpUtilsTest.CreateSpdyHeadersFromHttpRequestProxyHTTP2 are now
identical, remove the latter.
Change-Id: I7ede44b463c8f231f11ea3fd78db9606ca38780f
Reviewed-on: https://chromium-review.googlesource.com/907130
Reviewed-by: Ryan Hamilton <[email protected]>
Commit-Queue: Bence Béky <[email protected]>
Cr-Commit-Position: refs/heads/master@{#535218}diff --git a/net/quic/chromium/quic_proxy_client_socket.cc b/net/quic/chromium/quic_proxy_client_socket.cc
index 82736b7..da18061f 100644
--- a/net/quic/chromium/quic_proxy_client_socket.cc
+++ b/net/quic/chromium/quic_proxy_client_socket.cc
@@ -363,8 +363,7 @@
base::Unretained(&request_.extra_headers), &request_line));
SpdyHeaderBlock headers;
- CreateSpdyHeadersFromHttpRequest(request_, request_.extra_headers, true,
- &headers);
+ CreateSpdyHeadersFromHttpRequest(request_, request_.extra_headers, &headers);
return stream_->WriteHeaders(std::move(headers), false, nullptr);
}
