Implement BluetoothGattNotifySession::Stop on Android, 2nd attempt

device/bluetooth/bluetooth_gatt_notify_session.cc

Index: device/bluetooth/bluetooth_gatt_notify_session.cc
diff --git a/device/bluetooth/bluetooth_gatt_notify_session.cc b/device/bluetooth/bluetooth_gatt_notify_session.cc
index 3c9a242388024c5321d4bf4bc1529bd1aaa98b23..9890afec7307ef3924378a0ffe4d3e491fad8531 100644
--- a/device/bluetooth/bluetooth_gatt_notify_session.cc
+++ b/device/bluetooth/bluetooth_gatt_notify_session.cc
@@ -4,12 +4,46 @@
 
 #include "device/bluetooth/bluetooth_gatt_notify_session.h"
 
+#include "base/bind.h"
+#include "base/logging.h"
+#include "device/bluetooth/bluetooth_remote_gatt_characteristic.h"
+
 namespace device {
 
-BluetoothGattNotifySession::BluetoothGattNotifySession() {
-}
+BluetoothGattNotifySession::BluetoothGattNotifySession(
+    base::WeakPtr<BluetoothRemoteGattCharacteristic> characteristic)
+    : characteristic_(characteristic),
+      characteristic_id_(characteristic.get() ? characteristic->GetIdentifier()
+                                              : std::string()),
+      active_(true) {}
 
 BluetoothGattNotifySession::~BluetoothGattNotifySession() {
+  if (active_) {
+    Stop(base::Bind(&base::DoNothing));
+  }
+}
+
+std::string BluetoothGattNotifySession::GetCharacteristicIdentifier() const {
+  return characteristic_id_;
+}
+
+BluetoothRemoteGattCharacteristic*
+BluetoothGattNotifySession::GetCharacteristic() const {
+  return characteristic_.get();
+}
+
+bool BluetoothGattNotifySession::IsActive() {
+  return active_ && characteristic_ != nullptr &&
+         characteristic_->IsNotifying();
+}
+
+void BluetoothGattNotifySession::Stop(const base::Closure& callback) {
+  active_ = false;
+  if (characteristic_ != nullptr) {
+    characteristic_->StopNotifySession(this, callback);
+  } else {
+    callback.Run();

[ortuno, 2016/08/09 21:57:55]

nit: post a task. Could you also add a test to check that this is called
asynchronously when the notify session has been stopped already?

[tommyt, 2016/08/10 12:07:18]

Done. I've added the StopNotifySession_AfterDeleted unittest to test for this.

+  }
 }
 
 }  // namespace device