Chrome Cleaner: Remove indirect base::FilePath mojo dependency.

chrome/browser/safe_browsing/chrome_cleaner/mock_chrome_cleaner_process_win.cc

 // Copyright 2017 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/safe_browsing/chrome_cleaner/mock_chrome_cleaner_process_win.h"
 
 #include <utility>
 
 #include "base/bind_helpers.h"
 #include "base/memory/ref_counted.h"
@@ skipping 183 matching lines @@
   chrome_prompt_ptr_->Bind(std::move(prompt_ptr_info));
 
   if (options_.crash_point() == CrashPoint::kAfterRequestSent) {
     // This task is posted to the IPC thread so that it will happen after the
     // request is sent to the parent process and before the response gets
     // handled on the IPC thread.
     base::SequencedTaskRunnerHandle::Get()->PostTask(
         FROM_HERE, base::Bind([]() { exit(kDeliberateCrashExitCode); }));
   }
 
+  std::vector<chrome_cleaner::mojom::FilePathPtr> files_to_delete;

[Fabio Tirelo, 2017/06/12 18:20:46]

This only needs to be done if condition in line 214 is true.

[proberge, 2017/06/12 19:02:53]

Line 214 looks like a pointer deref? Not a condition.

+  for (const base::FilePath& file_path : options_.files_to_delete()) {
+    auto data = reinterpret_cast<const uint16_t*>(file_path.value().c_str());
+    size_t data_size = file_path.value().size();
+    chrome_cleaner::mojom::FilePathPtr path =
+        chrome_cleaner::mojom::FilePath::New(
+            std::vector<uint16_t>(data, data + data_size));
+    files_to_delete.push_back(std::move(path));
+  }
+
   (*chrome_prompt_ptr_)
       ->PromptUser(
-          std::vector<base::FilePath>(options_.files_to_delete().begin(),
-                                      options_.files_to_delete().end()),
+          std::move(files_to_delete),
           base::BindOnce(&MockChromeCleanerProcess::PromptUserCallback,
                          base::Unretained(this), std::move(quit_closure)));
 }
 
 void MockChromeCleanerProcess::PromptUserCallback(
     base::OnceClosure quit_closure,
     PromptAcceptance prompt_acceptance) {
   delete chrome_prompt_ptr_;
   chrome_prompt_ptr_ = nullptr;
 
   received_prompt_acceptance_ = prompt_acceptance;
 
   if (options_.crash_point() == CrashPoint::kAfterResponseReceived)
     exit(kDeliberateCrashExitCode);
 
   std::move(quit_closure).Run();
 }
 
 }  // namespace safe_browsing