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

Unified Diff: net/tools/quic/quic_spdy_client_stream.cc

Issue 2631613002: Revert of Add quic_logging (Closed)
Patch Set: Created 3 years, 11 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/tools/quic/quic_simple_server_stream.cc ('k') | net/tools/quic/quic_spdy_client_stream_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/quic/quic_spdy_client_stream.cc
diff --git a/net/tools/quic/quic_spdy_client_stream.cc b/net/tools/quic/quic_spdy_client_stream.cc
index 554c173d0aca0943b9b6c7ed1645293a2060895b..f54b6a468c4dd5c666eac47b35df6ed5deea8924 100644
--- a/net/tools/quic/quic_spdy_client_stream.cc
+++ b/net/tools/quic/quic_spdy_client_stream.cc
@@ -6,11 +6,11 @@
#include <utility>
+#include "base/logging.h"
#include "base/stl_util.h"
#include "net/quic/core/quic_alarm.h"
#include "net/quic/core/quic_client_promised_info.h"
#include "net/quic/core/spdy_utils.h"
-#include "net/quic/platform/api/quic_logging.h"
#include "net/spdy/spdy_protocol.h"
#include "net/tools/quic/quic_client_session.h"
@@ -33,8 +33,8 @@
void QuicSpdyClientStream::OnStreamFrame(const QuicStreamFrame& frame) {
if (!allow_bidirectional_data() && !write_side_closed()) {
- QUIC_DLOG(INFO) << "Got a response before the request was complete. "
- << "Aborting request.";
+ DVLOG(1) << "Got a response before the request was complete. "
+ << "Aborting request.";
CloseWriteSide();
}
QuicSpdyStream::OnStreamFrame(frame);
@@ -50,15 +50,14 @@
header_bytes_read_ += frame_len;
if (!SpdyUtils::CopyAndValidateHeaders(header_list, &content_length_,
&response_headers_)) {
- QUIC_DLOG(ERROR) << "Failed to parse header list: "
- << header_list.DebugString();
+ DLOG(ERROR) << "Failed to parse header list: " << header_list.DebugString();
Reset(QUIC_BAD_APPLICATION_PAYLOAD);
return;
}
if (!ParseHeaderStatusCode(response_headers_, &response_code_)) {
- QUIC_DLOG(ERROR) << "Received invalid response code: "
- << response_headers_[":status"].as_string();
+ DLOG(ERROR) << "Received invalid response code: "
+ << response_headers_[":status"].as_string();
Reset(QUIC_BAD_APPLICATION_PAYLOAD);
return;
}
@@ -73,7 +72,7 @@
}
ConsumeHeaderList();
- QUIC_DVLOG(1) << "headers complete for stream " << id();
+ DVLOG(1) << "headers complete for stream " << id();
session_->OnInitialHeadersComplete(id(), response_headers_);
}
@@ -95,8 +94,8 @@
SpdyHeaderBlock promise_headers;
if (!SpdyUtils::CopyAndValidateHeaders(header_list, &content_length,
&promise_headers)) {
- QUIC_DLOG(ERROR) << "Failed to parse promise headers: "
- << header_list.DebugString();
+ DLOG(ERROR) << "Failed to parse promise headers: "
+ << header_list.DebugString();
Reset(QUIC_BAD_APPLICATION_PAYLOAD);
return;
}
@@ -119,14 +118,14 @@
// No more data to read.
break;
}
- QUIC_DVLOG(1) << "Client processed " << iov.iov_len << " bytes for stream "
- << id();
+ DVLOG(1) << "Client processed " << iov.iov_len << " bytes for stream "
+ << id();
data_.append(static_cast<char*>(iov.iov_base), iov.iov_len);
if (content_length_ >= 0 &&
data_.size() > static_cast<uint64_t>(content_length_)) {
- QUIC_DLOG(ERROR) << "Invalid content length (" << content_length_
- << ") with data of size " << data_.size();
+ DLOG(ERROR) << "Invalid content length (" << content_length_
+ << ") with data of size " << data_.size();
Reset(QUIC_BAD_APPLICATION_PAYLOAD);
return;
}
« no previous file with comments | « net/tools/quic/quic_simple_server_stream.cc ('k') | net/tools/quic/quic_spdy_client_stream_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698