Bluetooth: Profile support for Chrome OS

chromeos/dbus/experimental_bluetooth_profile_service_provider.h

 // Copyright (c) 2013 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 CHROMEOS_DBUS_EXPERIMENTAL_BLUETOOTH_PROFILE_SERVICE_PROVIDER_H_
 #define CHROMEOS_DBUS_EXPERIMENTAL_BLUETOOTH_PROFILE_SERVICE_PROVIDER_H_
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
+#include "base/memory/scoped_ptr.h"
 #include "chromeos/chromeos_export.h"
 #include "dbus/bus.h"
 #include "dbus/file_descriptor.h"
 #include "dbus/object_path.h"
 
 namespace chromeos {
 
 // ExperimentalBluetoothProfileServiceProvider is used to provide a D-Bus
 // object that BlueZ can communicate with to connect application profiles.
 //
@@ skipping 45 matching lines @@
     // Bluetooth daemon, generally at shutdown or at the applications' request.
     // It may be used to perform cleanup tasks.
     virtual void Release() = 0;
 
     // This method will be called when a profile connection to the device
     // with object path |device_path| is established. |callback| must be called
     // to confirm the connection, or indicate rejection or cancellation.
     //
     // A file descriptor for the connection socket is provided in |fd|, and
     // details about the specific implementation of the profile in |options|.
-    // The delegate should take the value and ownership
-
-    // The file descriptor is owned by the delegate after this call so must be
-    // cleaned up if the connection is cancelled or rejected, the |options|
-    // structure is not so information out of it must be copied if required.
+    //
+    // IMPORTANT: Ownership of the file descriptor object |fd| is passed to
+    // the delegate by this call. The delegate is responsible for checking the
+    // validity of |fd| on a thread where I/O is permitted before taking the
+    // value. If the value is not taken, the file descriptor is closed.
+    //
+    // Ownership of |options| is NOT passed so information out of it must be
+    // copied if required.
     virtual void NewConnection(const dbus::ObjectPath& device_path,
-                               dbus::FileDescriptor* fd,
+                               scoped_ptr<dbus::FileDescriptor> fd,
                                const Options& options,
                                const ConfirmationCallback& callback) = 0;
 
     // This method will be called when a profile connection to the device
     // with object path |device_path| is disconnected. Any file descriptors
     // owned by the service should be cleaned up and |callback| called to
     // confirm, or indicate rejection or cancellation of the disconnection.
     virtual void RequestDisconnection(const dbus::ObjectPath& device_path,
                                       const ConfirmationCallback& callback) = 0;
 
@@ skipping 14 matching lines @@
  protected:
   ExperimentalBluetoothProfileServiceProvider();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ExperimentalBluetoothProfileServiceProvider);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_EXPERIMENTAL_BLUETOOTH_PROFILE_SERVICE_PROVIDER_H_