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

Unified Diff: device/bluetooth/bluetooth_adapter_profile_chromeos.h

Issue 851123002: Manage profiles in BluetoothAdapter on ChromeOS (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix profile memory leaks when adapter is gone Created 5 years, 11 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_adapter_profile_chromeos.h
diff --git a/device/bluetooth/bluetooth_adapter_profile_chromeos.h b/device/bluetooth/bluetooth_adapter_profile_chromeos.h
new file mode 100644
index 0000000000000000000000000000000000000000..5f1edc2ead5e57aa892079cac232e8b71460515c
--- /dev/null
+++ b/device/bluetooth/bluetooth_adapter_profile_chromeos.h
@@ -0,0 +1,101 @@
+// Copyright 2015 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.
+
+#ifndef DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_PROFILE_CHROMEOS_H_
+#define DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_PROFILE_CHROMEOS_H_
+
+#include "base/memory/weak_ptr.h"
+#include "chromeos/dbus/bluetooth_profile_manager_client.h"
+#include "chromeos/dbus/bluetooth_profile_service_provider.h"
+#include "device/bluetooth/bluetooth_adapter_chromeos.h"
+#include "device/bluetooth/bluetooth_export.h"
+
+namespace device {
+class BluetoothUUID;
+} // namespace device
+
+namespace chromeos {
+
+// The BluetoothAdapterProfileChromeOS class implements a multiplexing
+// profile for custom Bluetooth services managed by a BluetoothAdapter.
+// Maintains a list of delegates which may serve the profile.
+// One delegate is allowed for each device.
+class DEVICE_BLUETOOTH_EXPORT BluetoothAdapterProfileChromeOS
+ : public chromeos::BluetoothProfileServiceProvider::Delegate {
+ public:
+ // Registers a profile with the BlueZ server for |uuid| with the
+ // options |options|. Returns a newly allocated pointer, or nullptr
+ // if there was a problem.
+ static BluetoothAdapterProfileChromeOS* Register(
+ BluetoothAdapterChromeOS* adapter,
+ const device::BluetoothUUID& uuid,
+ const BluetoothProfileManagerClient::Options& options,
+ const base::Closure& success_callback,
+ const BluetoothProfileManagerClient::ErrorCallback& error_callback);
+
+ ~BluetoothAdapterProfileChromeOS() override;
+
+ // The object path of the profile.
+ const dbus::ObjectPath& object_path() const { return object_path_; }
+
+ // Add a delegate for a device associated with this profile.
+ // An empty |device_path| indicates a local listening service.
+ // Returns true if the delegate was set, and false if the |device_path|
+ // already had a delegate set.
+ bool SetDelegate(const dbus::ObjectPath& device_path,
+ BluetoothProfileServiceProvider::Delegate* delegate);
+
+ // Remove the delegate for a device. |unregistered_callback| will be called
+ // if this unregisters the profile.
+ void RemoveDelegate(const dbus::ObjectPath& device_path,
+ const base::Closure& unregistered_callback);
+
+ // Returns the number of delegates for this profile.
+ size_t DelegateCount() const { return delegates_.size(); }
+
+ private:
+ BluetoothAdapterProfileChromeOS(BluetoothAdapterChromeOS* adapter,
+ const device::BluetoothUUID& uuid);
+
+ // BluetoothProfileServiceProvider::Delegate:
+ void Released() override;
+ void NewConnection(
+ const dbus::ObjectPath& device_path,
+ scoped_ptr<dbus::FileDescriptor> fd,
+ const BluetoothProfileServiceProvider::Delegate::Options& options,
+ const ConfirmationCallback& callback) override;
+ void RequestDisconnection(const dbus::ObjectPath& device_path,
+ const ConfirmationCallback& callback) override;
+ void Cancel() override;
+
+ // Called by dbus:: on completion of the D-Bus method to unregister a profile.
+ void OnUnregisterProfileError(const base::Closure& unregistered_callback,
+ const std::string& error_name,
+ const std::string& error_message);
+
+ // List of delegates which this profile is multiplexing to.
+ std::map<std::string, BluetoothProfileServiceProvider::Delegate*> delegates_;
+
+ // The UUID that this profile represents.
+ const device::BluetoothUUID& uuid_;
+
+ // Registered dbus object path for this profile.
+ dbus::ObjectPath object_path_;
+
+ // Profile dbus object for receiving profile method calls from BlueZ
+ scoped_ptr<BluetoothProfileServiceProvider> profile_;
+
+ // Adapter that owns this profile.
+ BluetoothAdapterChromeOS* adapter_;
+
+ // 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<BluetoothAdapterProfileChromeOS> weak_ptr_factory_;
+
+ DISALLOW_COPY_AND_ASSIGN(BluetoothAdapterProfileChromeOS);
+};
+
+} // namespace chromeos
+
+#endif
« no previous file with comments | « device/bluetooth/bluetooth_adapter_chromeos.cc ('k') | device/bluetooth/bluetooth_adapter_profile_chromeos.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698