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

Unified Diff: net/tools/quic/test_tools/packet_dropping_test_writer.cc

Issue 2629723003: 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/stateless_rejector_test.cc ('k') | net/tools/quic/test_tools/quic_test_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/quic/test_tools/packet_dropping_test_writer.cc
diff --git a/net/tools/quic/test_tools/packet_dropping_test_writer.cc b/net/tools/quic/test_tools/packet_dropping_test_writer.cc
index c350c6b395c8eea21e4a1e5f384b3ac0c587c9b8..df1057a1e603023a5c4196d70d7a9bcbbc919e24 100644
--- a/net/tools/quic/test_tools/packet_dropping_test_writer.cc
+++ b/net/tools/quic/test_tools/packet_dropping_test_writer.cc
@@ -7,7 +7,6 @@
#include <limits>
#include "base/rand_util.h"
-#include "net/quic/platform/api/quic_logging.h"
#include "net/tools/quic/platform/impl/quic_socket_utils.h"
#include "net/tools/quic/quic_epoll_connection_helper.h"
@@ -22,7 +21,7 @@
: writer_(writer) {}
void OnAlarm() override {
- QUIC_DLOG(INFO) << "Unblocking socket.";
+ DVLOG(1) << "Unblocking socket.";
writer_->OnCanWrite();
}
@@ -60,7 +59,7 @@
fake_bandwidth_(QuicBandwidth::Zero()),
buffer_size_(0) {
uint32_t seed = base::RandInt(0, std::numeric_limits<int32_t>::max());
- QUIC_LOG(INFO) << "Seeding packet loss with " << seed;
+ VLOG(1) << "Seeding packet loss with " << seed;
simple_random_.set_seed(seed);
}
@@ -89,21 +88,21 @@
if (fake_drop_first_n_packets_ > 0 &&
num_calls_to_write_ <=
static_cast<uint64_t>(fake_drop_first_n_packets_)) {
- QUIC_DVLOG(1) << "Dropping first " << fake_drop_first_n_packets_
- << " packets (packet number " << num_calls_to_write_ << ")";
+ DVLOG(1) << "Dropping first " << fake_drop_first_n_packets_
+ << " packets (packet number " << num_calls_to_write_ << ")";
return WriteResult(WRITE_STATUS_OK, buf_len);
}
if (fake_packet_loss_percentage_ > 0 &&
simple_random_.RandUint64() % 100 <
static_cast<uint64_t>(fake_packet_loss_percentage_)) {
- QUIC_DVLOG(1) << "Dropping packet.";
+ DVLOG(1) << "Dropping packet.";
return WriteResult(WRITE_STATUS_OK, buf_len);
}
if (fake_blocked_socket_percentage_ > 0 &&
simple_random_.RandUint64() % 100 <
static_cast<uint64_t>(fake_blocked_socket_percentage_)) {
CHECK(on_can_write_.get() != nullptr);
- QUIC_DVLOG(1) << "Blocking socket.";
+ DVLOG(1) << "Blocking socket.";
if (!write_unblocked_alarm_->IsSet()) {
// Set the alarm to fire immediately.
write_unblocked_alarm_->Set(clock_->ApproximateNow());
@@ -114,7 +113,7 @@
if (!fake_packet_delay_.IsZero() || !fake_bandwidth_.IsZero()) {
if (buffer_size_ > 0 && buf_len + cur_buffer_size_ > buffer_size_) {
// Drop packets which do not fit into the buffer.
- QUIC_DVLOG(1) << "Dropping packet because the buffer is full.";
+ DVLOG(1) << "Dropping packet because the buffer is full.";
return WriteResult(WRITE_STATUS_OK, buf_len);
}
@@ -175,14 +174,14 @@
if (delayed_packets_.size() > 1 && fake_packet_reorder_percentage_ > 0 &&
simple_random_.RandUint64() % 100 <
static_cast<uint64_t>(fake_packet_reorder_percentage_)) {
- QUIC_DLOG(INFO) << "Reordering packets.";
+ DVLOG(1) << "Reordering packets.";
++iter;
// Swap the send times when re-ordering packets.
delayed_packets_.begin()->send_time = iter->send_time;
}
- QUIC_DLOG(INFO) << "Releasing packet. " << (delayed_packets_.size() - 1)
- << " remaining.";
+ DVLOG(1) << "Releasing packet. " << (delayed_packets_.size() - 1)
+ << " remaining.";
// Grab the next one off the queue and send it.
QuicPacketWriterWrapper::WritePacket(
iter->buffer.data(), iter->buffer.length(), iter->self_address,
« no previous file with comments | « net/tools/quic/stateless_rejector_test.cc ('k') | net/tools/quic/test_tools/quic_test_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698