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

Unified Diff: net/quic/core/quic_sent_packet_manager.cc

Issue 2556813003: Delete unused code from QuicSentPacketManager. No behavior change. (Closed)
Patch Set: Created 4 years 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/core/quic_sent_packet_manager.h ('k') | net/quic/test_tools/quic_sent_packet_manager_peer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/core/quic_sent_packet_manager.cc
diff --git a/net/quic/core/quic_sent_packet_manager.cc b/net/quic/core/quic_sent_packet_manager.cc
index 43ac40042920011cf6258b872020dcfb60446a48..ce4ca59e516e9f3898f064e2b79f28115c8c33e3 100644
--- a/net/quic/core/quic_sent_packet_manager.cc
+++ b/net/quic/core/quic_sent_packet_manager.cc
@@ -73,7 +73,6 @@ QuicSentPacketManager::QuicSentPacketManager(
loss_algorithm_(&general_loss_algorithm_),
general_loss_algorithm_(loss_type),
n_connection_simulation_(false),
- receive_buffer_bytes_(kDefaultSocketReceiveBuffer),
least_packet_awaited_by_peer_(1),
first_rto_transmission_(0),
consecutive_rto_count_(0),
@@ -498,29 +497,6 @@ QuicPacketNumber QuicSentPacketManager::GetNewestRetransmission(
return packet_number;
}
-void QuicSentPacketManager::MarkPacketNotRetransmittable(
- QuicPacketNumber packet_number,
- QuicTime::Delta ack_delay_time) {
- if (!unacked_packets_.IsUnacked(packet_number)) {
- return;
- }
-
- const QuicTransmissionInfo& transmission_info =
- unacked_packets_.GetTransmissionInfo(packet_number);
- QuicPacketNumber newest_transmission =
- GetNewestRetransmission(packet_number, transmission_info);
- // We do not need to retransmit this packet anymore.
- if (delegate_ != nullptr) {
- delegate_->OnPacketMarkedNotRetransmittable(path_id_, newest_transmission,
- ack_delay_time);
- } else {
- pending_retransmissions_.erase(newest_transmission);
- }
-
- unacked_packets_.NotifyAndClearListeners(newest_transmission, ack_delay_time);
- unacked_packets_.RemoveRetransmittability(packet_number);
-}
-
void QuicSentPacketManager::MarkPacketHandled(QuicPacketNumber packet_number,
QuicTransmissionInfo* info,
QuicTime::Delta ack_delay_time) {
@@ -1046,15 +1022,6 @@ size_t QuicSentPacketManager::GetConsecutiveTlpCount() const {
return consecutive_tlp_count_;
}
-QuicTransmissionInfo* QuicSentPacketManager::GetMutableTransmissionInfo(
- QuicPacketNumber packet_number) {
- return unacked_packets_.GetMutableTransmissionInfo(packet_number);
-}
-
-void QuicSentPacketManager::RemoveObsoletePackets() {
- unacked_packets_.RemoveObsoletePackets();
-}
-
void QuicSentPacketManager::OnApplicationLimited() {
send_algorithm_->OnApplicationLimited(unacked_packets_.bytes_in_flight());
}
« no previous file with comments | « net/quic/core/quic_sent_packet_manager.h ('k') | net/quic/test_tools/quic_sent_packet_manager_peer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698