[WebSocket] Incoming close frame should not overtake data frames

content/browser/renderer_host/websocket_host.cc

 // Copyright 2013 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 "content/browser/renderer_host/websocket_host.h"
 
 #include <inttypes.h>
 
 #include <utility>
 
@@ skipping 500 matching lines @@
            << " routing_id=" << routing_id_ << " quota=" << quota;
 
   if (!channel_) {
     // WebSocketChannel is not yet created due to the delay introduced by
     // per-renderer WebSocket throttling.
     // SendFlowControl() is called after WebSocketChannel is created.
     pending_flow_control_quota_ += quota;
     return;
   }
 
-  channel_->SendFlowControl(quota);
+  ignore_result(channel_->SendFlowControl(quota));
 }
 
 void WebSocketHost::OnDropChannel(bool was_clean,
                                   uint16_t code,
                                   const std::string& reason) {
   DVLOG(3) << "WebSocketHost::OnDropChannel"
            << " routing_id=" << routing_id_ << " was_clean=" << was_clean
            << " code=" << code << " reason=\"" << reason << "\"";
 
   if (!channel_) {
     // WebSocketChannel is not yet created due to the delay introduced by
     // per-renderer WebSocket throttling.
     WebSocketDispatcherHost::WebSocketHostState result =
         dispatcher_->DoDropChannel(routing_id_, false,
                                    net::kWebSocketErrorAbnormalClosure, "");
     DCHECK_EQ(WebSocketDispatcherHost::WEBSOCKET_HOST_DELETED, result);
     return;
   }
 
   blob_sender_.reset();
   // TODO(yhirano): Handle |was_clean| appropriately.
-  channel_->StartClosingHandshake(code, reason);
+  ignore_result(channel_->StartClosingHandshake(code, reason));
 }
 
 void WebSocketHost::BlobSendComplete(int result) {
   DVLOG(3) << "WebSocketHost::BlobSendComplete"
            << " routing_id=" << routing_id_
            << " result=" << net::ErrorToString(result);
 
   // All paths through this method must reset blob_sender_, so take ownership
   // at the beginning.
   std::unique_ptr<WebSocketBlobSender> blob_sender(std::move(blob_sender_));
@@ skipping 20 matching lines @@
     default:
       ignore_result(dispatcher_->NotifyFailure(
           routing_id_,
           "Failed to load Blob: error code = " + net::ErrorToString(result)));
       // |this| is destroyed here.
       return;
   }
 }
 
 }  // namespace content