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

Unified Diff: net/quic/quic_framer_test.cc

Issue 11961022: Remove default constructors for QuicTime and QuicTime::Delta and used static Zero() methods instead… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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
Index: net/quic/quic_framer_test.cc
diff --git a/net/quic/quic_framer_test.cc b/net/quic/quic_framer_test.cc
index 9799719ec720dfae355c1fb3c0a93481b98e0173..9b40488274d46e9062a58321ef34b653ecdd4707 100644
--- a/net/quic/quic_framer_test.cc
+++ b/net/quic/quic_framer_test.cc
@@ -19,6 +19,7 @@
using base::hash_set;
using base::StringPiece;
+using std::make_pair;
using std::map;
using std::string;
using std::vector;
@@ -1196,13 +1197,15 @@ TEST_F(QuicFramerTest, ConstructCongestionFeedbackFramePacketInterArrival) {
frame.type = kInterArrival;
frame.inter_arrival.accumulated_number_of_lost_packets
= 0x0302;
- frame.inter_arrival.received_packet_times[GG_UINT64_C(0x0123456789ABA)] =
- QuicTime::FromMicroseconds(GG_UINT64_C(0x07E1D2C3B4A59687));
- frame.inter_arrival.received_packet_times[GG_UINT64_C(0x0123456789ABB)] =
- QuicTime::FromMicroseconds(GG_UINT64_C(0x07E1D2C3B4A59688));
- frame.inter_arrival.received_packet_times[GG_UINT64_C(0x0123456789ABD)] =
- QuicTime::FromMicroseconds(GG_UINT64_C(0x07E1D2C3B4A59689));
-
+ frame.inter_arrival.received_packet_times.insert(
+ make_pair(GG_UINT64_C(0x0123456789ABA),
+ QuicTime::FromMicroseconds(GG_UINT64_C(0x07E1D2C3B4A59687))));
+ frame.inter_arrival.received_packet_times.insert(
+ make_pair(GG_UINT64_C(0x0123456789ABB),
+ QuicTime::FromMicroseconds(GG_UINT64_C(0x07E1D2C3B4A59688))));
+ frame.inter_arrival.received_packet_times.insert(
+ make_pair(GG_UINT64_C(0x0123456789ABD),
+ QuicTime::FromMicroseconds(GG_UINT64_C(0x07E1D2C3B4A59689))));
QuicFrames frames;
frames.push_back(QuicFrame(&frame));

Powered by Google App Engine
This is Rietveld 408576698