bluetooth: Invalidate services upon disconnection

third_party/WebKit/Source/modules/bluetooth/BluetoothDevice.h

Index: third_party/WebKit/Source/modules/bluetooth/BluetoothDevice.h
diff --git a/third_party/WebKit/Source/modules/bluetooth/BluetoothDevice.h b/third_party/WebKit/Source/modules/bluetooth/BluetoothDevice.h
index 87c9ab816561293dff707ebf5c347e32e69e7575..26066fe5d71f1a746782d178d95c15e9b5fef343 100644
--- a/third_party/WebKit/Source/modules/bluetooth/BluetoothDevice.h
+++ b/third_party/WebKit/Source/modules/bluetooth/BluetoothDevice.h
@@ -48,6 +48,7 @@ class BluetoothDevice final : public EventTargetWithInlineData,
 
   BluetoothRemoteGATTService* getOrCreateBluetoothRemoteGATTService(
       std::unique_ptr<WebBluetoothRemoteGATTService>);
+  bool isValidService(const String& serviceInstanceId);
 
   // We should disconnect from the device in all of the following cases:
   // 1. When the object gets GarbageCollected e.g. it went out of scope.
@@ -67,8 +68,14 @@ class BluetoothDevice final : public EventTargetWithInlineData,
 
   // If gatt is connected then disconnects and sets gatt.connected to false.
   // Returns true if gatt was disconnected.
+  // Should only be used when the object is being garbage collected or the

[scheib, 2016/11/04 22:53:50]

Explain why it is only to be used in those cases, and that
CleanupDisconnectedDeviceAndFireEvent should be used for other reasons.

[ortuno, 2016/11/06 22:44:12]

Done.

+  // context is being destroyed.
   bool disconnectGATTIfConnected();
 
+  // Performs necessary cleanup when a device disconnects and fires
+  // gattserverdisconnected event.
+  void CleanupDisconnectedDeviceAndFireEvent();
+
   // EventTarget methods:
   const AtomicString& interfaceName() const override;
   ExecutionContext* getExecutionContext() const override;