Index: chrome/common/safe_browsing/download_protection_util.cc |
diff --git a/chrome/common/safe_browsing/download_protection_util.cc b/chrome/common/safe_browsing/download_protection_util.cc |
index a5a4befa3696ddcb9c723f5d9d97621a6a6cf14f..ee09cd1e1fe28982e58a8a4778923b48729df34e 100644 |
--- a/chrome/common/safe_browsing/download_protection_util.cc |
+++ b/chrome/common/safe_browsing/download_protection_util.cc |
@@ -11,7 +11,6 @@ namespace safe_browsing { |
namespace download_protection_util { |
bool IsArchiveFile(const base::FilePath& file) { |
- // TODO(mattm): should .dmg be checked here instead of IsBinaryFile? |
return file.MatchesExtension(FILE_PATH_LITERAL(".zip")); |
} |
@@ -34,11 +33,6 @@ bool IsBinaryFile(const base::FilePath& file) { |
// Chrome extensions and android APKs are also reported. |
file.MatchesExtension(FILE_PATH_LITERAL(".crx")) || |
file.MatchesExtension(FILE_PATH_LITERAL(".apk")) || |
- // Mac extensions. |
- file.MatchesExtension(FILE_PATH_LITERAL(".dmg")) || |
- file.MatchesExtension(FILE_PATH_LITERAL(".pkg")) || |
- file.MatchesExtension(FILE_PATH_LITERAL(".osx")) || |
- file.MatchesExtension(FILE_PATH_LITERAL(".app")) || |
// Archives _may_ contain binaries, we'll check in ExtractFileFeatures. |
IsArchiveFile(file)); |
} |
@@ -54,11 +48,6 @@ ClientDownloadRequest::DownloadType GetDownloadType( |
// the pingback if we find an executable inside the zip archive. |
else if (file.MatchesExtension(FILE_PATH_LITERAL(".zip"))) |
return ClientDownloadRequest::ZIPPED_EXECUTABLE; |
- else if (file.MatchesExtension(FILE_PATH_LITERAL(".dmg")) || |
- file.MatchesExtension(FILE_PATH_LITERAL(".pkg")) || |
- file.MatchesExtension(FILE_PATH_LITERAL(".osx")) || |
- file.MatchesExtension(FILE_PATH_LITERAL(".app"))) |
- return ClientDownloadRequest::MAC_EXECUTABLE; |
return ClientDownloadRequest::WIN_EXECUTABLE; |
} |