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"
@@ skipping 56 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, so it must be deleted in all code paths
+    // including successful connnection or if the connection is cancelled or
+    // rejected. The delegate is responsible for checking the validity of |fd|
+    // on a thread where I/O is permitted before taking the value.
+    //
+    // 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,
                                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,
@@ skipping 16 matching lines @@
  protected:
   ExperimentalBluetoothProfileServiceProvider();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ExperimentalBluetoothProfileServiceProvider);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_EXPERIMENTAL_BLUETOOTH_PROFILE_SERVICE_PROVIDER_H_