chromeos: Remove dbus::FileDescriptor from LorgnetteManagerClient

chromeos/dbus/lorgnette_manager_client.cc

Index: chromeos/dbus/lorgnette_manager_client.cc
diff --git a/chromeos/dbus/lorgnette_manager_client.cc b/chromeos/dbus/lorgnette_manager_client.cc
index f3fd54130a73d81970baa848e6fc08f1155f23ac..f46fa73ab328fcd57cc9f465a4b25718b39a9110 100644
--- a/chromeos/dbus/lorgnette_manager_client.cc
+++ b/chromeos/dbus/lorgnette_manager_client.cc
@@ -9,6 +9,7 @@
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
+#include "base/files/scoped_file.h"
 #include "base/location.h"
 #include "base/macros.h"
 #include "base/task_runner_util.h"
@@ -18,7 +19,6 @@
 #include "dbus/message.h"
 #include "dbus/object_path.h"
 #include "dbus/object_proxy.h"
-#include "net/base/file_stream.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
 
 namespace chromeos {
@@ -26,6 +26,8 @@ namespace chromeos {
 // The LorgnetteManagerClient implementation used in production.
 class LorgnetteManagerClientImpl : public LorgnetteManagerClient {
  public:
+  typedef base::Callback<void(bool succeeded)> ScanImageToFileCallback;

[stevenjb, 2016/09/01 15:51:43]

nit: This name is confusing now, maybe ScanToStringCallback ?

[hashimoto, 2016/09/02 05:25:51]

Moved this callback to ScanToStringCompletion class and renamed it to
CompletionCallback.
ScanToStringCallback can also be a bit confusing as we already have
ScanImageToStringCallback.

+
   LorgnetteManagerClientImpl() :
       lorgnette_daemon_proxy_(NULL), weak_ptr_factory_(this) {}
 
@@ -43,28 +45,6 @@ class LorgnetteManagerClientImpl : public LorgnetteManagerClient {
   }
 
   // LorgnetteManagerClient override.
-  void ScanImageToFile(
-      std::string device_name,
-      const ScanProperties& properties,
-      const ScanImageToFileCallback& callback,
-      base::File* file) override {
-    dbus::FileDescriptor* file_descriptor = new dbus::FileDescriptor();
-    file_descriptor->PutValue(file->TakePlatformFile());
-    // Punt descriptor validity check to a worker thread; on return we'll
-    // issue the D-Bus request to stop tracing and collect results.
-    base::WorkerPool::PostTaskAndReply(
-        FROM_HERE,
-        base::Bind(&LorgnetteManagerClientImpl::CheckValidity,
-                   file_descriptor),
-        base::Bind(&LorgnetteManagerClientImpl::OnCheckValidityScanImage,
-                   weak_ptr_factory_.GetWeakPtr(),
-                   base::Owned(file_descriptor),
-                   device_name,
-                   properties,
-                   callback),
-        false);
-  }
-
   void ScanImageToString(
       std::string device_name,
       const ScanProperties& properties,
@@ -75,8 +55,35 @@ class LorgnetteManagerClientImpl : public LorgnetteManagerClient {
     base::ScopedFD fd;
     ScanImageToFileCallback file_callback =
         scan_to_string_completion->Start(callback, &fd);
-    base::File file(fd.release());
-    ScanImageToFile(device_name, properties, file_callback, &file);
+
+    // Issue the dbus request to scan an image.
+    dbus::MethodCall method_call(lorgnette::kManagerServiceInterface,
+                                 lorgnette::kScanImageMethod);
+    dbus::MessageWriter writer(&method_call);
+    writer.AppendString(device_name);
+    writer.AppendFileDescriptor(fd.get());
+
+    dbus::MessageWriter option_writer(NULL);
+    dbus::MessageWriter element_writer(NULL);
+    writer.OpenArray("{sv}", &option_writer);
+    if (!properties.mode.empty()) {
+      option_writer.OpenDictEntry(&element_writer);
+      element_writer.AppendString(lorgnette::kScanPropertyMode);
+      element_writer.AppendVariantOfString(properties.mode);
+      option_writer.CloseContainer(&element_writer);
+    }
+    if (properties.resolution_dpi) {
+      option_writer.OpenDictEntry(&element_writer);
+      element_writer.AppendString(lorgnette::kScanPropertyResolution);
+      element_writer.AppendVariantOfUint32(properties.resolution_dpi);
+      option_writer.CloseContainer(&element_writer);
+    }
+    writer.CloseContainer(&option_writer);
+
+    lorgnette_daemon_proxy_->CallMethod(
+        &method_call, dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
+        base::Bind(&LorgnetteManagerClientImpl::OnScanImageComplete,
+                   weak_ptr_factory_.GetWeakPtr(), file_callback));
   }
 
  protected:
@@ -186,55 +193,6 @@ class LorgnetteManagerClientImpl : public LorgnetteManagerClient {
     }
   }
 
-  // Called to check descriptor validity on a thread where i/o is permitted.
-  static void CheckValidity(dbus::FileDescriptor* file_descriptor) {
-    file_descriptor->CheckValidity();
-  }
-
-  // Called when a CheckValidity response is received.
-  void OnCheckValidityScanImage(
-      dbus::FileDescriptor* file_descriptor,
-      std::string device_name,
-      const ScanProperties& properties,
-      const ScanImageToFileCallback& callback) {
-    if (!file_descriptor->is_valid()) {
-      LOG(ERROR) << "Failed to scan image: file descriptor is invalid";
-      callback.Run(false);
-      return;
-    }
-    // Issue the dbus request to scan an image.
-    dbus::MethodCall method_call(
-        lorgnette::kManagerServiceInterface,
-        lorgnette::kScanImageMethod);
-    dbus::MessageWriter writer(&method_call);
-    writer.AppendString(device_name);
-    writer.AppendFileDescriptor(*file_descriptor);
-
-    dbus::MessageWriter option_writer(NULL);
-    dbus::MessageWriter element_writer(NULL);
-    writer.OpenArray("{sv}", &option_writer);
-    if (!properties.mode.empty()) {
-      option_writer.OpenDictEntry(&element_writer);
-      element_writer.AppendString(lorgnette::kScanPropertyMode);
-      element_writer.AppendVariantOfString(properties.mode);
-      option_writer.CloseContainer(&element_writer);
-    }
-    if (properties.resolution_dpi) {
-      option_writer.OpenDictEntry(&element_writer);
-      element_writer.AppendString(lorgnette::kScanPropertyResolution);
-      element_writer.AppendVariantOfUint32(properties.resolution_dpi);
-      option_writer.CloseContainer(&element_writer);
-    }
-    writer.CloseContainer(&option_writer);
-
-    lorgnette_daemon_proxy_->CallMethod(
-        &method_call,
-        dbus::ObjectProxy::TIMEOUT_USE_DEFAULT,
-        base::Bind(&LorgnetteManagerClientImpl::OnScanImageComplete,
-                   weak_ptr_factory_.GetWeakPtr(),
-                   callback));
-  }
-
   // Called when a response for ScanImage() is received.
   void OnScanImageComplete(const ScanImageToFileCallback& callback,
                            dbus::Response* response) {