commit 28bb06cdda73dc9d9fc5b652f05833dc1e64cb33
author bnc <bnc@google.com> Wed Jul 17 05:21:10 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Wed Jul 17 05:22:20 2019 -0700
tree 79be0de0a8084c8d3b3ee1d6406e6a4b95ca082d
parent bf3dbe5ddeef461357da55e7d34c225c9473f7c1

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

quic/core/qpack/qpack_encoder_stream_sender.cc

diff --git a/quic/core/qpack/qpack_encoder_stream_sender.cc b/quic/core/qpack/qpack_encoder_stream_sender.cc
index ca1da4f..dce183a 100644
--- a/quic/core/qpack/qpack_encoder_stream_sender.cc
+++ b/quic/core/qpack/qpack_encoder_stream_sender.cc
@@ -23,39 +23,42 @@
     bool is_static,
     uint64_t name_index,
     QuicStringPiece value) {
-  instruction_encoder_.set_s_bit(is_static);
-  instruction_encoder_.set_varint(name_index);
-  instruction_encoder_.set_value(value);
+  values_.s_bit = is_static;
+  values_.varint = name_index;
+  values_.value = value;
 
   std::string output;
-  instruction_encoder_.Encode(InsertWithNameReferenceInstruction(), &output);
+  instruction_encoder_.Encode(InsertWithNameReferenceInstruction(), values_,
+                              &output);
   delegate_->WriteStreamData(output);
 }
 
 void QpackEncoderStreamSender::SendInsertWithoutNameReference(
     QuicStringPiece name,
     QuicStringPiece value) {
-  instruction_encoder_.set_name(name);
-  instruction_encoder_.set_value(value);
+  values_.name = name;
+  values_.value = value;
 
   std::string output;
-  instruction_encoder_.Encode(InsertWithoutNameReferenceInstruction(), &output);
+  instruction_encoder_.Encode(InsertWithoutNameReferenceInstruction(), values_,
+                              &output);
   delegate_->WriteStreamData(output);
 }
 
 void QpackEncoderStreamSender::SendDuplicate(uint64_t index) {
-  instruction_encoder_.set_varint(index);
+  values_.varint = index;
 
   std::string output;
-  instruction_encoder_.Encode(DuplicateInstruction(), &output);
+  instruction_encoder_.Encode(DuplicateInstruction(), values_, &output);
   delegate_->WriteStreamData(output);
 }
 
 void QpackEncoderStreamSender::SendSetDynamicTableCapacity(uint64_t capacity) {
-  instruction_encoder_.set_varint(capacity);
+  values_.varint = capacity;
 
   std::string output;
-  instruction_encoder_.Encode(SetDynamicTableCapacityInstruction(), &output);
+  instruction_encoder_.Encode(SetDynamicTableCapacityInstruction(), values_,
+                              &output);
   delegate_->WriteStreamData(output);
 }