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

Side by Side Diff: net/quic/quic_packet_creator_test.cc

Issue 1780573006: Deprecate FLAG_quic_no_unencrypted_fec. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@116180343
Patch Set: Created 4 years, 9 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 unified diff | Download patch
« no previous file with comments | « net/quic/quic_flags.cc ('k') | net/quic/quic_packet_generator_test.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/quic/quic_packet_creator.h" 5 #include "net/quic/quic_packet_creator.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/stl_util.h" 10 #include "base/stl_util.h"
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 FLAGS_quic_always_log_bugs_for_tests = true; 134 FLAGS_quic_always_log_bugs_for_tests = true;
135 creator_.set_connection_id_length(GetParam().connection_id_length); 135 creator_.set_connection_id_length(GetParam().connection_id_length);
136 136
137 creator_.SetEncrypter(ENCRYPTION_INITIAL, new NullEncrypter()); 137 creator_.SetEncrypter(ENCRYPTION_INITIAL, new NullEncrypter());
138 creator_.SetEncrypter(ENCRYPTION_FORWARD_SECURE, new NullEncrypter()); 138 creator_.SetEncrypter(ENCRYPTION_FORWARD_SECURE, new NullEncrypter());
139 client_framer_.set_visitor(&framer_visitor_); 139 client_framer_.set_visitor(&framer_visitor_);
140 client_framer_.set_received_entropy_calculator(&entropy_calculator_); 140 client_framer_.set_received_entropy_calculator(&entropy_calculator_);
141 server_framer_.set_visitor(&framer_visitor_); 141 server_framer_.set_visitor(&framer_visitor_);
142 // TODO(ianswett): Fix this test so it uses a non-null encrypter. 142 // TODO(ianswett): Fix this test so it uses a non-null encrypter.
143 FLAGS_quic_never_write_unencrypted_data = false; 143 FLAGS_quic_never_write_unencrypted_data = false;
144 FLAGS_quic_no_unencrypted_fec = false;
145 } 144 }
146 145
147 ~QuicPacketCreatorTest() override { 146 ~QuicPacketCreatorTest() override {
148 delete[] serialized_packet_.encrypted_buffer; 147 delete[] serialized_packet_.encrypted_buffer;
149 ClearSerializedPacket(&serialized_packet_); 148 ClearSerializedPacket(&serialized_packet_);
150 } 149 }
151 150
152 SerializedPacket SerializeAllFrames(const QuicFrames& frames) { 151 SerializedPacket SerializeAllFrames(const QuicFrames& frames) {
153 SerializedPacket packet = QuicPacketCreatorPeer::SerializeAllFrames( 152 SerializedPacket packet = QuicPacketCreatorPeer::SerializeAllFrames(
154 &creator_, frames, buffer_, kMaxPacketSize); 153 &creator_, frames, buffer_, kMaxPacketSize);
(...skipping 984 matching lines...) Expand 10 before | Expand all | Expand 10 after
1139 EXPECT_CALL(delegate_, OnUnrecoverableError(_, _)); 1138 EXPECT_CALL(delegate_, OnUnrecoverableError(_, _));
1140 QuicStreamFrame stream_frame(kHeadersStreamId, /*fin=*/false, 0u, 1139 QuicStreamFrame stream_frame(kHeadersStreamId, /*fin=*/false, 0u,
1141 StringPiece()); 1140 StringPiece());
1142 EXPECT_DFATAL(creator_.AddSavedFrame(QuicFrame(&stream_frame)), 1141 EXPECT_DFATAL(creator_.AddSavedFrame(QuicFrame(&stream_frame)),
1143 "Cannot send stream data without encryption."); 1142 "Cannot send stream data without encryption.");
1144 } 1143 }
1145 1144
1146 } // namespace 1145 } // namespace
1147 } // namespace test 1146 } // namespace test
1148 } // namespace net 1147 } // namespace net
OLDNEW
« no previous file with comments | « net/quic/quic_flags.cc ('k') | net/quic/quic_packet_generator_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698