bluetooth: Make public BluetoothDevice::GetName method.

device/bluetooth/bluetooth_low_energy_win.cc

Index: device/bluetooth/bluetooth_low_energy_win.cc
diff --git a/device/bluetooth/bluetooth_low_energy_win.cc b/device/bluetooth/bluetooth_low_energy_win.cc
index b96277c18304793884ede9fdf120e52dde686a4e..51485992e62969cad63f822a1c99a1b7e8bf6167 100644
--- a/device/bluetooth/bluetooth_low_energy_win.cc
+++ b/device/bluetooth/bluetooth_low_energy_win.cc
@@ -467,14 +467,8 @@ bool CollectBluetoothLowEnergyDeviceInfo(
           device_info_handle, &device_info_data, result, error)) {
     return false;
   }
-  if (!CollectBluetoothLowEnergyDeviceFriendlyName(
-          device_info_handle, &device_info_data, result, error)) {
-    // Only fail if not the GATT service device interface, which doesn't have a
-    // friendly name.
-    if (device_interface_data->InterfaceClassGuid !=

[ortuno, 2016/07/14 21:11:41]

Hmm this would mean that we no longer fail if we can't get the device's name
which I guess is OK, but I wish we knew if we are always expected to be able to
get a name...

[scheib, 2016/07/16 00:21:36]

Acknowledged, would be great to know if we'd always get one or not. But since we
can tolerate not receiving one I don't see a need for the awkward old check.

-        GUID_BLUETOOTH_GATT_SERVICE_DEVICE_INTERFACE)
-      return false;
-  }
+  CollectBluetoothLowEnergyDeviceFriendlyName(device_info_handle,
+                                              &device_info_data, result, error);
   if (!CollectBluetoothLowEnergyDeviceAddress(
           device_info_handle, &device_info_data, result, error)) {
     return false;