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

Unified Diff: net/quic/test_tools/quic_sent_packet_manager_peer.cc

Issue 2002083002: Add QuicSentPacketManagerInterface, and QuicSentPacketManager implements it. No functional change e… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
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
« no previous file with comments | « net/quic/test_tools/quic_sent_packet_manager_peer.h ('k') | net/quic/test_tools/quic_test_utils.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/test_tools/quic_sent_packet_manager_peer.cc
diff --git a/net/quic/test_tools/quic_sent_packet_manager_peer.cc b/net/quic/test_tools/quic_sent_packet_manager_peer.cc
index ed5e554b6000874f87b626c5f41aabb035f3f5ae..7dc83ca0b46027b6236bfa2aaebb185c66c37554 100644
--- a/net/quic/test_tools/quic_sent_packet_manager_peer.cc
+++ b/net/quic/test_tools/quic_sent_packet_manager_peer.cc
@@ -104,9 +104,10 @@ QuicTime QuicSentPacketManagerPeer::GetSentTime(
// static
bool QuicSentPacketManagerPeer::IsRetransmission(
QuicSentPacketManager* sent_packet_manager,
+ QuicPathId path_id,
QuicPacketNumber packet_number) {
- DCHECK(sent_packet_manager->HasRetransmittableFrames(packet_number));
- if (!sent_packet_manager->HasRetransmittableFrames(packet_number)) {
+ DCHECK(sent_packet_manager->HasRetransmittableFrames(path_id, packet_number));
+ if (!sent_packet_manager->HasRetransmittableFrames(path_id, packet_number)) {
return false;
}
for (auto transmission_info : sent_packet_manager->unacked_packets_) {
@@ -120,6 +121,7 @@ bool QuicSentPacketManagerPeer::IsRetransmission(
// static
void QuicSentPacketManagerPeer::MarkForRetransmission(
QuicSentPacketManager* sent_packet_manager,
+ QuicPathId path_id,
QuicPacketNumber packet_number,
TransmissionType transmission_type) {
sent_packet_manager->MarkForRetransmission(packet_number, transmission_type);
@@ -158,7 +160,7 @@ QuicByteCount QuicSentPacketManagerPeer::GetBytesInFlight(
}
// static
-QuicSentPacketManager::NetworkChangeVisitor*
+QuicSentPacketManagerInterface::NetworkChangeVisitor*
QuicSentPacketManagerPeer::GetNetworkChangeVisitor(
const QuicSentPacketManager* sent_packet_manager) {
return sent_packet_manager->network_change_visitor_;
@@ -184,5 +186,11 @@ QuicSustainedBandwidthRecorder& QuicSentPacketManagerPeer::GetBandwidthRecorder(
return sent_packet_manager->sustained_bandwidth_recorder_;
}
+// static
+bool QuicSentPacketManagerPeer::UsingPacing(
+ const QuicSentPacketManager* sent_packet_manager) {
+ return sent_packet_manager->using_pacing_;
+}
+
} // namespace test
} // namespace net
« no previous file with comments | « net/quic/test_tools/quic_sent_packet_manager_peer.h ('k') | net/quic/test_tools/quic_test_utils.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698