dbus: No dbus::FileDescriptor in BluetoothProfileServiceProvider

device/bluetooth/bluez/bluetooth_adapter_profile_bluez_unittest.cc

 // 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.
 
 #include "device/bluetooth/bluez/bluetooth_adapter_profile_bluez.h"
 
 #include <memory>
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 79 matching lines @@
       device_path_ = dbus::ObjectPath(device_path);
     }
 
     // bluez::BluetoothProfileServiceProvider::Delegate:
     void Released() override {
       // noop
     }
 
     void NewConnection(
         const dbus::ObjectPath& device_path,
-        std::unique_ptr<dbus::FileDescriptor> fd,
+        base::ScopedFD fd,
         const bluez::BluetoothProfileServiceProvider::Delegate::Options&
             options,
         const ConfirmationCallback& callback) override {
       ++connections_;
-      fd->CheckValidity();

[Rahul Chaturvedi, 2016/09/06 23:01:19]

We no longer care to check validity?

[hashimoto, 2016/09/07 05:55:17]

This "validity" check was introduced to avoid sending a directory FD to a Chrome
renderer process.
However, it turned out this check was not needed at all.
Please see http://crbug.com/621841 for the detail.

-      close(fd->TakeValue());
+      fd.reset();
       callback.Run(SUCCESS);
       if (device_path_.value() != "")
         ASSERT_EQ(device_path_, device_path);
     }
 
     void RequestDisconnection(const dbus::ObjectPath& device_path,
                               const ConfirmationCallback& callback) override {
       ++disconnections_;
     }
 
@@ skipping 265 matching lines @@
                  base::Unretained(this)));
 
   base::RunLoop().RunUntilIdle();
 
   EXPECT_FALSE(profile_user_ptr_);
   EXPECT_EQ(0U, success_callback_count_);
   EXPECT_EQ(2U, error_callback_count_);
 }
 
 }  // namespace bluez