Convert utility process CheckMediaFile IPC to mojo

chrome/browser/media_galleries/fileapi/safe_audio_video_checker.cc

 // Copyright 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.
 
 #include "chrome/browser/media_galleries/fileapi/safe_audio_video_checker.h"
 
 #include <stdint.h>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/process/process_handle.h"
 #include "base/single_thread_task_runner.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "chrome/common/chrome_utility_messages.h"
 #include "chrome/common/extensions/chrome_utility_extensions_messages.h"
 #include "chrome/grit/generated_resources.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/child_process_data.h"
 #include "content/public/browser/utility_process_host.h"
 #include "ipc/ipc_message_macros.h"
 #include "ipc/ipc_platform_file.h"
+#include "services/service_manager/public/cpp/interface_provider.h"
 #include "ui/base/l10n/l10n_util.h"
 
 SafeAudioVideoChecker::SafeAudioVideoChecker(
     base::File file,
     const storage::CopyOrMoveFileValidator::ResultCallback& callback)
     : state_(INITIAL_STATE), file_(std::move(file)), callback_(callback) {
   DCHECK(!callback.is_null());
 }
 
 void SafeAudioVideoChecker::Start() {
   DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
   if (state_ != INITIAL_STATE)
     return;
   state_ = STARTED_STATE;
 
   if (!file_.IsValid()) {
     callback_.Run(base::File::FILE_ERROR_SECURITY);
     state_ = FINISHED_STATE;
     return;
   }
 
-  IPC::PlatformFileForTransit file_for_transit =
-      IPC::TakePlatformFileForTransit(std::move(file_));
-  if (file_for_transit == IPC::InvalidPlatformFileForTransit()) {
-    OnCheckingFinished(false /* valid? */);
-    return;
-  }
-
   utility_process_host_ = content::UtilityProcessHost::Create(
       this, base::ThreadTaskRunnerHandle::Get())->AsWeakPtr();
   utility_process_host_->SetName(l10n_util::GetStringUTF16(
       IDS_UTILITY_PROCESS_MEDIA_FILE_CHECKER_NAME));
 
+  utility_process_host_->Start();
+
+  utility_process_host_->GetRemoteInterfaces()->GetInterface(&interface_);
+  interface_.set_connection_error_handler(
+      base::Bind(&SafeAudioVideoChecker::OnCheckingFinished, this, false));
+
   const int64_t kFileDecodeTimeInMS = 250;

[Sam McNally, 2017/01/18 07:09:09]

constexpr base::TimeDelta kFileDecodeTime =
base::TimeDelta::FromFromMilliseconds(250);

[Noel Gordon, 2017/01/19 06:33:11]

That worked well. Done.

-  utility_process_host_->Send(new ChromeUtilityMsg_CheckMediaFile(
-      kFileDecodeTimeInMS, file_for_transit));
+  interface_->CheckMediaFile(
+      kFileDecodeTimeInMS, std::move(file_),
+      base::Bind(&SafeAudioVideoChecker::OnCheckingFinished, this));
 }
 
 SafeAudioVideoChecker::~SafeAudioVideoChecker() {}
 
 void SafeAudioVideoChecker::OnCheckingFinished(bool valid) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::IO);
   if (state_ != STARTED_STATE)
     return;
   state_ = FINISHED_STATE;
 
+  interface_.reset();
   callback_.Run(valid ? base::File::FILE_OK :
                         base::File::FILE_ERROR_SECURITY);
 }
 
 void SafeAudioVideoChecker::OnProcessCrashed(int exit_code) {
   OnCheckingFinished(false);
 }
 
 bool SafeAudioVideoChecker::OnMessageReceived(const IPC::Message& message) {
-  bool handled = true;
-  IPC_BEGIN_MESSAGE_MAP(SafeAudioVideoChecker, message)
-    IPC_MESSAGE_HANDLER(ChromeUtilityHostMsg_CheckMediaFile_Finished,
-        OnCheckingFinished)
-    IPC_MESSAGE_UNHANDLED(handled = false)
-  IPC_END_MESSAGE_MAP()
-  return handled;
+  return false;
 }