Allow socket API to send binary data.

chrome/browser/extensions/api/socket/socket_api.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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_SOCKET_SOCKET_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_SOCKET_SOCKET_API_H_
 #pragma once
 
+#include "base/memory/ref_counted.h"
+#include "base/values.h"

[asargent_no_longer_on_chrome, 2012/04/16 22:51:13]

nit: do you need to include values.h here? I don't see any *Value types being
used except for in the .cc file.

[miket_OOO, 2012/04/16 23:15:53]

Right you are!

 #include "chrome/browser/extensions/api/api_function.h"
+#include "net/base/io_buffer.h"
 
 #include <string>
 
 namespace extensions {
 
 class APIResourceController;
 class APIResourceEventNotifier;
 
 extern const char kBytesWrittenKey[];
 extern const char kSocketIdKey[];
@@ skipping 72 matching lines @@
   virtual void Work() OVERRIDE;
   virtual bool Respond() OVERRIDE;
 
  private:
   int socket_id_;
 
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.read")
 };
 
 class SocketWriteFunction : public AsyncIOAPIFunction {
+ public:
+  SocketWriteFunction();
+  virtual ~SocketWriteFunction();
+
  protected:
   virtual bool Prepare() OVERRIDE;
   virtual void Work() OVERRIDE;
   virtual bool Respond() OVERRIDE;
 
  private:
   int socket_id_;
-  std::string message_;
+  scoped_refptr<net::IOBufferWithSize> io_buffer_;
 
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.socket.write")
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_SOCKET_SOCKET_API_H_