Revive experiment to isolate shell operations.

chrome/utility/shell_handler_impl_win.cc

 // Copyright 2016 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/utility/shell_handler_impl_win.h"
 
 #include <shldisp.h>
 
 #include "base/files/file_enumerator.h"
 #include "base/files/file_path.h"
 #include "base/path_service.h"
 #include "base/scoped_native_library.h"
 #include "base/win/scoped_bstr.h"
 #include "base/win/scoped_com_initializer.h"
 #include "base/win/scoped_comptr.h"
 #include "base/win/scoped_variant.h"
 #include "base/win/shortcut.h"
 #include "chrome/installer/util/install_util.h"
 #include "content/public/utility/utility_thread.h"
+#include "ui/base/win/open_file_name_win.h"
 
 namespace {
 
 // This class checks if the current executable is pinned to the taskbar. It also
 // keeps track of the errors that occurs that prevents it from getting a result.
 class IsPinnedToTaskbarHelper {
  public:
   IsPinnedToTaskbarHelper() = default;
 
   // Returns true if the current executable is pinned to the taskbar.
@@ skipping 185 matching lines @@
     : binding_(this, std::move(request)) {}
 
 ShellHandlerImpl::~ShellHandlerImpl() = default;
 
 void ShellHandlerImpl::IsPinnedToTaskbar(
     const IsPinnedToTaskbarCallback& callback) {
   IsPinnedToTaskbarHelper helper;
   bool is_pinned_to_taskbar = helper.GetResult();
   callback.Run(!helper.error_occured(), is_pinned_to_taskbar);
 }
+
+void ShellHandlerImpl::DoGetOpenFileName(
+    uint64_t owner,
+    uint32_t flags,
+    const std::vector<std::tuple<base::string16, base::string16>>& filters,
+    const base::FilePath& initial_directory,
+    const base::FilePath& filename,
+    const DoGetOpenFileNameCallback& callback) {
+  ui::win::OpenFileName open_file_name(reinterpret_cast<HWND>(owner), flags);
+  open_file_name.SetInitialSelection(initial_directory, filename);
+  open_file_name.SetFilters(filters);
+
+  base::FilePath directory;
+  std::vector<base::FilePath> filenames;
+
+  if (::GetOpenFileName(open_file_name.GetOPENFILENAME()))
+    open_file_name.GetResult(&directory, &filenames);
+
+  if (filenames.size()) {

[sky, 2016/07/08 15:28:21]

nit: no {}, also !empty()

[Patrick Monette, 2016/07/08 19:36:10]

Done.

+    callback.Run(directory, filenames);
+  } else {
+    callback.Run(base::FilePath(), std::vector<base::FilePath>());
+  }
+}
+
+void ShellHandlerImpl::DoGetSaveFileName(
+    uint64_t owner,
+    uint32_t flags,
+    const std::vector<std::tuple<base::string16, base::string16>>& filters,
+    uint32_t one_based_filter_index,
+    const base::FilePath& initial_directory,
+    const base::FilePath& suggested_filename,
+    const base::FilePath& default_extension,
+    const DoGetSaveFileNameCallback& callback) {
+  ui::win::OpenFileName open_file_name(reinterpret_cast<HWND>(owner), flags);
+  open_file_name.SetInitialSelection(initial_directory, suggested_filename);
+  open_file_name.SetFilters(filters);
+  open_file_name.GetOPENFILENAME()->nFilterIndex = one_based_filter_index;
+  open_file_name.GetOPENFILENAME()->lpstrDefExt =
+      default_extension.value().c_str();
+
+  if (::GetSaveFileName(open_file_name.GetOPENFILENAME())) {
+    callback.Run(base::FilePath(open_file_name.GetOPENFILENAME()->lpstrFile),
+                 open_file_name.GetOPENFILENAME()->nFilterIndex);
+    return;
+  }
+
+  // Zero means the dialog was closed, otherwise we had an error.
+  DWORD error_code = ::CommDlgExtendedError();
+  if (error_code != 0)
+    NOTREACHED() << "GetSaveFileName failed with code: " << error_code;
+
+  callback.Run(base::FilePath(), 0);
+}