Split qpack_streams_lib into two build targets. Also add some includes. After some hand crafting, BUILD file is pretty-printed with buildifier and dependencies are cleaned by build_cleaner. The motivation is easier future development. For example, qpack_receive_stream_test should not depend on qpack_stream_sender_delegate_lib. If, hypothetically, I change an API elsewhere that affects both qpack_receive_stream_lib and qpack_stream_sender_delegate_lib, then it is convenient for me to be able to work on the former in isolation and run tests without having to worry about making the latter compile. (Sure this is hypothetical here, because there is no such API, but this is the principle I keep in mind when creating atomic build targets.) gfe-relnote: n/a, change in QUIC v99-only classes, only in BUILD and includes. PiperOrigin-RevId: 255627779 Change-Id: I6395dd97b8099c152e3be6aa15761d10bc9abb73
diff --git a/quic/core/qpack/qpack_receive_stream.h b/quic/core/qpack/qpack_receive_stream.h index 2f09f30..0e5204a 100644 --- a/quic/core/qpack/qpack_receive_stream.h +++ b/quic/core/qpack/qpack_receive_stream.h
@@ -9,6 +9,7 @@ #include "net/third_party/quiche/src/quic/platform/api/quic_export.h" namespace quic { + class QuicSpdySession; // QPACK 4.2.1 Encoder and Decoder Streams.
diff --git a/quic/core/qpack/qpack_send_stream.h b/quic/core/qpack/qpack_send_stream.h index aa3cb79..4a10e7f 100644 --- a/quic/core/qpack/qpack_send_stream.h +++ b/quic/core/qpack/qpack_send_stream.h
@@ -5,9 +5,12 @@ #ifndef QUICHE_QUIC_CORE_QPACK_QPACK_SEND_STREAM_H_ #define QUICHE_QUIC_CORE_QPACK_QPACK_SEND_STREAM_H_ +#include <cstdint> + #include "net/third_party/quiche/src/quic/core/qpack/qpack_stream_sender_delegate.h" #include "net/third_party/quiche/src/quic/core/quic_stream.h" #include "net/third_party/quiche/src/quic/platform/api/quic_export.h" +#include "net/third_party/quiche/src/quic/platform/api/quic_string_piece.h" namespace quic {