Comment #7 on issue 7455 by [email protected]: Dangerous download
warning not shown, if exe file extension is in caps
http://code.google.com/p/chromium/issues/detail?id=7455
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=10303
------------------------------------------------------------------------
r10303 | [email protected] | 2009-02-24 15:09:45 -0800 (Tue, 24 Feb
2009) | 13 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/branches/chrome_official_branch/src/chrome/browser/download/download_exe.cc?r1=10303&r2=10302
M
http://src.chromium.org/viewvc/chrome/branches/chrome_official_branch/src/chrome/browser/download/download_manager.cc?r1=10303&r2=10302
M
http://src.chromium.org/viewvc/chrome/branches/chrome_official_branch/src/chrome/browser/download/download_manager.h?r1=10303&r2=10302
M
http://src.chromium.org/viewvc/chrome/branches/chrome_official_branch/src/chrome/browser/download/download_util.h?r1=10303&r2=10302
Merge 10246 - Fixes issue with download shelf not warning on .exe files
Make download manager treat mixed-case extensions the same as lower-case
extensions when judging whether a file is executable.
Also, store known executable file extensions as narrow strings since they
are all ASCII anyway.
r=jcampan
BUG=7455
Review URL: http://codereview.chromium.org/29003
TBR=estade
Review URL: http://codereview.chromium.org/28096
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
