relnote: Change QuicConnectionVisitorInterface::OnStreamFrames to

net/quic/quic_connection.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/quic/quic_connection.h"
 
 #include <string.h>
 #include <sys/types.h>
 
 #include <algorithm>
@@ skipping 944 matching lines @@
   // processing stream frames, since the processing may result in a response
   // packet with a bundled ack.
   if (last_packet_revived_) {
     received_packet_manager_.RecordPacketRevived(
         last_header_.packet_sequence_number);
   } else {
     received_packet_manager_.RecordPacketReceived(
         last_size_, last_header_, time_of_last_received_packet_);
   }
 
-  if (!last_stream_frames_.empty()) {
-    visitor_->OnStreamFrames(last_stream_frames_);
+  for (const QuicStreamFrame& frame : last_stream_frames_) {
+    visitor_->OnStreamFrame(frame);
+    stats_.stream_bytes_received += frame.data.size();
     if (!connected_) {
       return;
     }
   }
 
-  for (const QuicStreamFrame& stream_frame : last_stream_frames_) {
-    stats_.stream_bytes_received += stream_frame.data.size();
-  }
   // Process window updates, blocked, stream resets, acks, then congestion
   // feedback.
-  if (!last_window_update_frames_.empty()) {
-    visitor_->OnWindowUpdateFrames(last_window_update_frames_);
+  for (const QuicWindowUpdateFrame& frame : last_window_update_frames_) {
+    visitor_->OnWindowUpdateFrame(frame);
     if (!connected_) {
       return;
     }
   }
-  if (!last_blocked_frames_.empty()) {
-    visitor_->OnBlockedFrames(last_blocked_frames_);
+  for (const QuicBlockedFrame& frame : last_blocked_frames_) {
+    visitor_->OnBlockedFrame(frame);
     if (!connected_) {
       return;
     }
   }
-  for (size_t i = 0; i < last_goaway_frames_.size(); ++i) {
-    visitor_->OnGoAway(last_goaway_frames_[i]);
+  for (const QuicGoAwayFrame& frame : last_goaway_frames_) {
+    visitor_->OnGoAway(frame);
     if (!connected_) {
       return;
     }
   }
-  for (size_t i = 0; i < last_rst_frames_.size(); ++i) {
-    visitor_->OnRstStream(last_rst_frames_[i]);
+  for (const QuicRstStreamFrame& frame : last_rst_frames_) {
+    visitor_->OnRstStream(frame);
     if (!connected_) {
       return;
     }
   }
-  for (size_t i = 0; i < last_ack_frames_.size(); ++i) {
-    ProcessAckFrame(last_ack_frames_[i]);
+  for (const QuicAckFrame& frame : last_ack_frames_) {
+    ProcessAckFrame(frame);
     if (!connected_) {
       return;
     }
   }
-  for (size_t i = 0; i < last_stop_waiting_frames_.size(); ++i) {
-    ProcessStopWaitingFrame(last_stop_waiting_frames_[i]);
+  for (const QuicStopWaitingFrame& frame : last_stop_waiting_frames_) {
+    ProcessStopWaitingFrame(frame);
     if (!connected_) {
       return;
     }
   }
   if (!last_close_frames_.empty()) {
     CloseConnection(last_close_frames_[0].error_code, true);
     DCHECK(!connected_);
     return;
   }
 
@@ skipping 1301 matching lines @@
       sequence_number_of_last_sent_packet_ + packets_between_mtu_probes_ + 1;
   ++mtu_probe_count_;
 
   DVLOG(2) << "Sending a path MTU discovery packet #" << mtu_probe_count_;
   SendMtuDiscoveryPacket(mtu_discovery_target_);
 
   DCHECK(!mtu_discovery_alarm_->IsSet());
 }
 
 }  // namespace net