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

Side by Side Diff: device/bluetooth/bluetooth_gatt_notify_session_chromeos.cc

Issue 1415573014: Reland "Add Linux support for the Bluetooth API" (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: build fix. Created 5 years, 1 month 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "device/bluetooth/bluetooth_gatt_notify_session_chromeos.h"
6
7 #include "base/bind.h"
8 #include "base/logging.h"
9 #include "device/bluetooth/bluetooth_adapter.h"
10 #include "device/bluetooth/bluetooth_device.h"
11 #include "device/bluetooth/bluetooth_gatt_service.h"
12 #include "device/bluetooth/bluetooth_remote_gatt_characteristic_chromeos.h"
13 #include "device/bluetooth/dbus/bluez_dbus_manager.h"
14
15 namespace chromeos {
16
17 BluetoothGattNotifySessionChromeOS::BluetoothGattNotifySessionChromeOS(
18 scoped_refptr<device::BluetoothAdapter> adapter,
19 const std::string& device_address,
20 const std::string& service_identifier,
21 const std::string& characteristic_identifier,
22 const dbus::ObjectPath& characteristic_path)
23 : active_(true),
24 adapter_(adapter),
25 device_address_(device_address),
26 service_id_(service_identifier),
27 characteristic_id_(characteristic_identifier),
28 object_path_(characteristic_path) {
29 DCHECK(adapter_.get());
30 DCHECK(!device_address_.empty());
31 DCHECK(!service_id_.empty());
32 DCHECK(!characteristic_id_.empty());
33 DCHECK(object_path_.IsValid());
34
35 bluez::BluezDBusManager::Get()
36 ->GetBluetoothGattCharacteristicClient()
37 ->AddObserver(this);
38 }
39
40 BluetoothGattNotifySessionChromeOS::~BluetoothGattNotifySessionChromeOS() {
41 bluez::BluezDBusManager::Get()
42 ->GetBluetoothGattCharacteristicClient()
43 ->RemoveObserver(this);
44 Stop(base::Bind(&base::DoNothing));
45 }
46
47 std::string BluetoothGattNotifySessionChromeOS::GetCharacteristicIdentifier()
48 const {
49 return characteristic_id_;
50 }
51
52 bool BluetoothGattNotifySessionChromeOS::IsActive() {
53 // Determine if the session is active. If |active_| is false, then it's
54 // been explicitly marked, so return false.
55 if (!active_)
56 return false;
57
58 // The fact that |active_| is true doesn't mean that the session is
59 // actually active, since the characteristic might have stopped sending
60 // notifications yet this method was called before we processed the
61 // observer event (e.g. because somebody else called this method in their
62 // bluez::BluetoothGattCharacteristicClient::Observer implementation, which
63 // was
64 // called before ours). Check the client to see if notifications are still
65 // being sent.
66 bluez::BluetoothGattCharacteristicClient::Properties* properties =
67 bluez::BluezDBusManager::Get()
68 ->GetBluetoothGattCharacteristicClient()
69 ->GetProperties(object_path_);
70 if (!properties || !properties->notifying.value())
71 active_ = false;
72
73 return active_;
74 }
75
76 void BluetoothGattNotifySessionChromeOS::Stop(const base::Closure& callback) {
77 if (!active_) {
78 VLOG(1) << "Notify session already inactive.";
79 callback.Run();
80 return;
81 }
82
83 // Mark this session as inactive no matter what.
84 active_ = false;
85
86 device::BluetoothDevice* device = adapter_->GetDevice(device_address_);
87 if (!device)
88 return;
89
90 device::BluetoothGattService* service = device->GetGattService(service_id_);
91 if (!service)
92 return;
93
94 BluetoothRemoteGattCharacteristicChromeOS* chrc =
95 static_cast<BluetoothRemoteGattCharacteristicChromeOS*>(
96 service->GetCharacteristic(characteristic_id_));
97 if (!chrc)
98 return;
99
100 chrc->RemoveNotifySession(callback);
101 }
102
103 void BluetoothGattNotifySessionChromeOS::GattCharacteristicRemoved(
104 const dbus::ObjectPath& object_path) {
105 if (object_path != object_path_)
106 return;
107
108 active_ = false;
109 }
110
111 void BluetoothGattNotifySessionChromeOS::GattCharacteristicPropertyChanged(
112 const dbus::ObjectPath& object_path,
113 const std::string& property_name) {
114 if (object_path != object_path_)
115 return;
116
117 if (!active_)
118 return;
119
120 bluez::BluetoothGattCharacteristicClient::Properties* properties =
121 bluez::BluezDBusManager::Get()
122 ->GetBluetoothGattCharacteristicClient()
123 ->GetProperties(object_path_);
124 if (!properties) {
125 active_ = false;
126 return;
127 }
128
129 if (property_name == properties->notifying.name() &&
130 !properties->notifying.value())
131 active_ = false;
132 }
133
134 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698