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

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

Issue 2849543002: Remove unused packet parameter from QuicTimeWaitListManager::ProcessPacket. (Closed)
Patch Set: Created 3 years, 8 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 | « no previous file | net/tools/quic/quic_dispatcher_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/quic/quic_dispatcher.cc
diff --git a/net/tools/quic/quic_dispatcher.cc b/net/tools/quic/quic_dispatcher.cc
index edd34d81c03fe9d561763cbeaa80c09c3de1f9a6..5e325f353c82c924f53ff99ec26db2398c4347dc 100644
--- a/net/tools/quic/quic_dispatcher.cc
+++ b/net/tools/quic/quic_dispatcher.cc
@@ -336,9 +336,9 @@ bool QuicDispatcher::OnUnauthenticatedHeader(const QuicPacketHeader& header) {
if (time_wait_list_manager_->IsConnectionIdInTimeWait(
header.public_header.connection_id)) {
// This connection ID is already in time-wait state.
- time_wait_list_manager_->ProcessPacket(
- current_server_address_, current_client_address_,
- header.public_header.connection_id, *current_packet_);
+ time_wait_list_manager_->ProcessPacket(current_server_address_,
+ current_client_address_,
+ header.public_header.connection_id);
return false;
}
@@ -380,9 +380,8 @@ void QuicDispatcher::ProcessUnauthenticatedHeaderFate(
/*connection_rejected_statelessly=*/false, nullptr);
}
DCHECK(time_wait_list_manager_->IsConnectionIdInTimeWait(connection_id));
- time_wait_list_manager_->ProcessPacket(current_server_address_,
- current_client_address_,
- connection_id, *current_packet_);
+ time_wait_list_manager_->ProcessPacket(
+ current_server_address_, current_client_address_, connection_id);
// Any packets which were buffered while the stateless rejector logic was
// running should be discarded. Do not inform the time wait list manager,
@@ -724,9 +723,9 @@ void QuicDispatcher::ProcessChlo() {
/*connection_rejected_statelessly=*/false,
/*termination_packets=*/nullptr);
// This will trigger sending Public Reset packet.
- time_wait_list_manager()->ProcessPacket(
- current_server_address(), current_client_address(),
- current_connection_id(), current_packet());
+ time_wait_list_manager()->ProcessPacket(current_server_address(),
+ current_client_address(),
+ current_connection_id());
return;
}
if (FLAGS_quic_reloadable_flag_quic_create_session_after_insertion &&
@@ -916,9 +915,9 @@ void QuicDispatcher::OnStatelessRejectorProcessDone(
// don't proceed.
if (time_wait_list_manager_->IsConnectionIdInTimeWait(
rejector->connection_id())) {
- time_wait_list_manager_->ProcessPacket(
- current_server_address, current_client_address,
- rejector->connection_id(), *current_packet);
+ time_wait_list_manager_->ProcessPacket(current_server_address,
+ current_client_address,
+ rejector->connection_id());
return;
}
« no previous file with comments | « no previous file | net/tools/quic/quic_dispatcher_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698