commit bd6ee225089aaca45192e9b33e715eda1b8f8cb7
author bnc <bnc@google.com> Tue Jun 25 18:16:32 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Jun 26 10:32:59 2019 -0700
tree 92b935cb693129379f462803c5766905ae6b5596
parent b9d07d9a8d1221b5ab8e09a36759d6a166bf975b

Make QuicSpdyStream methods only called by HttpDecoderVisitor private.

These methods should really only be called by HttpDecoderVisitor, which is a
member class and therefore can access private methods anyway.

gfe-relnote: n/a, no functional change.
PiperOrigin-RevId: 255091594
Change-Id: If7821fee9e1821e2aeb4ac786cbf244bb12a7934

quic/core/http/quic_spdy_stream.h

diff --git a/quic/core/http/quic_spdy_stream.h b/quic/core/http/quic_spdy_stream.h
index c0431f1..1618f0e 100644
--- a/quic/core/http/quic_spdy_stream.h
+++ b/quic/core/http/quic_spdy_stream.h
@@ -207,14 +207,6 @@
   using QuicStream::CloseWriteSide;
 
  protected:
-  // HTTP/3
-  bool OnDataFrameStart(Http3FrameLengths frame_lengths);
-  bool OnDataFramePayload(QuicStringPiece payload);
-  bool OnDataFrameEnd();
-  bool OnHeadersFrameStart(Http3FrameLengths frame_length);
-  bool OnHeadersFramePayload(QuicStringPiece payload);
-  bool OnHeadersFrameEnd();
-
   // Called when the received headers are too large. By default this will
   // reset the stream.
   virtual void OnHeadersTooLarge();
@@ -246,6 +238,14 @@
   friend class QuicStreamUtils;
   class HttpDecoderVisitor;
 
+  // Called by HttpDecoderVisitor.
+  bool OnDataFrameStart(Http3FrameLengths frame_lengths);
+  bool OnDataFramePayload(QuicStringPiece payload);
+  bool OnDataFrameEnd();
+  bool OnHeadersFrameStart(Http3FrameLengths frame_length);
+  bool OnHeadersFramePayload(QuicStringPiece payload);
+  bool OnHeadersFrameEnd();
+
   // Call QuicStreamSequencer::MarkConsumed() with
   // |headers_bytes_to_be_marked_consumed_| if appropriate.
   void MaybeMarkHeadersBytesConsumed();