commit 978573522275f7c493ebf2eba329577770a5ccba
author fayang <fayang@google.com> Mon Jul 01 06:15:26 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Jul 02 19:30:51 2019 -0700
tree 48ade02718f7be3c39d7be38ee36c7a5a5bc17b7
parent 360b5a9881654061595dfb4145caa38bb5091dd0

gfe-relnote: In QUIC, inline WritevDataInner. Refactoring only, not protected.

PiperOrigin-RevId: 255941214
Change-Id: I13fb4d91c27b69ab431b8e3b19560be5b9a4f95c

quic/core/quic_stream.cc

diff --git a/quic/core/quic_stream.cc b/quic/core/quic_stream.cc
index 668dbdb..b0699d4 100644
--- a/quic/core/quic_stream.cc
+++ b/quic/core/quic_stream.cc
@@ -657,16 +657,6 @@
          (fin && fin_outstanding_);
 }
 
-QuicConsumedData QuicStream::WritevDataInner(size_t write_length,
-                                             QuicStreamOffset offset,
-                                             bool fin) {
-  StreamSendingState state = fin ? FIN : NO_FIN;
-  if (fin && add_random_padding_after_fin_) {
-    state = FIN_AND_PADDING;
-  }
-  return session()->WritevData(this, id(), write_length, offset, state);
-}
-
 void QuicStream::CloseReadSide() {
   if (read_side_closed_) {
     return;
@@ -974,8 +964,13 @@
   if (session_->session_decides_what_to_write()) {
     session_->SetTransmissionType(NOT_RETRANSMISSION);
   }
-  QuicConsumedData consumed_data =
-      WritevDataInner(write_length, stream_bytes_written(), fin);
+
+  StreamSendingState state = fin ? FIN : NO_FIN;
+  if (fin && add_random_padding_after_fin_) {
+    state = FIN_AND_PADDING;
+  }
+  QuicConsumedData consumed_data = session_->WritevData(
+      this, id(), write_length, stream_bytes_written(), state);
 
   OnStreamDataConsumed(consumed_data.bytes_consumed);