Experimentally isolate GetOpenFileName in a utility process.

ui/base/win/open_file_name_win.cc

 // Copyright (c) 2014 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 "ui/base/win/open_file_name_win.h"
 
 #include "base/files/file_path.h"
 #include "base/strings/string_util.h"
 
 namespace ui {
 namespace win {
 
 OpenFileName::OpenFileName(HWND parent_window, DWORD flags) {
   ::ZeroMemory(&openfilename_, sizeof(openfilename_));
   openfilename_.lStructSize = sizeof(openfilename_);
 
   // According to http://support.microsoft.com/?scid=kb;en-us;222003&x=8&y=12,
   // The lpstrFile Buffer MUST be NULL Terminated.
   filename_buffer_[0] = 0;
   openfilename_.lpstrFile = filename_buffer_;
   openfilename_.nMaxFile = arraysize(filename_buffer_);
 
   openfilename_.Flags = flags;
   openfilename_.hwndOwner = parent_window;
 }
 
 OpenFileName::~OpenFileName() {
 }
 
+void OpenFileName::SetFilters(
+    const std::vector<Tuple2<base::string16, base::string16> >& filters) {
+  openfilename_.lpstrFilter = NULL;
+  filter_buffer_.clear();
+  if (filters.empty())
+    return;
+  for (std::vector<Tuple2<base::string16, base::string16> >::const_iterator
+           it = filters.begin();
+       it != filters.end();
+       ++it) {
+    filter_buffer_.append(it->a);
+    filter_buffer_.push_back(0);
+    filter_buffer_.append(it->b);
+    filter_buffer_.push_back(0);
+  }
+  filter_buffer_.push_back(0);
+  openfilename_.lpstrFilter = filter_buffer_.c_str();
+}
+
 void OpenFileName::SetInitialSelection(const base::FilePath& initial_directory,
                                        const base::FilePath& initial_filename) {
   // First reset to the default case.
   // According to http://support.microsoft.com/?scid=kb;en-us;222003&x=8&y=12,
   // The lpstrFile Buffer MUST be NULL Terminated.
   filename_buffer_[0] = 0;
   openfilename_.lpstrFile = filename_buffer_;
   openfilename_.nMaxFile = arraysize(filename_buffer_);
   openfilename_.lpstrInitialDir = NULL;
   initial_directory_buffer_.clear();
@@ skipping 40 matching lines @@
     // filenames.
     *directory = (*filenames)[0];
     filenames->erase(filenames->begin());
   }
 }
 
 OPENFILENAME* OpenFileName::Get() {
   return &openfilename_;
 }
 
+const OPENFILENAME* OpenFileName::Get() const {
+  return &openfilename_;

[sky, 2014/07/31 20:47:46]

nit: this is trivial enough that I would inline it.

[erikwright (departed), 2014/08/01 13:53:15]

Done.

+}
+
+// static
+void OpenFileName::SetResult(const base::FilePath& directory,
+                             const std::vector<base::FilePath>& filenames,
+                             OPENFILENAME* openfilename) {
+  base::string16 filename_value;
+  if (filenames.size() == 1) {
+    filename_value = directory.Append(filenames[0]).value();
+  } else {
+    filename_value = directory.value();
+    filename_value.push_back(0);
+    for (std::vector<base::FilePath>::const_iterator it = filenames.begin();
+         it != filenames.end();
+         ++it) {
+      filename_value.append(it->value());
+      filename_value.push_back(0);
+    }
+  }
+  if (filename_value.size() + 1 < openfilename->nMaxFile) {

[sky, 2014/07/31 20:47:46]

What about the else case here?

[erikwright (departed), 2014/08/01 13:53:15]

Done. Also added a test case.

+    // Because the result has embedded nulls, we must memcpy.
+    memcpy(openfilename->lpstrFile,
+           filename_value.c_str(),
+           (filename_value.size() + 1) * sizeof(filename_value[0]));
+  }
+}
+
+// static
+std::vector<Tuple2<base::string16, base::string16> > OpenFileName::GetFilters(
+    const OPENFILENAME* openfilename) {
+  std::vector<Tuple2<base::string16, base::string16> > filters;
+
+  const base::char16* name = openfilename->lpstrFilter;

[sky, 2014/07/31 20:47:46]

Assigning 'name' to 'filter' is rather confusing.

[erikwright (departed), 2014/08/01 13:53:15]

Done.

+  if (!name)
+    return filters;
+
+  while (*name) {
+    const base::char16* name_end = name;
+    while (*name_end) {

[sky, 2014/07/31 20:47:46]

nit: no {} here and 154

[erikwright (departed), 2014/08/01 13:53:15]

Done.

+      ++name_end;
+    }
+    const base::char16* value = name_end + 1;
+    const base::char16* value_end = value;
+    while (*value_end) {
+      ++value_end;
+    }
+    filters.push_back(MakeTuple(base::string16(name, name_end),
+                                base::string16(value, value_end)));
+    name = value_end + 1;
+  }
+
+  return filters;
+}
+
 }  // namespace win
 }  // namespace ui