chrome.bluetooth API improvements.

device/bluetooth/bluetooth_socket.h

Index: device/bluetooth/bluetooth_socket.h
diff --git a/device/bluetooth/bluetooth_socket.h b/device/bluetooth/bluetooth_socket.h
index 6ae349e9ac92c2f1480f4360b6546287e73cef41..6f23caa458cacbe1301cc75c5398a2f79588b31c 100644
--- a/device/bluetooth/bluetooth_socket.h
+++ b/device/bluetooth/bluetooth_socket.h
@@ -7,12 +7,14 @@
 
 #include <string>
 
+#include "base/callback.h"
 #include "base/memory/ref_counted.h"
 
 namespace net {
 
 class DrainableIOBuffer;
 class GrowableIOBuffer;
+class IOBuffer;
 
 }  // namespace net
 
@@ -21,24 +23,50 @@ namespace device {
 // BluetoothSocket represents a socket to a specific service on a
 // BluetoothDevice.  BluetoothSocket objects are ref counted and may outlive
 // both the BluetoothDevice and BluetoothAdapter that were involved in their
-// creation.
-class BluetoothSocket : public base::RefCounted<BluetoothSocket> {
+// creation.  In terms of threading, platform specific implementation may differ
+// slightly, but platform independent callers must guarantee calling various
+// methods on the same thread for a given instance, and in a context that allows
+// IO.
+class BluetoothSocket : public base::RefCountedThreadSafe<BluetoothSocket> {
  public:
-  // Receives data from the socket and stores it in |buffer|. It returns whether
-  // the reception has been successful. If it fails, the caller can get the
-  // error message through |GetLastErrorMessage()|.
-  virtual bool Receive(net::GrowableIOBuffer* buffer) = 0;
+  enum ErrorReason { kSystemError, kIOPending, kDisconnected };
 
-  // Sends |buffer| to the socket. It returns whether the sending has been
-  // successful. If it fails, the caller can get the error message through
-  // |GetLastErrorMessage()|.
-  virtual bool Send(net::DrainableIOBuffer* buffer) = 0;
+  typedef base::Callback<void(int)> SendCompletionCallback;
+  typedef base::Callback<void(int, scoped_refptr<net::IOBuffer> io_buffer)>
+      ReceiveCompletionCallback;
+  typedef base::Callback<void(const std::string& error_message)>
+      ErrorCompletionCallback;
+  typedef base::Callback<void(ErrorReason, const std::string& error_message)>
+      ReceiveErrorCompletionCallback;
 
-  virtual std::string GetLastErrorMessage() const = 0;
+  // Connected to the peer device and calls |success_callback| when the
+  // connection has been established successfully. If an error occurs, calls
+  // |error_callback| with a system error message.
+  virtual void Connect(const base::Closure& success_callback,
+                       const ErrorCompletionCallback& error_callback) = 0;
+
+  // Disconnects a connected socket and calls |callback| upon completion. There
+  // is no failure case, as this is a best effort operation.
+  virtual void Disconnect(const base::Closure& callback) = 0;
+
+  // Receives data from the socket and calls |success_callback| when data is
+  // available. If an error occurs, calls |error_callback| with a reason and a
+  // message.
+  virtual void Receive(
+      int count,
+      const ReceiveCompletionCallback& success_callback,
+      const ReceiveErrorCompletionCallback& error_callback) = 0;
+
+  // Sends |buffer| to the socket and calls |success_callback| when data has
+  // been successfully sent. If an error occurs, calls |error_callback| with a
+  // reason and a message.
+  virtual void Send(scoped_refptr<net::DrainableIOBuffer> buffer,
+                    const SendCompletionCallback& success_callback,
+                    const ErrorCompletionCallback& error_callback) = 0;
 
  protected:
-  friend class base::RefCounted<BluetoothSocket>;
-  virtual ~BluetoothSocket() {}
+  friend class base::RefCountedThreadSafe<BluetoothSocket>;
+  virtual ~BluetoothSocket();
 };
 
 }  // namespace device