Server push cancellation: clean up the code to make sure shared code is in sync with internal code.

net/quic/core/quic_client_promised_info.cc

 // Copyright (c) 2016 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/core/quic_client_promised_info.h"
 
 #include "base/logging.h"
 #include "net/quic/core/spdy_utils.h"
 
 using net::SpdyHeaderBlock;
@@ skipping 95 matching lines @@
 QuicAsyncStatus QuicClientPromisedInfo::HandleClientRequest(
     const SpdyHeaderBlock& request_headers,
     QuicClientPushPromiseIndex::Delegate* delegate) {
   if (session_->IsClosedStream(id_)) {
     // There was a RST on the response stream.
     session_->DeletePromised(this);
     return QUIC_FAILURE;
   }
 
   if (is_validating()) {
+    // The push promise has already been matched to another request though
+    // pending for validation. Returns QUIC_FAILURE to the caller as it couldn't
+    // match a new request any more. This will not affect the validation of the
+    // other request.
     return QUIC_FAILURE;
   }
 
   client_request_delegate_ = delegate;
   client_request_headers_.reset(new SpdyHeaderBlock(request_headers.Clone()));
   if (!response_headers_) {
     return QUIC_PENDING;
   }
   return FinalValidation();
 }
 
 void QuicClientPromisedInfo::Cancel() {
   // Don't fire OnRendezvousResult() for client initiated cancel.
   client_request_delegate_ = nullptr;
   Reset(QUIC_STREAM_CANCELLED);
 }
 
 }  // namespace net