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

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

Powered by Google App Engine
This is Rietveld 408576698