Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1736)

Unified Diff: device/bluetooth/bluetooth_gatt_notify_session.cc

Issue 2051333004: Implement BluetoothGattNotifySession::Stop on Android, 2nd attempt (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address new review comments Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..5e22e71d08695cd06157dff8922e6c78126144d4 100644
--- a/device/bluetooth/bluetooth_gatt_notify_session.cc
+++ b/device/bluetooth/bluetooth_gatt_notify_session.cc
@@ -4,12 +4,48 @@
#include "device/bluetooth/bluetooth_gatt_notify_session.h"
+#include "base/bind.h"
+#include "base/bind_helpers.h"
+#include "base/logging.h"
+#include "base/threading/thread_task_runner_handle.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 {
+ base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE, callback);
+ }
}
} // namespace device
« no previous file with comments | « device/bluetooth/bluetooth_gatt_notify_session.h ('k') | device/bluetooth/bluetooth_gatt_notify_session_android.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698