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

Unified Diff: net/quic/quic_protocol.cc

Issue 1661863002: Remove a test-only SerializedPacket constructor from QUIC. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@113110142
Patch Set: Created 4 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/quic/quic_protocol.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_protocol.cc
diff --git a/net/quic/quic_protocol.cc b/net/quic/quic_protocol.cc
index 4e5eb8921aa74581aeaf510d8d91fac4cc437dba..8604d2f27a2b72cde940c3ef1ff73bafe62b8a4c 100644
--- a/net/quic/quic_protocol.cc
+++ b/net/quic/quic_protocol.cc
@@ -753,36 +753,6 @@ SerializedPacket::SerializedPacket(QuicPathId path_id,
original_packet_number(0),
transmission_type(NOT_RETRANSMISSION) {}
-SerializedPacket::SerializedPacket(QuicPathId path_id,
- QuicPacketNumber packet_number,
- QuicPacketNumberLength packet_number_length,
- char* encrypted_buffer,
- size_t encrypted_length,
- bool owns_buffer,
- QuicPacketEntropyHash entropy_hash,
- QuicFrames* retransmittable_frames,
- bool padding,
- IsHandshake is_handshake,
- bool has_ack,
- bool has_stop_waiting,
- EncryptionLevel level)
- : SerializedPacket(path_id,
- packet_number,
- packet_number_length,
- new QuicEncryptedPacket(encrypted_buffer,
- encrypted_length,
- owns_buffer),
- entropy_hash,
- retransmittable_frames,
- has_ack,
- has_stop_waiting) {
- // TODO(ianswett): Move into the initializer list once SerializedPacket
- // no longer contains an encrypted packet.
- encryption_level = level;
- needs_padding = padding;
- has_crypto_handshake = is_handshake;
-}
-
SerializedPacket::~SerializedPacket() {}
QuicEncryptedPacket* QuicEncryptedPacket::Clone() const {
« no previous file with comments | « net/quic/quic_protocol.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698