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

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

Issue 2611613003: Add quic_logging (Closed)
Patch Set: fix failed test? 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 f54b6a468c4dd5c666eac47b35df6ed5deea8924..554c173d0aca0943b9b6c7ed1645293a2060895b 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 @@ QuicSpdyClientStream::~QuicSpdyClientStream() {}
void QuicSpdyClientStream::OnStreamFrame(const QuicStreamFrame& frame) {
if (!allow_bidirectional_data() && !write_side_closed()) {
- DVLOG(1) << "Got a response before the request was complete. "
- << "Aborting request.";
+ QUIC_DLOG(INFO) << "Got a response before the request was complete. "
+ << "Aborting request.";
CloseWriteSide();
}
QuicSpdyStream::OnStreamFrame(frame);
@@ -50,14 +50,15 @@ void QuicSpdyClientStream::OnInitialHeadersComplete(
header_bytes_read_ += frame_len;
if (!SpdyUtils::CopyAndValidateHeaders(header_list, &content_length_,
&response_headers_)) {
- DLOG(ERROR) << "Failed to parse header list: " << header_list.DebugString();
+ QUIC_DLOG(ERROR) << "Failed to parse header list: "
+ << header_list.DebugString();
Reset(QUIC_BAD_APPLICATION_PAYLOAD);
return;
}
if (!ParseHeaderStatusCode(response_headers_, &response_code_)) {
- DLOG(ERROR) << "Received invalid response code: "
- << response_headers_[":status"].as_string();
+ QUIC_DLOG(ERROR) << "Received invalid response code: "
+ << response_headers_[":status"].as_string();
Reset(QUIC_BAD_APPLICATION_PAYLOAD);
return;
}
@@ -72,7 +73,7 @@ void QuicSpdyClientStream::OnInitialHeadersComplete(
}
ConsumeHeaderList();
- DVLOG(1) << "headers complete for stream " << id();
+ QUIC_DVLOG(1) << "headers complete for stream " << id();
session_->OnInitialHeadersComplete(id(), response_headers_);
}
@@ -94,8 +95,8 @@ void QuicSpdyClientStream::OnPromiseHeaderList(
SpdyHeaderBlock promise_headers;
if (!SpdyUtils::CopyAndValidateHeaders(header_list, &content_length,
&promise_headers)) {
- DLOG(ERROR) << "Failed to parse promise headers: "
- << header_list.DebugString();
+ QUIC_DLOG(ERROR) << "Failed to parse promise headers: "
+ << header_list.DebugString();
Reset(QUIC_BAD_APPLICATION_PAYLOAD);
return;
}
@@ -118,14 +119,14 @@ void QuicSpdyClientStream::OnDataAvailable() {
// No more data to read.
break;
}
- DVLOG(1) << "Client processed " << iov.iov_len << " bytes for stream "
- << id();
+ QUIC_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_)) {
- DLOG(ERROR) << "Invalid content length (" << content_length_
- << ") with data of size " << data_.size();
+ QUIC_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