Change to optimize QUIC's RetransmittableFrames::HasCryptoHandshake by

net/quic/quic_packet_creator.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_packet_creator.h"
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "net/quic/crypto/quic_random.h"
 #include "net/quic/quic_ack_notifier.h"
@@ skipping 484 matching lines @@
       queued_frames_.push_back(
           queued_retransmittable_frames_->AddNonStreamFrame(frame));
     }
   } else {
     queued_frames_.push_back(frame);
   }
   return true;
 }
 
 void QuicPacketCreator::MaybeAddPadding() {
+  if (queued_retransmittable_frames_.get() == NULL) {
+    return;
+  }
+  if (!queued_retransmittable_frames_->HasCryptoHandshake()) {
+    return;
+  }
   if (BytesFree() == 0) {
     // Don't pad full packets.
     return;
   }
-
-  // If any of the frames in the current packet are on the crypto stream
-  // then they contain handshake messagses, and we should pad them.
-  bool is_handshake = false;
-  for (size_t i = 0; i < queued_frames_.size(); ++i) {
-    if (queued_frames_[i].type == STREAM_FRAME &&
-        queued_frames_[i].stream_frame->stream_id == kCryptoStreamId) {
-      is_handshake = true;
-      break;
-    }
-  }
-  if (!is_handshake) {
-    return;
-  }
-
   QuicPaddingFrame padding;
   bool success = AddFrame(QuicFrame(&padding), false);
   DCHECK(success);
 }
 
 }  // namespace net