Fix error handling for message parse errors that occur during connection setup.

chrome/browser/extensions/api/cast_channel/cast_socket.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_CAST_CHANNEL_CAST_SOCKET_H_
 #define CHROME_BROWSER_EXTENSIONS_API_CAST_CHANNEL_CAST_SOCKET_H_
 
 #include <queue>
 #include <string>
 
@@ skipping 105 matching lines @@
   // READY_STATE_CLOSED.
   // It is fine to delete the CastSocket object in |callback|.
   virtual void Close(const net::CompletionCallback& callback);
 
  protected:
   // Message header struct. If fields are added, be sure to update
   // header_size().  Protected to allow use of *_size() methods in unit tests.
   struct MessageHeader {
     MessageHeader();
     // Sets the message size.
-    void SetMessageSize(size_t message_size);
+    bool SetMessageSize(size_t message_size);

[Wez, 2014/07/24 00:55:55]

This no longer needs the Boolean return. :)

     // Prepends this header to |str|.
     void PrependToString(std::string* str);
     // Reads |header| from the beginning of |buffer|.
     static void ReadFromIOBuffer(net::GrowableIOBuffer* buffer,
                                  MessageHeader* header);
     // Size (in bytes) of the message header.
     static uint32 header_size() { return sizeof(uint32); }
 
     // Maximum size (in bytes) of a message payload on the wire (does not
     // include header).
@@ skipping 210 matching lines @@
   FRIEND_TEST_ALL_PREFIXES(CastSocketTest, TestReadMany);
   FRIEND_TEST_ALL_PREFIXES(CastSocketTest, TestWriteErrorLargeMessage);
   DISALLOW_COPY_AND_ASSIGN(CastSocket);
 };
 
 }  // namespace cast_channel
 }  // namespace api
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_CAST_CHANNEL_CAST_SOCKET_H_