Factor out QpackInstructionEncoder::Values.
Factor out some QpackInstructionEncoder members into a struct that the caller
can pass to Encode(). Also add DCHECK to Encode() and rename DoStaticBit() to
DoSBit() (since is can be used for either static bit or sign bit).
gfe-relnote: n/a, change to QUIC v99-only code.
PiperOrigin-RevId: 258548817
Change-Id: I1c5d02b16af1e2db4c71813cdfad9548774a8c48
diff --git a/quic/core/qpack/qpack_decoder_stream_sender.cc b/quic/core/qpack/qpack_decoder_stream_sender.cc
index 4586fd3..63f1721 100644
--- a/quic/core/qpack/qpack_decoder_stream_sender.cc
+++ b/quic/core/qpack/qpack_decoder_stream_sender.cc
@@ -20,27 +20,30 @@
}
void QpackDecoderStreamSender::SendInsertCountIncrement(uint64_t increment) {
- instruction_encoder_.set_varint(increment);
+ values_.varint = increment;
std::string output;
- instruction_encoder_.Encode(InsertCountIncrementInstruction(), &output);
+ instruction_encoder_.Encode(InsertCountIncrementInstruction(), values_,
+ &output);
delegate_->WriteStreamData(output);
}
void QpackDecoderStreamSender::SendHeaderAcknowledgement(
QuicStreamId stream_id) {
- instruction_encoder_.set_varint(stream_id);
+ values_.varint = stream_id;
std::string output;
- instruction_encoder_.Encode(HeaderAcknowledgementInstruction(), &output);
+ instruction_encoder_.Encode(HeaderAcknowledgementInstruction(), values_,
+ &output);
delegate_->WriteStreamData(output);
}
void QpackDecoderStreamSender::SendStreamCancellation(QuicStreamId stream_id) {
- instruction_encoder_.set_varint(stream_id);
+ values_.varint = stream_id;
std::string output;
- instruction_encoder_.Encode(StreamCancellationInstruction(), &output);
+ instruction_encoder_.Encode(StreamCancellationInstruction(), values_,
+ &output);
delegate_->WriteStreamData(output);
}