Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1827)

Unified Diff: chrome/browser/download/download_exe.cc

Issue 29003: Make download manager treat mixed-case extensions the same as lower-case exte... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/download/download_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/download/download_exe.cc
===================================================================
--- chrome/browser/download/download_exe.cc (revision 10233)
+++ chrome/browser/download/download_exe.cc (working copy)
@@ -52,92 +52,92 @@
*
* ***** END LICENSE BLOCK ***** */
-static const wchar_t* const g_executables[] = {
- L"ad",
- L"ade",
- L"adp",
- L"app",
- L"application",
- L"asp",
- L"asx",
- L"bas",
- L"bat",
- L"chm",
- L"cmd",
- L"com",
- L"cpl",
- L"crt",
- L"dll",
- L"exe",
- L"fxp",
- L"hlp",
- L"hta",
- L"htm",
- L"html",
- L"inf",
- L"ins",
- L"isp",
- L"jar",
- L"js",
- L"jse",
- L"lnk",
- L"mad",
- L"maf",
- L"mag",
- L"mam",
- L"maq",
- L"mar",
- L"mas",
- L"mat",
- L"mau",
- L"mav",
- L"maw",
- L"mda",
- L"mdb",
- L"mde",
- L"mdt",
- L"mdw",
- L"mdz",
- L"msc",
- L"msh",
- L"mshxml",
- L"msi",
- L"msp",
- L"mst",
- L"ops",
- L"pcd",
- L"pif",
- L"plg",
- L"prf",
- L"prg",
- L"pst",
- L"reg",
- L"scf",
- L"scr",
- L"sct",
- L"shb",
- L"shs",
- L"shtm",
- L"shtml",
- L"url",
- L"vb",
- L"vbe",
- L"vbs",
- L"vsd",
- L"vsmacros",
- L"vss",
- L"vst",
- L"vsw",
- L"ws",
- L"wsc",
- L"wsf",
- L"wsh",
- L"xht",
- L"xhtm",
- L"xhtml"
+static const char* const g_executables[] = {
+ "ad",
+ "ade",
+ "adp",
+ "app",
+ "application",
+ "asp",
+ "asx",
+ "bas",
+ "bat",
+ "chm",
+ "cmd",
+ "com",
+ "cpl",
+ "crt",
+ "dll",
+ "exe",
+ "fxp",
+ "hlp",
+ "hta",
+ "htm",
+ "html",
+ "inf",
+ "ins",
+ "isp",
+ "jar",
+ "js",
+ "jse",
+ "lnk",
+ "mad",
+ "maf",
+ "mag",
+ "mam",
+ "maq",
+ "mar",
+ "mas",
+ "mat",
+ "mau",
+ "mav",
+ "maw",
+ "mda",
+ "mdb",
+ "mde",
+ "mdt",
+ "mdw",
+ "mdz",
+ "msc",
+ "msh",
+ "mshxml",
+ "msi",
+ "msp",
+ "mst",
+ "ops",
+ "pcd",
+ "pif",
+ "plg",
+ "prf",
+ "prg",
+ "pst",
+ "reg",
+ "scf",
+ "scr",
+ "sct",
+ "shb",
+ "shs",
+ "shtm",
+ "shtml",
+ "url",
+ "vb",
+ "vbe",
+ "vbs",
+ "vsd",
+ "vsmacros",
+ "vss",
+ "vst",
+ "vsw",
+ "ws",
+ "wsc",
+ "wsf",
+ "wsh",
+ "xht",
+ "xhtm",
+ "xhtml"
};
-void InitializeExeTypes(std::set<std::wstring>* exe_extensions) {
+void InitializeExeTypes(std::set<std::string>* exe_extensions) {
DCHECK(exe_extensions);
for (size_t i = 0; i < arraysize(g_executables); ++i)
exe_extensions->insert(g_executables[i]);
« no previous file with comments | « no previous file | chrome/browser/download/download_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698