summaryrefslogtreecommitdiffstats
path: root/chrome/common
diff options
context:
space:
mode:
authoravi@chromium.org <avi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-04-22 17:21:40 +0000
committeravi@chromium.org <avi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-04-22 17:21:40 +0000
commit35c79d30ce5c7a199004d548286aca7338e8ca10 (patch)
treebc0f6d0ae5457d08d4344d367b1d28324ec8b249 /chrome/common
parent00462ad45125e34e27399a53830f5f56d86d1d51 (diff)
downloadchromium_src-35c79d30ce5c7a199004d548286aca7338e8ca10.zip
chromium_src-35c79d30ce5c7a199004d548286aca7338e8ca10.tar.gz
chromium_src-35c79d30ce5c7a199004d548286aca7338e8ca10.tar.bz2
Properly construct shell filters (forgetting the asterisk means it works but doesn't show existing files).
BUG=10734 Review URL: http://codereview.chromium.org/93019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14202 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/common')
-rw-r--r--chrome/common/win_util.cc2
-rw-r--r--chrome/common/win_util.h4
2 files changed, 3 insertions, 3 deletions
diff --git a/chrome/common/win_util.cc b/chrome/common/win_util.cc
index 480355b..fa3ce49 100644
--- a/chrome/common/win_util.cc
+++ b/chrome/common/win_util.cc
@@ -277,7 +277,7 @@ bool SaveFileAs(HWND owner,
const std::wstring& suggested_name,
std::wstring* final_name) {
std::wstring file_ext = file_util::GetFileExtensionFromPath(suggested_name);
- file_ext.insert(L'.', 0);
+ file_ext.insert(0, L"*.");
std::wstring filter = FormatFilterForExtensions(
std::vector<std::wstring>(1, file_ext),
std::vector<std::wstring>(),
diff --git a/chrome/common/win_util.h b/chrome/common/win_util.h
index 679f405..e838e4d 100644
--- a/chrome/common/win_util.h
+++ b/chrome/common/win_util.h
@@ -142,9 +142,9 @@ bool OpenItemWithExternalApp(const std::wstring& full_path);
// 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" }
+// file_ext: { "*.txt", "*.htm;*.html" }
// ext_desc: { "Text Document" }
-// returned: "Text Document\0*.txt\0HTML Document\0.htm;.html\0"
+// returned: "Text Document\0*.txt\0HTML Document\0*.htm;*.html\0"
// "All Files\0*.*\0\0" (in one big string)
// If a description is not provided for a file extension, it will be retrieved
// from the registry. If the file extension does not exist in the registry, it