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

chrome/browser/safe_browsing/chrome_cleaner/mock_chrome_cleaner_process_win.cc

Index: chrome/browser/safe_browsing/chrome_cleaner/mock_chrome_cleaner_process_win.cc
diff --git a/chrome/browser/safe_browsing/chrome_cleaner/mock_chrome_cleaner_process_win.cc b/chrome/browser/safe_browsing/chrome_cleaner/mock_chrome_cleaner_process_win.cc
index 97e83b6802acfbb6938b0f3fb68f848542cecacb..5a548b3f9caa7ad40fbbb40319a440aea03c43c1 100644
--- a/chrome/browser/safe_browsing/chrome_cleaner/mock_chrome_cleaner_process_win.cc
+++ b/chrome/browser/safe_browsing/chrome_cleaner/mock_chrome_cleaner_process_win.cc
@@ -201,10 +201,19 @@ void MockChromeCleanerProcess::SendScanResults(
         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)));
 }