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

Unified Diff: chromeos/dbus/experimental_bluetooth_profile_service_provider.cc

Issue 14487002: Bluetooth: Profile support for Chrome OS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add unit tests, and class comments Created 7 years, 7 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: chromeos/dbus/experimental_bluetooth_profile_service_provider.cc
diff --git a/chromeos/dbus/experimental_bluetooth_profile_service_provider.cc b/chromeos/dbus/experimental_bluetooth_profile_service_provider.cc
index a58c8bc8992be3025994b6b67c530fe0c53aff6a..2b2384ff06e7102e8ebdf3b38382f08f240815f3 100644
--- a/chromeos/dbus/experimental_bluetooth_profile_service_provider.cc
+++ b/chromeos/dbus/experimental_bluetooth_profile_service_provider.cc
@@ -113,10 +113,10 @@ class ExperimentalBluetoothProfileServiceProviderImpl
dbus::MessageReader reader(method_call);
dbus::ObjectPath device_path;
- dbus::FileDescriptor fd;
+ scoped_ptr<dbus::FileDescriptor> fd(new dbus::FileDescriptor());
dbus::MessageReader array_reader(NULL);
if (!reader.PopObjectPath(&device_path) ||
- !reader.PopFileDescriptor(&fd) ||
+ !reader.PopFileDescriptor(fd.get()) ||
!reader.PopArray(&array_reader)) {
LOG(WARNING) << "NewConnection called with incorrect paramters: "
<< method_call->ToString();
@@ -145,9 +145,7 @@ class ExperimentalBluetoothProfileServiceProviderImpl
method_call,
response_sender);
- delegate_->NewConnection(device_path, &fd, options, callback);
-
- response_sender.Run(dbus::Response::FromMethodCall(method_call));
+ delegate_->NewConnection(device_path, fd.Pass(), options, callback);
}
// Called by dbus:: when the Bluetooth daemon is about to disconnect the
« no previous file with comments | « chromeos/dbus/experimental_bluetooth_profile_service_provider.h ('k') | chromeos/dbus/fake_bluetooth_device_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698