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

Unified Diff: device/bluetooth/device.cc

Issue 2448713002: bluetooth: Add Device connection logic and accompanying user interface. (Closed)
Patch Set: Remove GetConnectionError Created 4 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 side-by-side diff with in-line comments
Download patch
Index: device/bluetooth/device.cc
diff --git a/device/bluetooth/device.cc b/device/bluetooth/device.cc
index 7e662802eabb5aed91c9859f782903049c866170..3d8a9791b532712f9fe3b81d171eb45d3bc2381e 100644
--- a/device/bluetooth/device.cc
+++ b/device/bluetooth/device.cc
@@ -3,6 +3,7 @@
// found in the LICENSE file.
#include <utility>
+#include <vector>
#include "base/strings/utf_string_conversions.h"
#include "device/bluetooth/device.h"
@@ -10,11 +11,15 @@
namespace bluetooth {
-Device::Device(const std::string& address,
- scoped_refptr<device::BluetoothAdapter> adapter)
- : address_(address), adapter_(std::move(adapter)) {}
+Device::Device(scoped_refptr<device::BluetoothAdapter> adapter,
+ std::unique_ptr<device::BluetoothGattConnection> connection)
+ : adapter_(std::move(adapter)), connection_(std::move(connection)) {
+ adapter_->AddObserver(this);
+}
-Device::~Device() {}
+Device::~Device() {
+ adapter_->RemoveObserver(this);
+}
// static
mojom::DeviceInfoPtr Device::ConstructDeviceInfoStruct(
@@ -25,6 +30,7 @@ mojom::DeviceInfoPtr Device::ConstructDeviceInfoStruct(
device_info->name_for_display =
base::UTF16ToUTF8(device->GetNameForDisplay());
device_info->address = device->GetAddress();
+ device_info->is_gatt_connected = device->IsGattConnected();
if (device->GetInquiryRSSI()) {
device_info->rssi = mojom::RSSIWrapper::New();
@@ -34,14 +40,89 @@ mojom::DeviceInfoPtr Device::ConstructDeviceInfoStruct(
return device_info;
}
+void Device::SetStrongBindingPtr(
+ mojo::StrongBindingPtr<mojom::Device> binding) {
+ binding_ = binding;
+}
+
+void Device::DeviceChanged(device::BluetoothAdapter* adapter,
+ device::BluetoothDevice* device) {
+ if (device->GetAddress() != GetAddress()) {
+ return;
+ }
+
+ if (!device->IsGattConnected()) {
+ binding_->Close();
+ }
+}
+
+void Device::GattServicesDiscovered(device::BluetoothAdapter* adapter,
+ device::BluetoothDevice* device) {
+ if (device->GetAddress() != GetAddress()) {
+ return;
+ }
+
+ std::vector<base::Closure> requests;
+ requests.swap(pending_services_requests_);
+ for (const base::Closure& request : requests) {
+ request.Run();
+ }
+}
+
+void Device::Disconnect() {
+ binding_->Close();
+}
+
void Device::GetInfo(const GetInfoCallback& callback) {
- device::BluetoothDevice* device = adapter_->GetDevice(address_);
- if (device) {
- mojom::DeviceInfoPtr device_info = ConstructDeviceInfoStruct(device);
- callback.Run(std::move(device_info));
- } else {
- callback.Run(nullptr);
+ device::BluetoothDevice* device = adapter_->GetDevice(GetAddress());
+ DCHECK(device);
+
+ mojom::DeviceInfoPtr device_info = ConstructDeviceInfoStruct(device);
+ callback.Run(std::move(device_info));
+}
+
+void Device::GetServices(const GetServicesCallback& callback) {
+ device::BluetoothDevice* device = adapter_->GetDevice(GetAddress());
+ DCHECK(device);
+
+ if (device->IsGattServicesDiscoveryComplete()) {
+ GetServicesImpl(callback);
+ return;
+ }
+
+ // pending_services_requests_ is owned by Device, so base::Unretained is
+ // safe.
+ pending_services_requests_.push_back(
+ base::Bind(&Device::GetServicesImpl, base::Unretained(this), callback));
+}
+
+void Device::GetServicesImpl(const GetServicesCallback& callback) {
+ device::BluetoothDevice* device = adapter_->GetDevice(GetAddress());
+ DCHECK(device);
+
+ std::vector<mojom::ServiceInfoPtr> services;
+
+ for (const device::BluetoothRemoteGattService* service :
+ device->GetGattServices()) {
+ mojom::ServiceInfoPtr service_info = ConstructServiceInfoStruct(*service);
+ services.push_back(std::move(service_info));
}
+
+ callback.Run(std::move(services));
+}
+
+mojom::ServiceInfoPtr Device::ConstructServiceInfoStruct(
+ const device::BluetoothRemoteGattService& service) {
+ mojom::ServiceInfoPtr service_info = mojom::ServiceInfo::New();
+
+ service_info->uuid = service.GetUUID();
+ service_info->is_primary = service.IsPrimary();
+
+ return service_info;
+}
+
+const std::string& Device::GetAddress() {
+ return connection_->GetDeviceAddress();
}
} // namespace bluetooth

Powered by Google App Engine
This is Rietveld 408576698