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

Unified Diff: device/bluetooth/dbus/bluetooth_gatt_service_service_provider.cc

Issue 1914893002: DBus changes for implementing local GATT attributes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@bluetooth_classes
Patch Set: test leak fix Created 4 years, 8 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/dbus/bluetooth_gatt_service_service_provider.cc
diff --git a/device/bluetooth/dbus/bluetooth_gatt_service_service_provider.cc b/device/bluetooth/dbus/bluetooth_gatt_service_service_provider.cc
index 686a1a5f0709d2a9cdc0197af505b1556bbf8e53..04d93d56b1ff3312cf3e7068d8292ba315bd4ceb 100644
--- a/device/bluetooth/dbus/bluetooth_gatt_service_service_provider.cc
+++ b/device/bluetooth/dbus/bluetooth_gatt_service_service_provider.cc
@@ -4,255 +4,11 @@
#include "device/bluetooth/dbus/bluetooth_gatt_service_service_provider.h"
-#include <memory>
-#include <utility>
-
-#include "base/bind.h"
-#include "base/logging.h"
-#include "base/macros.h"
-#include "base/memory/ref_counted.h"
-#include "base/memory/weak_ptr.h"
-#include "base/threading/platform_thread.h"
-#include "dbus/exported_object.h"
-#include "dbus/message.h"
+#include "device/bluetooth/dbus/bluetooth_gatt_service_service_provider_impl.h"
#include "device/bluetooth/dbus/bluez_dbus_manager.h"
#include "device/bluetooth/dbus/fake_bluetooth_gatt_service_service_provider.h"
-#include "third_party/cros_system_api/dbus/service_constants.h"
namespace bluez {
-namespace {
-const char kErrorInvalidArgs[] = "org.freedesktop.DBus.Error.InvalidArgs";
-const char kErrorPropertyReadOnly[] =
- "org.freedesktop.DBus.Error.PropertyReadOnly";
-} // namespace
-
-// The BluetoothGattServiceServiceProvider implementation used in production.
-class BluetoothGattServiceServiceProviderImpl
- : public BluetoothGattServiceServiceProvider {
- public:
- BluetoothGattServiceServiceProviderImpl(
- dbus::Bus* bus,
- const dbus::ObjectPath& object_path,
- const std::string& uuid,
- const std::vector<dbus::ObjectPath>& includes)
- : origin_thread_id_(base::PlatformThread::CurrentId()),
- uuid_(uuid),
- includes_(includes),
- bus_(bus),
- object_path_(object_path),
- weak_ptr_factory_(this) {
- VLOG(1) << "Creating Bluetooth GATT service: " << object_path_.value()
- << " UUID: " << uuid;
- DCHECK(!uuid_.empty());
- DCHECK(object_path_.IsValid());
- DCHECK(bus_);
-
- exported_object_ = bus_->GetExportedObject(object_path_);
-
- exported_object_->ExportMethod(
- dbus::kDBusPropertiesInterface, dbus::kDBusPropertiesGet,
- base::Bind(&BluetoothGattServiceServiceProviderImpl::Get,
- weak_ptr_factory_.GetWeakPtr()),
- base::Bind(&BluetoothGattServiceServiceProviderImpl::OnExported,
- weak_ptr_factory_.GetWeakPtr()));
-
- exported_object_->ExportMethod(
- dbus::kDBusPropertiesInterface, dbus::kDBusPropertiesSet,
- base::Bind(&BluetoothGattServiceServiceProviderImpl::Set,
- weak_ptr_factory_.GetWeakPtr()),
- base::Bind(&BluetoothGattServiceServiceProviderImpl::OnExported,
- weak_ptr_factory_.GetWeakPtr()));
-
- exported_object_->ExportMethod(
- dbus::kDBusPropertiesInterface, dbus::kDBusPropertiesGetAll,
- base::Bind(&BluetoothGattServiceServiceProviderImpl::GetAll,
- weak_ptr_factory_.GetWeakPtr()),
- base::Bind(&BluetoothGattServiceServiceProviderImpl::OnExported,
- weak_ptr_factory_.GetWeakPtr()));
- }
-
- ~BluetoothGattServiceServiceProviderImpl() override {
- VLOG(1) << "Cleaning up Bluetooth GATT service: " << object_path_.value();
- bus_->UnregisterExportedObject(object_path_);
- }
-
- private:
- // Returns true if the current thread is on the origin thread.
- bool OnOriginThread() {
- return base::PlatformThread::CurrentId() == origin_thread_id_;
- }
-
- // Called by dbus:: when the Bluetooth daemon fetches a single property of
- // the service.
- void Get(dbus::MethodCall* method_call,
- dbus::ExportedObject::ResponseSender response_sender) {
- VLOG(2) << "BluetoothGattServiceServiceProvider::Get: "
- << object_path_.value();
- DCHECK(OnOriginThread());
-
- dbus::MessageReader reader(method_call);
-
- std::string interface_name;
- std::string property_name;
- if (!reader.PopString(&interface_name) ||
- !reader.PopString(&property_name) || reader.HasMoreData()) {
- std::unique_ptr<dbus::ErrorResponse> error_response =
- dbus::ErrorResponse::FromMethodCall(method_call, kErrorInvalidArgs,
- "Expected 'ss'.");
- response_sender.Run(std::move(error_response));
- return;
- }
-
- // Only the GATT service interface is allowed.
- if (interface_name !=
- bluetooth_gatt_service::kBluetoothGattServiceInterface) {
- std::unique_ptr<dbus::ErrorResponse> error_response =
- dbus::ErrorResponse::FromMethodCall(
- method_call, kErrorInvalidArgs,
- "No such interface: '" + interface_name + "'.");
- response_sender.Run(std::move(error_response));
- return;
- }
-
- // Return error if |property_name| is unknown.
- if (property_name != bluetooth_gatt_service::kUUIDProperty &&
- property_name != bluetooth_gatt_service::kIncludesProperty) {
- std::unique_ptr<dbus::ErrorResponse> error_response =
- dbus::ErrorResponse::FromMethodCall(
- method_call, kErrorInvalidArgs,
- "No such property: '" + property_name + "'.");
- response_sender.Run(std::move(error_response));
- return;
- }
-
- std::unique_ptr<dbus::Response> response =
- dbus::Response::FromMethodCall(method_call);
- dbus::MessageWriter writer(response.get());
- dbus::MessageWriter variant_writer(NULL);
-
- if (property_name == bluetooth_gatt_service::kUUIDProperty) {
- writer.OpenVariant("s", &variant_writer);
- variant_writer.AppendString(uuid_);
- writer.CloseContainer(&variant_writer);
- } else {
- writer.OpenVariant("ao", &variant_writer);
- variant_writer.AppendArrayOfObjectPaths(includes_);
- writer.CloseContainer(&variant_writer);
- }
-
- response_sender.Run(std::move(response));
- }
-
- // Called by dbus:: when the Bluetooth daemon sets a single property of the
- // service.
- void Set(dbus::MethodCall* method_call,
- dbus::ExportedObject::ResponseSender response_sender) {
- VLOG(2) << "BluetoothGattServiceServiceProvider::Set: "
- << object_path_.value();
- DCHECK(OnOriginThread());
-
- // All of the properties on this interface are read-only, so just return
- // error.
- std::unique_ptr<dbus::ErrorResponse> error_response =
- dbus::ErrorResponse::FromMethodCall(method_call, kErrorPropertyReadOnly,
- "All properties are read-only.");
- response_sender.Run(std::move(error_response));
- }
-
- // Called by dbus:: when the Bluetooth daemon fetches all properties of the
- // service.
- void GetAll(dbus::MethodCall* method_call,
- dbus::ExportedObject::ResponseSender response_sender) {
- VLOG(2) << "BluetoothGattServiceServiceProvider::GetAll: "
- << object_path_.value();
- DCHECK(OnOriginThread());
-
- dbus::MessageReader reader(method_call);
-
- std::string interface_name;
- if (!reader.PopString(&interface_name) || reader.HasMoreData()) {
- std::unique_ptr<dbus::ErrorResponse> error_response =
- dbus::ErrorResponse::FromMethodCall(method_call, kErrorInvalidArgs,
- "Expected 's'.");
- response_sender.Run(std::move(error_response));
- return;
- }
-
- // Only the GATT service interface is allowed.
- if (interface_name !=
- bluetooth_gatt_service::kBluetoothGattServiceInterface) {
- std::unique_ptr<dbus::ErrorResponse> error_response =
- dbus::ErrorResponse::FromMethodCall(
- method_call, kErrorInvalidArgs,
- "No such interface: '" + interface_name + "'.");
- response_sender.Run(std::move(error_response));
- return;
- }
-
- std::unique_ptr<dbus::Response> response =
- dbus::Response::FromMethodCall(method_call);
- dbus::MessageWriter writer(response.get());
- dbus::MessageWriter array_writer(NULL);
- dbus::MessageWriter dict_entry_writer(NULL);
- dbus::MessageWriter variant_writer(NULL);
-
- writer.OpenArray("{sv}", &array_writer);
-
- array_writer.OpenDictEntry(&dict_entry_writer);
- dict_entry_writer.AppendString(bluetooth_gatt_service::kUUIDProperty);
- dict_entry_writer.AppendVariantOfString(uuid_);
- array_writer.CloseContainer(&dict_entry_writer);
-
- array_writer.OpenDictEntry(&dict_entry_writer);
- dict_entry_writer.AppendString(bluetooth_gatt_service::kIncludesProperty);
- dict_entry_writer.OpenVariant("ao", &variant_writer);
- variant_writer.AppendArrayOfObjectPaths(includes_);
- dict_entry_writer.CloseContainer(&variant_writer);
- array_writer.CloseContainer(&dict_entry_writer);
-
- writer.CloseContainer(&array_writer);
-
- response_sender.Run(std::move(response));
- }
-
- // Called by dbus:: when a method is exported.
- void OnExported(const std::string& interface_name,
- const std::string& method_name,
- bool success) {
- LOG_IF(WARNING, !success) << "Failed to export " << interface_name << "."
- << method_name;
- }
-
- // Origin thread (i.e. the UI thread in production).
- base::PlatformThreadId origin_thread_id_;
-
- // 128-bit service UUID of this object.
- std::string uuid_;
-
- // List of object paths that represent other exported GATT services that are
- // included from this service.
- std::vector<dbus::ObjectPath> includes_;
-
- // D-Bus bus object is exported on, not owned by this object and must
- // outlive it.
- dbus::Bus* bus_;
-
- // D-Bus object path of object we are exporting, kept so we can unregister
- // again in our destructor.
- dbus::ObjectPath object_path_;
-
- // D-Bus object we are exporting, owned by this object.
- scoped_refptr<dbus::ExportedObject> exported_object_;
-
- // Weak pointer factory for generating 'this' pointers that might live longer
- // than we do.
- // Note: This should remain the last member so it'll be destroyed and
- // invalidate its weak pointers before any other members are destroyed.
- base::WeakPtrFactory<BluetoothGattServiceServiceProviderImpl>
- weak_ptr_factory_;
-
- DISALLOW_COPY_AND_ASSIGN(BluetoothGattServiceServiceProviderImpl);
-};
BluetoothGattServiceServiceProvider::BluetoothGattServiceServiceProvider() {}

Powered by Google App Engine
This is Rietveld 408576698