bluetooth: Move connect/disconnect to mojo

content/browser/bluetooth/bluetooth_connected_devices_map.h

+// Copyright 2016 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 CONTENT_BROWSER_BLUETOOTH_BLUETOOTH_CONNECTED_DEVICES_MAP_H_
+#define CONTENT_BROWSER_BLUETOOTH_BLUETOOTH_CONNECTED_DEVICES_MAP_H_
+
+#include <memory>
+#include <string>
+#include <unordered_map>
+
+#include "content/common/content_export.h"
+#include "url/origin.h"
+
+namespace device {
+class BluetoothGattConnection;
+}  // namespace device
+
+namespace content {
+
+class WebContents;
+class WebContentsImpl;
+
+// Holds information about connected devices and updates the WebContents
+// when new connections are made or connections closed. WebContents should

[Jeffrey Yasskin, 2016/05/03 01:15:57]

s/should/must/

[ortuno, 2016/05/03 16:11:53]

Done.

+// outlive this class.
+// This class does not keep track of the status of the connections. Owners of
+// this class should inform it when a connection is terminated so that the
+// connection removed from the appropriate maps.
+class CONTENT_EXPORT BluetoothConnectedDevicesMap final {

[Jeffrey Yasskin, 2016/05/03 01:15:57]

I think I'd name this "FrameConnectedBluetoothDevices" or similar, to emphasize
that this records the devices that are connected to a particular frame, and not
any wider scope.

[ortuno, 2016/05/03 16:11:53]

Done.

+ public:
+  explicit BluetoothConnectedDevicesMap(WebContents* web_contents);

[Jeffrey Yasskin, 2016/05/03 01:15:57]

If this argument flows directly into this->web_contents_impl_, you should have a
comment here that matches the one on web_contents_impl_.

[ortuno, 2016/05/03 16:11:53]

Done.

+  ~BluetoothConnectedDevicesMap();
+
+  // Returns true if the map holds a connection to |device_id|.
+  bool IsConnectedToDeviceWithId(const std::string& device_id);
+
+  // If a connection doesn't exist already for |device_id| adds a connection to

[Jeffrey Yasskin, 2016/05/03 01:15:57]

Add a comma before "adds".

[ortuno, 2016/05/03 16:11:53]

Done.

+  // the map and increases the WebContents count of connected devices.
+  void Insert(const std::string& device_id,
+              std::unique_ptr<device::BluetoothGattConnection> connection);
+
+  // Deletes the BluetoothGattConnection for |device_id| and decrements the
+  // WebContents count of connected devices if |device_id| had a connection.
+  void CloseConnectionToDeviceWithId(const std::string& device_id);
+
+  // Deletes the BluetoothGattConnection for |device_address| and decrements the
+  // WebContents count of connected devices if |device_address| had a
+  // connection. Returns the device_id of the device associated with the
+  // connection.
+  std::string CloseConnectionToDeviceWithAddress(
+      const std::string& device_address);
+
+ private:
+  // Increments the Connected Devices count of the frame's WebContents.
+  void IncrementDevicesConnectedCount();
+  // Decrements the Connected Devices count of the frame's WebContents.
+  void DecrementDevicesConnectedCount();
+
+  // WebContentsImpl that owns the WebBluetoothServiceImpl that owns this map.
+  WebContentsImpl* web_contents_impl_;
+
+  // Keeps the BluetoothGattConnection objects alive so that connections don't
+  // get closed..

[Jeffrey Yasskin, 2016/05/03 01:15:57]

s/.././

[ortuno, 2016/05/03 16:11:54]

Done.

+  std::unordered_map<std::string,
+                     std::unique_ptr<device::BluetoothGattConnection>>
+      device_id_to_connection_map_;
+
+  // Keeps track of which device addresses correspond to which ids.
+  std::unordered_map<std::string, std::string> device_address_to_id_map_;
+
+  DISALLOW_COPY_AND_ASSIGN(BluetoothConnectedDevicesMap);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_BLUETOOTH_BLUETOOTH_CONNECTED_DEVICES_MAP_H_