bluetooth: Remove disconnect when page hidden

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

 // Copyright 2014 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.
 
 #ifndef BluetoothDevice_h
 #define BluetoothDevice_h
 
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "core/dom/ActiveDOMObject.h"
-#include "core/page/PageLifecycleObserver.h"
 #include "modules/EventTargetModules.h"
 #include "modules/bluetooth/BluetoothAdvertisingData.h"
 #include "modules/bluetooth/BluetoothRemoteGATTServer.h"
 #include "platform/heap/Heap.h"
 #include "public/platform/modules/bluetooth/WebBluetoothDevice.h"
 #include "wtf/OwnPtr.h"
 #include "wtf/PassOwnPtr.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
 class BluetoothRemoteGATTServer;
 class ScriptPromise;
 class ScriptPromiseResolver;
 class ScriptState;
 
 // BluetoothDevice represents a physical bluetooth device in the DOM. See IDL.
 //
 // Callbacks providing WebBluetoothDevice objects are handled by
 // CallbackPromiseAdapter templatized with this class. See this class's
 // "Interface required by CallbackPromiseAdapter" section and the
 // CallbackPromiseAdapter class comments.
 class BluetoothDevice final
     : public RefCountedGarbageCollectedEventTargetWithInlineData<BluetoothDevice>
-    , public ActiveDOMObject
-    , public PageLifecycleObserver {
+    , public ActiveDOMObject {
     USING_PRE_FINALIZER(BluetoothDevice, dispose);
     DEFINE_WRAPPERTYPEINFO();
     REFCOUNTED_GARBAGE_COLLECTED_EVENT_TARGET(BluetoothDevice);
     USING_GARBAGE_COLLECTED_MIXIN(BluetoothDevice);
 public:
     BluetoothDevice(ExecutionContext*, PassOwnPtr<WebBluetoothDevice>);
 
     // Interface required by CallbackPromiseAdapter:
     using WebType = OwnPtr<WebBluetoothDevice>;
     static BluetoothDevice* take(ScriptPromiseResolver*, PassOwnPtr<WebBluetoothDevice>);
 
     // 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.
     // dispose() is called in this case.
     // 2. When the parent document gets detached e.g. reloading a page.
     // stop() is called in this case.
-    // 3. For now (https://crbug.com/579746), when the tab is no longer in the
-    // foreground e.g. change tabs.
-    // pageVisibilityChanged() is called in this case.
     // TODO(ortuno): Users should be able to turn on notifications for
     // events on navigator.bluetooth and still remain connected even if the
     // BluetoothDevice object is garbage collected.
-    // TODO(ortuno): Allow connections when the tab is in the background.
-    // This is a short term solution instead of implementing a tab indicator
-    // for bluetooth connections.
 
     // USING_PRE_FINALIZER interface.
     // Called before the object gets garbage collected.
     void dispose();
 
     // ActiveDOMObject interface.
     void stop() override;
 
-    // PageLifecycleObserver interface.
-    void pageVisibilityChanged() override;
-
     // If gatt is connected then disconnects and sets gatt.connected to false.
     // Returns true if gatt was disconnected.
     bool disconnectGATTIfConnected();
 
     // EventTarget methods:
     const AtomicString& interfaceName() const override;
     ExecutionContext* getExecutionContext() const override;
 
     // Interface required by Garbage Collection:
     DECLARE_VIRTUAL_TRACE();
@@ skipping 17 matching lines @@
 
 private:
     OwnPtr<WebBluetoothDevice> m_webDevice;
     Member<BluetoothAdvertisingData> m_adData;
     Member<BluetoothRemoteGATTServer> m_gatt;
 };
 
 } // namespace blink
 
 #endif // BluetoothDevice_h