Refactoring; switch the |filter| parameter into something that is more amenab...

chrome/common/win_util.h

 // Copyright (c) 2006-2008 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.
 
 #ifndef CHROME_COMMON_WIN_UTIL_H_
 #define CHROME_COMMON_WIN_UTIL_H_
 
 #include <objbase.h>
 
 #include <string>
@@ skipping 115 matching lines @@
 // zone on all downloads. This function is equivalent to OpenItemViaShell
 // without showing the zone warning dialog.
 bool OpenItemViaShellNoZoneCheck(const FilePath& full_path,
                                  bool ask_for_app);
 
 // Ask the user, via the Windows "Open With" dialog, for an application to use
 // to open the file specified by 'full_path'.
 // Returns 'true' on successful open, 'false' otherwise.
 bool OpenItemWithExternalApp(const std::wstring& full_path);
 
-std::wstring GetFileFilterFromPath(const std::wstring& file_name);
-
-// Returns a file filter whose description comes from the OS for the first file
-// extension in |extensions|. |extensions| is a semicolon separated list of
-// extensions. Each extension is specified as '*.foo' where foo is the
-// extension.
-std::wstring GetFileFilterFromExtensions(const std::wstring& extensions,
-                                         bool include_all_files);
+// Set up a filter for a Save/Open dialog, which will consist of |file_ext| file
+// extensions (internally separated by semicolons), |ext_desc| as the text
+// descriptions of the |file_ext| types (optional), and (optionally) the default
+// 'All Files' view. The purpose of the filter is to show only files of a
+// particular type in a Windows Save/Open dialog box. The resulting filter is
+// returned. The filters created here are:
+//   1. only files that have 'file_ext' as their extension
+//   2. all files (only added if 'include_all_files' is true)
+// Example:
+//   file_ext: { ".txt", ".htm;.html" }
+//   ext_desc: { "Text Document" }
+//   returned: "Text Document\0*.txt\0HTML Document\0.htm;.html\0"
+//             "All Files\0*.*\0\0" (in one big string)
+std::wstring FormatFilterForExtensions(
+    const std::vector<std::wstring>& file_ext,
+    const std::vector<std::wstring>& ext_desc,
+    bool include_all_files);
 
 // Prompt the user for location to save a file. 'suggested_name' is a full path
 // that gives the dialog box a hint as to how to initialize itself.
 // For example, a 'suggested_name' of:
 //   "C:\Documents and Settings\jojo\My Documents\picture.png"
 // will start the dialog in the "C:\Documents and Settings\jojo\My Documents\"
 // directory, and filter for .png file types.
 // 'owner' is the window to which the dialog box is modal, NULL for a modeless
 // dialog box.
 // On success,  returns true and 'final_name' contains the full path of the file
@@ skipping 120 matching lines @@
 
 // Returns the system set window title font.
 ChromeFont GetWindowTitleFont();
 
 // The thickness of an auto-hide taskbar in pixels.
 extern const int kAutoHideTaskbarThicknessPx;
 
 }  // namespace win_util
 
 #endif  // CHROME_COMMON_WIN_UTIL_H_