Prototype for copresenceSockets.

components/copresence_sockets/transports/bluetooth/copresence_socket_bluetooth.h

Index: components/copresence_sockets/transports/bluetooth/copresence_socket_bluetooth.h
diff --git a/components/copresence_sockets/transports/bluetooth/copresence_socket_bluetooth.h b/components/copresence_sockets/transports/bluetooth/copresence_socket_bluetooth.h
new file mode 100644
index 0000000000000000000000000000000000000000..783a164cb8b15a9ce277cdee9fdfd640a6feea05
--- /dev/null
+++ b/components/copresence_sockets/transports/bluetooth/copresence_socket_bluetooth.h
@@ -0,0 +1,51 @@
+// Copyright 2014 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 COMPONENTS_COPRESENCE_SOCKETS_TRANSPORTS_COPRESENCE_SOCKET_BLUETOOTH_H_
+#define COMPONENTS_COPRESENCE_SOCKETS_TRANSPORTS_COPRESENCE_SOCKET_BLUETOOTH_H_
+
+#include <string>
+
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/weak_ptr.h"
+#include "components/copresence_sockets/public/copresence_socket.h"
+#include "device/bluetooth/bluetooth_socket.h"
+
+namespace net {
+class IOBuffer;
+}
+
+namespace copresence_sockets {
+
+// A CopresenceSocketBluetooth is the Bluetooth implementation of a
+// CopresenceSocket. This is currently a thin wrapper around BluetoothSocket.
+class CopresenceSocketBluetooth
+    : public CopresenceSocket,
+      public base::SupportsWeakPtr<CopresenceSocketBluetooth> {

[Ryan Sleevi, 2014/10/01 19:47:25]

DESIGN: DO NOT expose SupportsWeakPtr on new objects.

It is a design flaw to expose the WeakPtr semantics of an object to external
consumers. This is a NOT LGTM until this can be resolved.

[rkc, 2014/10/01 22:38:48]

Done.

+ public:
+  CopresenceSocketBluetooth(scoped_refptr<device::BluetoothSocket> socket);

[Ryan Sleevi, 2014/10/01 19:47:26]

STYLE: const-ref

[rkc, 2014/10/01 22:38:48]

Done.

+  virtual ~CopresenceSocketBluetooth();
+
+  // CopresenceSocket overrides:
+  virtual bool Send(const std::string& data) override;
+  virtual void Receive(ReceiveCallback callback) override;
+
+ private:
+  void OnSendComplete(int status);
+  void OnSendError(const std::string& message);
+
+  void OnReceive(int size, scoped_refptr<net::IOBuffer> io_buffer);

[Ryan Sleevi, 2014/10/01 19:47:25]

STYLE: const-ref

[rkc, 2014/10/01 22:38:48]

As mentioned elsewhere, this is required to be this way since it is a callback
to BT code.

+  void OnReceiveError(device::BluetoothSocket::ErrorReason reason,
+                      const std::string& message);
+
+  scoped_refptr<device::BluetoothSocket> socket_;
+  ReceiveCallback receive_callback_;
+
+  DISALLOW_COPY_AND_ASSIGN(CopresenceSocketBluetooth);
+};
+
+}  // namespace copresence_sockets
+
+#endif  // COMPONENTS_COPRESENCE_SOCKETS_TRANSPORTS_COPRESENCE_SOCKET_BLUETOOTH_H_