Make QUIC's SerializedPacket contain QuicFrames, rather than a pointer

net/quic/quic_packet_creator_test.cc

Index: net/quic/quic_packet_creator_test.cc
diff --git a/net/quic/quic_packet_creator_test.cc b/net/quic/quic_packet_creator_test.cc
index fc6b7a23212e5c0116f49c5f94dc12dce3b269ac..9b1ff0307a092396401afb0585c219f4ca7eeafd 100644
--- a/net/quic/quic_packet_creator_test.cc
+++ b/net/quic/quic_packet_creator_test.cc
@@ -106,6 +106,7 @@ class QuicPacketCreatorTest : public ::testing::TestWithParam<TestParams> {
     serialized_packet_ = *serialized_packet;
     serialized_packet_.packet = serialized_packet->packet->Clone();
     delete serialized_packet->packet;
+    serialized_packet->retransmittable_frames.clear();
   }
 
  protected:
@@ -1207,11 +1208,11 @@ TEST_P(QuicPacketCreatorTest, AddFrameAndFlush) {
 
   // Ensure the packet is successfully created.
   ASSERT_TRUE(serialized_packet_.packet);
-  ASSERT_TRUE(serialized_packet_.retransmittable_frames);
-  QuicFrames* retransmittable = serialized_packet_.retransmittable_frames;
-  ASSERT_EQ(1u, retransmittable->size());
-  EXPECT_EQ(STREAM_FRAME, (*retransmittable)[0].type);
-  ASSERT_TRUE((*retransmittable)[0].stream_frame);
+  ASSERT_FALSE(serialized_packet_.retransmittable_frames.empty());
+  const QuicFrames& retransmittable = serialized_packet_.retransmittable_frames;
+  ASSERT_EQ(1u, retransmittable.size());
+  EXPECT_EQ(STREAM_FRAME, retransmittable[0].type);
+  ASSERT_TRUE(retransmittable[0].stream_frame);
   ClearSerializedPacket(&serialized_packet_);
 
   EXPECT_FALSE(creator_.HasPendingFrames());