Remove max_packet_length from QuicPacketGenerator, because it is no longer necessary with FEC gone.…

net/quic/quic_packet_generator.h

 // 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.
 //
 // Responsible for generating packets on behalf of a QuicConnection.
 // Packets are serialized just-in-time.  Control frames are queued.
 // Ack and Feedback frames will be requested from the Connection
 // just-in-time.  When a packet needs to be sent, the Generator
 // will serialize a packet and pass it to QuicConnection::SendOrQueuePacket()
 //
@@ skipping 130 matching lines @@
   // Sets the encrypter to use for the encryption level.
   void SetEncrypter(EncryptionLevel level, QuicEncrypter* encrypter);
 
   // Sets the encryption level that will be applied to new packets.
   void set_encryption_level(EncryptionLevel level);
 
   // packet number of the last created packet, or 0 if no packets have been
   // created.
   QuicPacketNumber packet_number() const;
 
-  // Returns the maximum packet length.  Note that this is the long-term maximum
-  // packet length, and it may not be the maximum length of the current packet,
-  // if the generator is in the middle of the packet (in batch mode).
-  QuicByteCount GetMaxPacketLength() const;
-  // Returns the maximum length current packet can actually have.
+  // Returns the maximum length a current packet can actually have.
   QuicByteCount GetCurrentMaxPacketLength() const;
 
-  // Set maximum packet length at the next opportunity.
+  // Set maximum packet length in the creator immediately.  May not be called
+  // when there are frames queued in the creator.
   void SetMaxPacketLength(QuicByteCount length);
 
   // Sets |path_id| to be the path on which next packet is generated.
   void SetCurrentPath(QuicPathId path_id,
                       QuicPacketNumber least_packet_awaited_by_peer,
                       QuicPacketCount max_packets_in_flight);
 
   void set_debug_delegate(QuicPacketCreator::DebugDelegate* debug_delegate) {
     packet_creator_.set_debug_delegate(debug_delegate);
   }
@@ skipping 25 matching lines @@
   // Flags to indicate the need for just-in-time construction of a frame.
   bool should_send_ack_;
   bool should_send_stop_waiting_;
   // If we put a non-retransmittable frame in this packet, then we have to hold
   // a reference to it until we flush (and serialize it). Retransmittable frames
   // are referenced elsewhere so that they can later be (optionally)
   // retransmitted.
   QuicAckFrame pending_ack_frame_;
   QuicStopWaitingFrame pending_stop_waiting_frame_;
 
-  // Stores the maximum packet size we are allowed to send.  This might not be
-  // the maximum size we are actually using now, if we are in the middle of the
-  // packet.
-  QuicByteCount max_packet_length_;
-
   DISALLOW_COPY_AND_ASSIGN(QuicPacketGenerator);
 };
 
 }  // namespace net
 
 #endif  // NET_QUIC_QUIC_PACKET_GENERATOR_H_