commit abfd9ecc744126fbf5624f7cdf5214d911af6547
author fkastenholz <fkastenholz@google.com> Thu Oct 31 07:59:12 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Oct 31 07:59:51 2019 -0700
tree 58743b168924879b8a0e1bc6687e8ec28c6e173f
parent 1c1dcabd1c0a362155015bb9c586d23fcde64421

Change GetMinStreamSize API to use size_t, rather than uint16_t avoiding truncation

gfe-relnote: N/A, api change only.
PiperOrigin-RevId: 277721016
Change-Id: I3e8fba1588d27c2ea1a3a4f6e5fa6de425bba732

quic/core/quic_framer.cc

diff --git a/quic/core/quic_framer.cc b/quic/core/quic_framer.cc
index 48eb295..756788f 100644
--- a/quic/core/quic_framer.cc
+++ b/quic/core/quic_framer.cc
@@ -449,7 +449,7 @@
                                          QuicStreamId stream_id,
                                          QuicStreamOffset offset,
                                          bool last_frame_in_packet,
-                                         QuicPacketLength data_length) {
+                                         size_t data_length) {
   if (VersionHasIetfQuicFrames(version)) {
     return kQuicFrameTypeSize + QuicDataWriter::GetVarInt62Len(stream_id) +
            (last_frame_in_packet

quic/core/quic_framer.h

diff --git a/quic/core/quic_framer.h b/quic/core/quic_framer.h
index 4e1b11e..d9e9d27 100644
--- a/quic/core/quic_framer.h
+++ b/quic/core/quic_framer.h
@@ -288,7 +288,7 @@
                                       QuicStreamId stream_id,
                                       QuicStreamOffset offset,
                                       bool last_frame_in_packet,
-                                      QuicPacketLength data_length);
+                                      size_t data_length);
   // Returns the overhead of framing a CRYPTO frame with the specific offset and
   // data length provided, but not counting the size of the data payload.
   static size_t GetMinCryptoFrameSize(QuicStreamOffset offset,