bluetooth: macOS: BluetoothLowEnergyDeviceMac::IsConnectable() not implemented

device/bluetooth/bluetooth_low_energy_device_mac.mm

 // Copyright 2015 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.
 
 #include "device/bluetooth/bluetooth_low_energy_device_mac.h"
 
 #import <CoreFoundation/CoreFoundation.h>
 #include <stddef.h>
 
 #include "base/mac/mac_util.h"
@@ skipping 82 matching lines @@
 
 bool BluetoothLowEnergyDeviceMac::IsConnected() const {
   return IsGattConnected();
 }
 
 bool BluetoothLowEnergyDeviceMac::IsGattConnected() const {
   return ([peripheral_ state] == CBPeripheralStateConnected);
 }
 
 bool BluetoothLowEnergyDeviceMac::IsConnectable() const {
-  return connectable_;
+  return true;

[ortuno, 2017/05/02 01:32:16]

Do we need this to return true? Otherwise just add a NOTIMPLEMENTED and return
false. Some devices are not connectable so we shouldn't always return true.

[jlebel, 2017/05/03 22:44:18]

Done.

 }
 
 bool BluetoothLowEnergyDeviceMac::IsConnecting() const {
   return ([peripheral_ state] == CBPeripheralStateConnecting);
 }
 
 bool BluetoothLowEnergyDeviceMac::ExpectingPinCode() const {
   return false;
 }
 
@@ skipping 352 matching lines @@
 std::ostream& operator<<(std::ostream& out,
                          const BluetoothLowEnergyDeviceMac& device) {
   // TODO(crbug.com/703878): Should use
   // BluetoothLowEnergyDeviceMac::GetNameForDisplay() instead.
   base::Optional<std::string> name = device.GetName();
   const char* name_cstr = name ? name->c_str() : "";
   return out << "<BluetoothLowEnergyDeviceMac " << device.GetAddress() << "/"
              << &device << ", \"" << name_cstr << "\">";
 }
 }  // namespace device