bluetooth: Invalidate services upon disconnection

third_party/WebKit/Source/modules/bluetooth/BluetoothRemoteGATTServer.cpp

 // 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 "modules/bluetooth/BluetoothRemoteGATTServer.h"
 
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ScriptPromise.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "core/dom/DOMException.h"
@@ skipping 82 matching lines @@
   ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
   ScriptPromise promise = resolver->promise();
   webbluetooth->connect(device()->id(), device(),
                         new ConnectCallback(device(), resolver));
   return promise;
 }
 
 void BluetoothRemoteGATTServer::disconnect(ScriptState* scriptState) {
   if (!m_connected)
     return;
-  m_connected = false;
-  ClearActiveAlgorithms();
+  device()->CleanupDisconnectedDeviceAndFireEvent();

[ortuno, 2016/11/04 03:09:48]

Note how we only clear the active algorithms map once: in
CleanupDisconectedDeviceAndFireEvent. The spec clears it twice once in
disconnect()[1] and once in Clean up disconnected device[2]. I believe that's
not necessary so I opened an issue about it[3].

[1]
https://webbluetoothcg.github.io/web-bluetooth/#dom-bluetoothremotegattserver...
[2]
https://webbluetoothcg.github.io/web-bluetooth/#clean-up-the-disconnected-device
[3] https://github.com/WebBluetoothCG/web-bluetooth/issues/333

   WebBluetooth* webbluetooth =
       BluetoothSupplement::fromScriptState(scriptState);
   webbluetooth->disconnect(device()->id());
-  device()->dispatchEvent(
-      Event::createBubble(EventTypeNames::gattserverdisconnected));
 }
 
 // Class that allows us to resolve the promise with a single service or
 // with a vector owning the services.
 class GetPrimaryServicesCallback
     : public WebBluetoothGetPrimaryServicesCallbacks {
  public:
   GetPrimaryServicesCallback(
       BluetoothDevice* device,
       mojom::blink::WebBluetoothGATTQueryQuantity quantity,
@@ skipping 107 matching lines @@
   WebBluetooth* webbluetooth =
       BluetoothSupplement::fromScriptState(scriptState);
   webbluetooth->getPrimaryServices(
       device()->id(), static_cast<int32_t>(quantity), servicesUUID,
       new GetPrimaryServicesCallback(device(), quantity, resolver));
 
   return promise;
 }
 
 }  // namespace blink