Remove RemoteServerDisconnect() from web_bluetooth.mojom

content/browser/bluetooth/web_bluetooth_service_impl.h

Index: content/browser/bluetooth/web_bluetooth_service_impl.h
diff --git a/content/browser/bluetooth/web_bluetooth_service_impl.h b/content/browser/bluetooth/web_bluetooth_service_impl.h
index 6e5e164b1684872e6f03e1f6741d40132cc07654..de84ea550aebdf367d33757df0b82f7f55d2ad4d 100644
--- a/content/browser/bluetooth/web_bluetooth_service_impl.h
+++ b/content/browser/bluetooth/web_bluetooth_service_impl.h
@@ -104,7 +104,6 @@ class CONTENT_EXPORT WebBluetoothServiceImpl
       const WebBluetoothDeviceId& device_id,
       blink::mojom::WebBluetoothServerClientAssociatedPtrInfo client,
       const RemoteServerConnectCallback& callback) override;
-  void RemoteServerDisconnect(const WebBluetoothDeviceId& device_id) override;
   void RemoteServerGetPrimaryServices(
       const WebBluetoothDeviceId& device_id,
       blink::mojom::WebBluetoothGATTQueryQuantity quantity,
@@ -147,6 +146,11 @@ class CONTENT_EXPORT WebBluetoothServiceImpl
       const RequestDeviceCallback& callback,
       device::BluetoothAdapter* adapter);
 
+  // Called in the following two situations:
+  // 1. When a device is disconnected on the browser side.

[scheib, 2017/03/22 01:11:08]

Let's reword to be a bit clearer, something like:

Renderer has closed the pipe because it no longer needs a connection
to this device. Because:
1) Content script code has called disconnect()
2) The actual device disconnected, WebBluetoothServiceImpl informed the
renderer, and renderer reacted.

[juncai, 2017/03/27 20:44:46]

Done.

+  // 2. When the renderer intentionally wants to disconnect from a device.
+  void HandleServerClientError(const WebBluetoothDeviceId& device_id);

[scheib, 2017/03/22 01:11:08]

Perhaps: OnGATTServerClientDisconnection

[juncai, 2017/03/27 20:44:46]

Done.

+
   // Should only be run after the services have been discovered for
   // |device_address|.
   void RemoteServerGetPrimaryServicesImpl(