No dbus::FileDescriptor in DebugDaemonClient

chromeos/dbus/debug_daemon_client.h

 // Copyright (c) 2012 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_DEBUG_DAEMON_CLIENT_H_
 #define CHROMEOS_DBUS_DEBUG_DAEMON_CLIENT_H_
 
 #include <stdint.h>
 
 #include <map>
 
 #include "base/callback.h"
 #include "base/files/file.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted_memory.h"
 #include "base/task_runner.h"
 #include "base/trace_event/tracing_agent.h"
 #include "chromeos/chromeos_export.h"
 #include "chromeos/dbus/dbus_client.h"
-#include "dbus/file_descriptor.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
 
 namespace chromeos {
 
 // DebugDaemonClient is used to communicate with the debug daemon.
 class CHROMEOS_EXPORT DebugDaemonClient
     : public DBusClient,
       public base::trace_event::TracingAgent {
  public:
   ~DebugDaemonClient() override;
 
   // Called once GetDebugLogs() is complete. Takes one parameter:
   // - succeeded: was the logs stored successfully.
   typedef base::Callback<void(bool succeeded)> GetDebugLogsCallback;
 
-  // Requests to store debug logs into |file| and calls |callback|
+  // Requests to store debug logs into |file_descriptor| and calls |callback|
   // when completed. Debug logs will be stored in the .tgz if
   // |is_compressed| is true, otherwise in logs will be stored in .tar format.
+  // This method duplicates |file_descriptor| so it's OK to close the FD without
+  // waiting for the result.
   virtual void DumpDebugLogs(bool is_compressed,
-                             base::File file,
-                             scoped_refptr<base::TaskRunner> task_runner,
+                             int file_descriptor,
                              const GetDebugLogsCallback& callback) = 0;
 
   // Called once SetDebugMode() is complete. Takes one parameter:
   // - succeeded: debug mode was changed successfully.
   typedef base::Callback<void(bool succeeded)> SetDebugModeCallback;
 
   // Requests to change debug mode to given |subsystem| and calls
   // |callback| when completed. |subsystem| should be one of the
   // following: "wifi", "ethernet", "cellular" or "none".
   virtual void SetDebugMode(const std::string& subsystem,
@@ skipping 39 matching lines @@
       const GetNetworkInterfacesCallback& callback) = 0;
 
   using DBusMethodErrorCallback =
       base::Callback<void(const std::string& error_name,
                           const std::string& error_message)>;
 
   // Runs perf (via quipper) with arguments for |duration| (converted to
   // seconds) and returns data collected over the passed |file_descriptor|.
   // |error_callback| is called if there is an error with the DBus call.
   // Note that quipper failures may occur after successfully running the DBus
-  // method. Such errors can be detected by |file_descriptor| being closed with
-  // no data written.
+  // method. Such errors can be detected by |file_descriptor| and all its
+  // duplicates being closed with no data written.
+  // This method duplicates |file_descriptor| so it's OK to close the FD without
+  // waiting for the result.
   virtual void GetPerfOutput(base::TimeDelta duration,
                              const std::vector<std::string>& perf_args,
-                             dbus::ScopedFileDescriptor file_descriptor,
+                             int file_descriptor,
                              const DBusMethodErrorCallback& error_callback) = 0;
 
   // Callback type for GetScrubbedLogs(), GetAllLogs() or GetUserLogFiles().
   using GetLogsCallback =
       base::Callback<void(bool succeeded,
                           const std::map<std::string, std::string>& logs)>;
 
   // Gets scrubbed logs from debugd.
   virtual void GetScrubbedLogs(const GetLogsCallback& callback) = 0;
 
@@ skipping 84 matching lines @@
   // Create() should be used instead.
   DebugDaemonClient();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(DebugDaemonClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_DEBUG_DAEMON_CLIENT_H_