Implemented BluetoothAdapterMac::AddDevices().

device/bluetooth/bluetooth_device.h

Index: device/bluetooth/bluetooth_device.h
diff --git a/device/bluetooth/bluetooth_device.h b/device/bluetooth/bluetooth_device.h
index b7dc9e2a2da713108c88acb595c6d5a7909175a2..87562ff6a05fcf42979329932964cfe4214330b9 100644
--- a/device/bluetooth/bluetooth_device.h
+++ b/device/bluetooth/bluetooth_device.h
@@ -242,6 +242,9 @@ class BluetoothDevice {
   typedef base::Callback<void(scoped_refptr<BluetoothSocket>)>
       SocketCallback;
 
+  // Sets whether the device is visible to the owning adapter to |visible|.
+  virtual void SetVisible(bool visible);
+

[keybuk, 2013/03/21 18:03:34]

I was planning to remove this method from Chrome OS (it's not relevant in LE)
... any reason for prompting it?

[youngki, 2013/03/21 18:43:28]

Okay I removed SetVisible. I wasn't aware of your plan to remove it. :) I
thought since visible_ belongs to BluetoothDevice and all the subclasses are
setting it, I thought SetVisible() should rather belong to this class instead.

[keybuk, 2013/03/21 18:46:25]

Basically Visible is meaningless ;) it's used in ChromeOS to avoid accidentally
deleting paired devices found during discovery

It's an API that shouldn't be part of BluetoothDevice at all - since to the UI
it's just the opposite of "paired"

[youngki, 2013/03/21 20:07:20]

Agreed.

   // Initiates a connection to the device, pairing first if necessary.
   //
   // Method calls will be made on the supplied object |pairing_delegate|
@@ -324,7 +327,11 @@ class BluetoothDevice {
       const ErrorCallback& error_callback) = 0;
 
  protected:
-  BluetoothDevice();
+  BluetoothDevice(const std::string& name,
+                  const std::string& address,
+                  uint32 bluetooth_class,
+                  bool connected,
+                  bool bonded);
 
   // The Bluetooth class of the device, a bitmask that may be decoded using
   // https://www.bluetooth.org/Technical/AssignedNumbers/baseband.htm