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

Unified Diff: net/quic/quic_http_stream.cc

Issue 185203003: Killing off QUICv12, including cleaning out all of the code for handling (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Deleted unused StripUint32 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
« no previous file with comments | « net/quic/quic_headers_stream.cc ('k') | net/quic/quic_http_stream_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_http_stream.cc
diff --git a/net/quic/quic_http_stream.cc b/net/quic/quic_http_stream.cc
index 992f25467df9022764939006f13d5596e48d1c8b..661562f764a8863dfa6d63e23579251b0e497ac6 100644
--- a/net/quic/quic_http_stream.cc
+++ b/net/quic/quic_http_stream.cc
@@ -412,17 +412,6 @@ int QuicHttpStream::DoSendHeaders() {
if (!stream_)
return ERR_UNEXPECTED;
- if (stream_->version() <= QUIC_VERSION_12) {
- if (request_.empty() && !stream_->CanWrite(
- base::Bind(&QuicHttpStream::OnIOComplete,
- weak_factory_.GetWeakPtr()))) {
- // Do not compress headers unless it is likely that they can be sent.
- next_state_ = STATE_SEND_HEADERS;
- return ERR_IO_PENDING;
- }
- request_ = stream_->compressor()->CompressHeadersWithPriority(
- ConvertRequestPriorityToQuicPriority(priority_), request_headers_);
- }
// Log the actual request with the URL Request's net log.
stream_net_log_.AddEvent(
NetLog::TYPE_HTTP_TRANSACTION_QUIC_SEND_REQUEST_HEADERS,
@@ -435,11 +424,7 @@ int QuicHttpStream::DoSendHeaders() {
bool has_upload_data = request_body_stream_ != NULL;
next_state_ = STATE_SEND_HEADERS_COMPLETE;
- int rv = (stream_->version() > QUIC_VERSION_12) ?
- stream_->WriteHeaders(request_headers_, !has_upload_data) :
- stream_->WriteStreamData(request_, !has_upload_data,
- base::Bind(&QuicHttpStream::OnIOComplete,
- weak_factory_.GetWeakPtr()));
+ int rv = stream_->WriteHeaders(request_headers_, !has_upload_data);
request_headers_.clear();
return rv;
}
« no previous file with comments | « net/quic/quic_headers_stream.cc ('k') | net/quic/quic_http_stream_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698