diff options
author | jamuraa <jamuraa@chromium.org> | 2015-06-25 14:55:43 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-06-25 21:56:20 +0000 |
commit | 512c2f57f0dbcb312ee5aa79451568f7a8233a02 (patch) | |
tree | dc83d0244157b88fb1a2ad2ff38e9ad000ad590c /chromeos/dbus/fake_bluetooth_profile_service_provider.h | |
parent | 917fac73b0b046b774ea10fa76074d7397568f8d (diff) | |
download | chromium_src-512c2f57f0dbcb312ee5aa79451568f7a8233a02.zip chromium_src-512c2f57f0dbcb312ee5aa79451568f7a8233a02.tar.gz chromium_src-512c2f57f0dbcb312ee5aa79451568f7a8233a02.tar.bz2 |
Submission for C++ Readability
Some files have had minor comments changes made, others just an added newline at the end for git to allow adding.
BUG=None
Review URL: https://codereview.chromium.org/1124883004
Cr-Commit-Position: refs/heads/master@{#336270}
Diffstat (limited to 'chromeos/dbus/fake_bluetooth_profile_service_provider.h')
-rw-r--r-- | chromeos/dbus/fake_bluetooth_profile_service_provider.h | 28 |
1 files changed, 15 insertions, 13 deletions
diff --git a/chromeos/dbus/fake_bluetooth_profile_service_provider.h b/chromeos/dbus/fake_bluetooth_profile_service_provider.h index 95a88c5..2c98048 100644 --- a/chromeos/dbus/fake_bluetooth_profile_service_provider.h +++ b/chromeos/dbus/fake_bluetooth_profile_service_provider.h @@ -18,28 +18,28 @@ namespace chromeos { // FakeBluetoothProfileServiceProvider simulates the behavior of a local // Bluetooth agent object and is used both in test cases in place of a // mock and on the Linux desktop. +// +// This class is only called from the dbus origin thread and is not thread-safe. class CHROMEOS_EXPORT FakeBluetoothProfileServiceProvider : public BluetoothProfileServiceProvider { public: FakeBluetoothProfileServiceProvider(const dbus::ObjectPath& object_path, - Delegate *delegate); + Delegate* delegate); ~FakeBluetoothProfileServiceProvider() override; // Each of these calls the equivalent // BluetoothProfileServiceProvider::Delegate method on the object passed on // construction. - virtual void Release(); - virtual void NewConnection( - const dbus::ObjectPath& device_path, - scoped_ptr<dbus::FileDescriptor> fd, - const Delegate::Options& options, - const Delegate::ConfirmationCallback& callback); - virtual void RequestDisconnection( - const dbus::ObjectPath& device_path, - const Delegate::ConfirmationCallback& callback); - virtual void Cancel(); - - const dbus::ObjectPath& object_path() { return object_path_; } + void Released(); + void NewConnection(const dbus::ObjectPath& device_path, + scoped_ptr<dbus::FileDescriptor> fd, + const Delegate::Options& options, + const Delegate::ConfirmationCallback& callback); + void RequestDisconnection(const dbus::ObjectPath& device_path, + const Delegate::ConfirmationCallback& callback); + void Cancel(); + + const dbus::ObjectPath& object_path() const { return object_path_; } private: friend class FakeBluetoothProfileManagerClient; @@ -51,6 +51,8 @@ class CHROMEOS_EXPORT FakeBluetoothProfileServiceProvider // passed to generate the reply. |delegate_| is generally the object that // owns this one, and must outlive it. Delegate* delegate_; + + DISALLOW_COPY_AND_ASSIGN(FakeBluetoothProfileServiceProvider); }; } // namespace chromeos |