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

Unified Diff: device/bluetooth/bluetooth_gatt_notify_session_win.cc

Issue 1749403002: Implement BluetoothRemoteGattCharacteristicWin::StartNotifySession and related unit tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: adjust comments Created 4 years, 10 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_win.cc
diff --git a/device/bluetooth/bluetooth_gatt_notify_session_win.cc b/device/bluetooth/bluetooth_gatt_notify_session_win.cc
new file mode 100644
index 0000000000000000000000000000000000000000..ee416ced679975daa69b0d625d27e3e866ea3bf6
--- /dev/null
+++ b/device/bluetooth/bluetooth_gatt_notify_session_win.cc
@@ -0,0 +1,34 @@
+// Copyright 2016 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 "device/bluetooth/bluetooth_gatt_notify_session_win.h"
+
+namespace device {
+BluetoothGattNotifySessionWin::BluetoothGattNotifySessionWin(
+ base::WeakPtr<BluetoothRemoteGattCharacteristicWin> characteristic)
+ : is_active_(true), characteristic_(characteristic) {}
+
+BluetoothGattNotifySessionWin::~BluetoothGattNotifySessionWin() {
+ if (characteristic_.get() != nullptr && is_active_)
+ characteristic_.get()->StopNotifySession();
+}
+
+std::string BluetoothGattNotifySessionWin::GetCharacteristicIdentifier() const {
+ if (characteristic_.get() != nullptr)
+ return characteristic_.get()->GetIdentifier();
+ return std::string();
+}
+
+bool BluetoothGattNotifySessionWin::IsActive() {
+ return is_active_ && characteristic_.get() != nullptr;
+}
+
+void BluetoothGattNotifySessionWin::Stop(const base::Closure& callback) {
+ is_active_ = false;
+ if (characteristic_.get() != nullptr)
+ return characteristic_.get()->StopNotifySession();
ortuno 2016/03/07 18:48:44 Split this: characteristic_.get()->StopNotifySess
gogerald1 2016/03/07 22:52:48 Done.
+ callback.Run();
ortuno 2016/03/07 18:48:44 Stop should be async. Post a task.
gogerald1 2016/03/07 22:52:48 Done.
+}
+
+} // namespace device

Powered by Google App Engine
This is Rietveld 408576698