Make QpackEncoder::EncodeHeaderList() and FirstPassEncode() take header_list by const ref.
Changing argument type from const* to const&, as discussed on cr/264499016.
gfe-relnote: n/a, change method signatures, no functional change.
PiperOrigin-RevId: 264808104
Change-Id: I84d6c8c6d4446648bd675e6d9cd58fdd50501693
diff --git a/quic/core/http/end_to_end_test.cc b/quic/core/http/end_to_end_test.cc
index 27d3785..6eb6ac6 100644
--- a/quic/core/http/end_to_end_test.cc
+++ b/quic/core/http/end_to_end_test.cc
@@ -2469,7 +2469,7 @@
qpack_encoder.set_qpack_stream_sender_delegate(
&encoder_stream_sender_delegate);
std::string encoded_headers =
- qpack_encoder.EncodeHeaderList(/* stream_id = */ 0, &headers);
+ qpack_encoder.EncodeHeaderList(/* stream_id = */ 0, headers);
header_size = encoded_headers.size();
}
diff --git a/quic/core/http/quic_spdy_session.cc b/quic/core/http/quic_spdy_session.cc
index 29c6d9d..c869459 100644
--- a/quic/core/http/quic_spdy_session.cc
+++ b/quic/core/http/quic_spdy_session.cc
@@ -545,7 +545,7 @@
// Encode header list.
std::string encoded_headers =
- qpack_encoder_->EncodeHeaderList(original_stream_id, &headers);
+ qpack_encoder_->EncodeHeaderList(original_stream_id, headers);
PushPromiseFrame frame;
frame.push_id = promised_stream_id;
frame.headers = encoded_headers;
diff --git a/quic/core/http/quic_spdy_stream.cc b/quic/core/http/quic_spdy_stream.cc
index 2471756..464c0a3 100644
--- a/quic/core/http/quic_spdy_stream.cc
+++ b/quic/core/http/quic_spdy_stream.cc
@@ -1018,7 +1018,7 @@
// Encode header list.
std::string encoded_headers =
- spdy_session_->qpack_encoder()->EncodeHeaderList(id(), &header_block);
+ spdy_session_->qpack_encoder()->EncodeHeaderList(id(), header_block);
// Write HEADERS frame.
std::unique_ptr<char[]> headers_frame_header;
diff --git a/quic/core/http/quic_spdy_stream_test.cc b/quic/core/http/quic_spdy_stream_test.cc
index f247df4..8b1c922 100644
--- a/quic/core/http/quic_spdy_stream_test.cc
+++ b/quic/core/http/quic_spdy_stream_test.cc
@@ -184,7 +184,7 @@
&encoder_stream_sender_delegate);
// QpackEncoder does not use the dynamic table by default,
// therefore the value of |stream_id| does not matter.
- return qpack_encoder->EncodeHeaderList(/* stream_id = */ 0, &header);
+ return qpack_encoder->EncodeHeaderList(/* stream_id = */ 0, header);
}
void Initialize(bool stream_should_process_data) {
