net/quic/quic_connection_test.cc - Issue 1986183002: Remove GetRttStats from QuicSentPacketManagerPeer.

Unified Diff: net/quic/quic_connection_test.cc

Issue 1986183002: Remove GetRttStats from QuicSentPacketManagerPeer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@121941143

Patch Set: Created 4 years, 7 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/quic/quic_connection_test.cc

diff --git a/net/quic/quic_connection_test.cc b/net/quic/quic_connection_test.cc

index 055da9fdfb134a4936080efec0aecd39aea978a8..5f253ac396ecb2fa9a7d73ad6d10a0656db04416 100644

--- a/net/quic/quic_connection_test.cc

+++ b/net/quic/quic_connection_test.cc

@@ -3584,7 +3584,7 @@ TEST_P(QuicConnectionTest, SendDelayedAckDecimation) {

QuicConnectionPeer::SetAckMode(&connection_, QuicConnection::ACK_DECIMATION);

const size_t kMinRttMs = 40;

- RttStats* rtt_stats = QuicSentPacketManagerPeer::GetRttStats(manager_);

+ RttStats* rtt_stats = const_cast<RttStats*>(manager_->GetRttStats());

rtt_stats->UpdateRtt(QuicTime::Delta::FromMilliseconds(kMinRttMs),

QuicTime::Delta::Zero(), QuicTime::Zero());

// The ack time should be based on min_rtt/4, since it's less than the

@@ -3637,7 +3637,7 @@ TEST_P(QuicConnectionTest, SendDelayedAckDecimationEighthRtt) {

QuicConnectionPeer::SetAckDecimationDelay(&connection_, 0.125);

const size_t kMinRttMs = 40;

- RttStats* rtt_stats = QuicSentPacketManagerPeer::GetRttStats(manager_);

+ RttStats* rtt_stats = const_cast<RttStats*>(manager_->GetRttStats());

rtt_stats->UpdateRtt(QuicTime::Delta::FromMilliseconds(kMinRttMs),

QuicTime::Delta::Zero(), QuicTime::Zero());

// The ack time should be based on min_rtt/8, since it's less than the

@@ -3690,7 +3690,7 @@ TEST_P(QuicConnectionTest, SendDelayedAckDecimationWithReordering) {

&connection_, QuicConnection::ACK_DECIMATION_WITH_REORDERING);

const size_t kMinRttMs = 40;

- RttStats* rtt_stats = QuicSentPacketManagerPeer::GetRttStats(manager_);

+ RttStats* rtt_stats = const_cast<RttStats*>(manager_->GetRttStats());

rtt_stats->UpdateRtt(QuicTime::Delta::FromMilliseconds(kMinRttMs),

QuicTime::Delta::Zero(), QuicTime::Zero());

// The ack time should be based on min_rtt/4, since it's less than the

@@ -3751,7 +3751,7 @@ TEST_P(QuicConnectionTest, SendDelayedAckDecimationWithLargeReordering) {

&connection_, QuicConnection::ACK_DECIMATION_WITH_REORDERING);

const size_t kMinRttMs = 40;

- RttStats* rtt_stats = QuicSentPacketManagerPeer::GetRttStats(manager_);

+ RttStats* rtt_stats = const_cast<RttStats*>(manager_->GetRttStats());

rtt_stats->UpdateRtt(QuicTime::Delta::FromMilliseconds(kMinRttMs),

QuicTime::Delta::Zero(), QuicTime::Zero());

// The ack time should be based on min_rtt/4, since it's less than the

@@ -3825,7 +3825,7 @@ TEST_P(QuicConnectionTest, SendDelayedAckDecimationWithReorderingEighthRtt) {

QuicConnectionPeer::SetAckDecimationDelay(&connection_, 0.125);

const size_t kMinRttMs = 40;

- RttStats* rtt_stats = QuicSentPacketManagerPeer::GetRttStats(manager_);

+ RttStats* rtt_stats = const_cast<RttStats*>(manager_->GetRttStats());

rtt_stats->UpdateRtt(QuicTime::Delta::FromMilliseconds(kMinRttMs),

QuicTime::Delta::Zero(), QuicTime::Zero());

// The ack time should be based on min_rtt/8, since it's less than the

@@ -3888,7 +3888,7 @@ TEST_P(QuicConnectionTest,

QuicConnectionPeer::SetAckDecimationDelay(&connection_, 0.125);

const size_t kMinRttMs = 40;

- RttStats* rtt_stats = QuicSentPacketManagerPeer::GetRttStats(manager_);

+ RttStats* rtt_stats = const_cast<RttStats*>(manager_->GetRttStats());

rtt_stats->UpdateRtt(QuicTime::Delta::FromMilliseconds(kMinRttMs),

QuicTime::Delta::Zero(), QuicTime::Zero());

// The ack time should be based on min_rtt/8, since it's less than the

« no previous file with comments | « no previous file | net/quic/quic_flow_controller_test.cc » ('j') | no next file with comments »