Implemented BluetoothAdapterMac::AddDevices().

device/bluetooth/bluetooth_device.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 DEVICE_BLUETOOTH_BLUETOOTH_DEVICE_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_DEVICE_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 224 matching lines @@
   // Indicates whether the device is currently pairing and expecting
   // confirmation of a displayed passkey.
   virtual bool ExpectingConfirmation() const = 0;
 
   // SocketCallback is used by ConnectToService to return a BluetoothSocket to
   // the caller, or NULL if there was an error.  The socket will remain open
   // until the last reference to the returned BluetoothSocket is released.
   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|
   // to indicate what display, and in response should make method calls
   // back to the device object. Not all devices require user responses
   // during pairing, so it is normal for |pairing_delegate| to receive no
   // calls. To explicitly force a low-security connection without bonding,
   // pass NULL, though this is ignored if the device is already paired.
   //
   // If the request fails, |error_callback| will be called; otherwise,
@@ skipping 62 matching lines @@
       const base::Closure& callback,
       const ErrorCallback& error_callback) = 0;
 
   // Clears the Out Of Band pairing data for this device.  Exactly one of
   // |callback| or |error_callback| will be run.
   virtual void ClearOutOfBandPairingData(
       const base::Closure& callback,
       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
   uint32 bluetooth_class_;
 
   // The name of the device, as supplied by the remote device.
   std::string name_;
 
   // The Bluetooth address of the device.
   std::string address_;
@@ skipping 16 matching lines @@
 
  private:
   // Returns a localized string containing the device's bluetooth address and
   // a device type for display when |name_| is empty.
   string16 GetAddressWithLocalizedDeviceTypeName() const;
 };
 
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_DEVICE_H_