Revert of Implement BluetoothGattNotifySession::Stop on Android

device/bluetooth/bluez/bluetooth_remote_gatt_characteristic_bluez.cc

Index: device/bluetooth/bluez/bluetooth_remote_gatt_characteristic_bluez.cc
diff --git a/device/bluetooth/bluez/bluetooth_remote_gatt_characteristic_bluez.cc b/device/bluetooth/bluez/bluetooth_remote_gatt_characteristic_bluez.cc
index 8dbe316378d57836b857c6759fb4892ea006e73e..b1d6a0ef80fae257db6831d27233915019a03b52 100644
--- a/device/bluetooth/bluez/bluetooth_remote_gatt_characteristic_bluez.cc
+++ b/device/bluetooth/bluez/bluetooth_remote_gatt_characteristic_bluez.cc
@@ -14,9 +14,9 @@
 #include "dbus/property.h"
 #include "device/bluetooth/bluetooth_device.h"
 #include "device/bluetooth/bluetooth_gatt_characteristic.h"
-#include "device/bluetooth/bluetooth_gatt_notify_session.h"
 #include "device/bluetooth/bluetooth_gatt_service.h"
 #include "device/bluetooth/bluez/bluetooth_adapter_bluez.h"
+#include "device/bluetooth/bluez/bluetooth_gatt_notify_session_bluez.h"
 #include "device/bluetooth/bluez/bluetooth_remote_gatt_descriptor_bluez.h"
 #include "device/bluetooth/bluez/bluetooth_remote_gatt_service_bluez.h"
 #include "device/bluetooth/dbus/bluetooth_gatt_characteristic_client.h"
@@ -81,44 +81,6 @@
     pending_start_notify_calls_.pop();
     callbacks.second.Run(device::BluetoothRemoteGattService::GATT_ERROR_FAILED);
   }
-}
-
-void BluetoothRemoteGattCharacteristicBlueZ::StopNotifySession(
-    device::BluetoothGattNotifySession* session,
-    const base::Closure& callback) {
-  VLOG(1) << __func__;
-
-  if (num_notify_sessions_ > 1) {
-    DCHECK(!notify_call_pending_);
-    --num_notify_sessions_;
-    callback.Run();
-    return;
-  }
-
-  // Notifications may have stopped outside our control. If the characteristic
-  // is no longer notifying, return success.
-  if (!IsNotifying()) {
-    num_notify_sessions_ = 0;
-    callback.Run();
-    return;
-  }
-
-  if (notify_call_pending_ || num_notify_sessions_ == 0) {
-    callback.Run();
-    return;
-  }
-
-  DCHECK(num_notify_sessions_ == 1);
-  notify_call_pending_ = true;
-  bluez::BluezDBusManager::Get()
-      ->GetBluetoothGattCharacteristicClient()
-      ->StopNotify(
-          object_path(),
-          base::Bind(
-              &BluetoothRemoteGattCharacteristicBlueZ::OnStopNotifySuccess,
-              weak_ptr_factory_.GetWeakPtr(), callback),
-          base::Bind(&BluetoothRemoteGattCharacteristicBlueZ::OnStopNotifyError,
-                     weak_ptr_factory_.GetWeakPtr(), callback));
 }
 
 device::BluetoothUUID BluetoothRemoteGattCharacteristicBlueZ::GetUUID() const {
@@ -244,8 +206,9 @@
       DCHECK(service_->GetAdapter());
       DCHECK(service_->GetDevice());
       std::unique_ptr<device::BluetoothGattNotifySession> session(
-          new device::BluetoothGattNotifySession(
-              weak_ptr_factory_.GetWeakPtr()));
+          new BluetoothGattNotifySessionBlueZ(
+              service_->GetAdapter(), service_->GetDevice()->GetAddress(),
+              service_->GetIdentifier(), GetIdentifier(), object_path()));
       callback.Run(std::move(session));
       return;
     }
@@ -301,20 +264,41 @@
                               weak_ptr_factory_.GetWeakPtr(), error_callback));
 }
 
-void BluetoothRemoteGattCharacteristicBlueZ::SubscribeToNotifications(
-    device::BluetoothRemoteGattDescriptor* ccc_descriptor,
-    const base::Closure& callback,
-    const ErrorCallback& error_callback) {
-  // TODO(http://crbug.com/636275): Implement this method
-  NOTIMPLEMENTED();
-}
-
-void BluetoothRemoteGattCharacteristicBlueZ::UnsubscribeFromNotifications(
-    device::BluetoothRemoteGattDescriptor* ccc_descriptor,
-    const base::Closure& callback,
-    const ErrorCallback& error_callback) {
-  // TODO(http://crbug.com/636275): Implement this method
-  NOTIMPLEMENTED();
+void BluetoothRemoteGattCharacteristicBlueZ::RemoveNotifySession(
+    const base::Closure& callback) {
+  VLOG(1) << __func__;
+
+  if (num_notify_sessions_ > 1) {
+    DCHECK(!notify_call_pending_);
+    --num_notify_sessions_;
+    callback.Run();
+    return;
+  }
+
+  // Notifications may have stopped outside our control. If the characteristic
+  // is no longer notifying, return success.
+  if (!IsNotifying()) {
+    num_notify_sessions_ = 0;
+    callback.Run();
+    return;
+  }
+
+  if (notify_call_pending_ || num_notify_sessions_ == 0) {
+    callback.Run();
+    return;
+  }
+
+  DCHECK(num_notify_sessions_ == 1);
+  notify_call_pending_ = true;
+  bluez::BluezDBusManager::Get()
+      ->GetBluetoothGattCharacteristicClient()
+      ->StopNotify(
+          object_path(),
+          base::Bind(
+              &BluetoothRemoteGattCharacteristicBlueZ::OnStopNotifySuccess,
+              weak_ptr_factory_.GetWeakPtr(), callback),
+          base::Bind(&BluetoothRemoteGattCharacteristicBlueZ::OnStopNotifyError,
+                     weak_ptr_factory_.GetWeakPtr(), callback));
 }
 
 void BluetoothRemoteGattCharacteristicBlueZ::GattDescriptorAdded(
@@ -410,7 +394,9 @@
   DCHECK(service_);
   DCHECK(service_->GetDevice());
   std::unique_ptr<device::BluetoothGattNotifySession> session(
-      new device::BluetoothGattNotifySession(weak_ptr_factory_.GetWeakPtr()));
+      new BluetoothGattNotifySessionBlueZ(
+          service_->GetAdapter(), service_->GetDevice()->GetAddress(),
+          service_->GetIdentifier(), GetIdentifier(), object_path()));
   callback.Run(std::move(session));
 
   ProcessStartNotifyQueue();