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

Side by Side Diff: net/quic/core/quic_server_session_base_test.cc

Issue 2353063002: Only inform QUIC's SentPacketManager of packets that are actually sent, not those that failed to be… (Closed)
Patch Set: Created 4 years, 3 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/core/quic_flags_list.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/core/quic_server_session_base.h" 5 #include "net/quic/core/quic_server_session_base.h"
6 6
7 #include <cstdint> 7 #include <cstdint>
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 StrictMock<MockQuicCryptoServerStreamHelper> stream_helper_; 155 StrictMock<MockQuicCryptoServerStreamHelper> stream_helper_;
156 MockQuicConnectionHelper helper_; 156 MockQuicConnectionHelper helper_;
157 MockAlarmFactory alarm_factory_; 157 MockAlarmFactory alarm_factory_;
158 StrictMock<MockQuicConnection>* connection_; 158 StrictMock<MockQuicConnection>* connection_;
159 QuicConfig config_; 159 QuicConfig config_;
160 QuicCryptoServerConfig crypto_config_; 160 QuicCryptoServerConfig crypto_config_;
161 QuicCompressedCertsCache compressed_certs_cache_; 161 QuicCompressedCertsCache compressed_certs_cache_;
162 std::unique_ptr<TestServerSession> session_; 162 std::unique_ptr<TestServerSession> session_;
163 std::unique_ptr<CryptoHandshakeMessage> handshake_message_; 163 std::unique_ptr<CryptoHandshakeMessage> handshake_message_;
164 QuicConnectionVisitorInterface* visitor_; 164 QuicConnectionVisitorInterface* visitor_;
165 QuicFlagSaver flags_; // Save/restore all QUIC flag values.
165 }; 166 };
166 167
167 // Compares CachedNetworkParameters. 168 // Compares CachedNetworkParameters.
168 MATCHER_P(EqualsProto, network_params, "") { 169 MATCHER_P(EqualsProto, network_params, "") {
169 CachedNetworkParameters reference(network_params); 170 CachedNetworkParameters reference(network_params);
170 return (arg->bandwidth_estimate_bytes_per_second() == 171 return (arg->bandwidth_estimate_bytes_per_second() ==
171 reference.bandwidth_estimate_bytes_per_second() && 172 reference.bandwidth_estimate_bytes_per_second() &&
172 arg->bandwidth_estimate_bytes_per_second() == 173 arg->bandwidth_estimate_bytes_per_second() ==
173 reference.bandwidth_estimate_bytes_per_second() && 174 reference.bandwidth_estimate_bytes_per_second() &&
174 arg->max_bandwidth_estimate_bytes_per_second() == 175 arg->max_bandwidth_estimate_bytes_per_second() ==
(...skipping 392 matching lines...) Expand 10 before | Expand all | Expand 10 after
567 EXPECT_FALSE( 568 EXPECT_FALSE(
568 QuicServerSessionBasePeer::IsBandwidthResumptionEnabled(session_.get())); 569 QuicServerSessionBasePeer::IsBandwidthResumptionEnabled(session_.get()));
569 session_->OnConfigNegotiated(); 570 session_->OnConfigNegotiated();
570 EXPECT_FALSE( 571 EXPECT_FALSE(
571 QuicServerSessionBasePeer::IsBandwidthResumptionEnabled(session_.get())); 572 QuicServerSessionBasePeer::IsBandwidthResumptionEnabled(session_.get()));
572 } 573 }
573 574
574 } // namespace 575 } // namespace
575 } // namespace test 576 } // namespace test
576 } // namespace net 577 } // namespace net
OLDNEW
« no previous file with comments | « net/quic/core/quic_flags_list.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698