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

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

Issue 2611613003: Add quic_logging (Closed)
Patch Set: fix for win 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
Index: net/tools/quic/quic_dispatcher_test.cc
diff --git a/net/tools/quic/quic_dispatcher_test.cc b/net/tools/quic/quic_dispatcher_test.cc
index 87382d91e88db61829e742e299e856fb745eeadb..e2d7c1b00160072765fd2eae3d2b5f9006bb9d2a 100644
--- a/net/tools/quic/quic_dispatcher_test.cc
+++ b/net/tools/quic/quic_dispatcher_test.cc
@@ -15,6 +15,7 @@
#include "net/quic/core/quic_crypto_stream.h"
#include "net/quic/core/quic_flags.h"
#include "net/quic/core/quic_utils.h"
+#include "net/quic/platform/api/quic_logging.h"
#include "net/quic/platform/api/quic_str_cat.h"
#include "net/quic/test_tools/crypto_test_utils.h"
#include "net/quic/test_tools/fake_proof_source.h"
@@ -153,7 +154,7 @@ class MockServerConnection : public MockQuicConnection {
dispatcher_(dispatcher) {}
void UnregisterOnConnectionClosed() {
- LOG(ERROR) << "Unregistering " << connection_id();
+ QUIC_LOG(ERROR) << "Unregistering " << connection_id();
dispatcher_->OnConnectionClosed(connection_id(), QUIC_NO_ERROR,
"Unregistering.");
}
@@ -814,8 +815,8 @@ TEST_P(QuicDispatcherStatelessRejectTest, CheapRejects) {
&QuicDispatcherTest::ValidatePacket, base::Unretained(this), 1))));
}
- VLOG(1) << "ExpectStatelessReject: " << ExpectStatelessReject();
- VLOG(1) << "Params: " << GetParam();
+ QUIC_LOG(INFO) << "ExpectStatelessReject: " << ExpectStatelessReject();
+ QUIC_LOG(INFO) << "Params: " << GetParam();
// Process the first packet for the connection.
// clang-format off
CryptoHandshakeMessage client_hello = CryptoTestUtils::Message(
@@ -921,7 +922,7 @@ class BlockingWriter : public QuicPacketWriterWrapper {
// It would be quite possible to actually implement this method here with
// the fake blocked status, but it would be significantly more work in
// Chromium, and since it's not called anyway, don't bother.
- LOG(DFATAL) << "Not supported";
+ QUIC_LOG(DFATAL) << "Not supported";
return WriteResult();
}

Powered by Google App Engine
This is Rietveld 408576698