Comment #6 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=10246

------------------------------------------------------------------------
r10246 | [email protected] | 2009-02-23 18:30:04 -0800 (Mon, 23 Feb 2009)  
| 7 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/download/download_exe.cc?r1=10246&r2=10245
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/download/download_manager.cc?r1=10246&r2=10245
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/download/download_manager.h?r1=10246&r2=10245
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/download/download_util.h?r1=10246&r2=10245

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
------------------------------------------------------------------------


--
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to