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

Side by Side Diff: chrome/browser/safe_browsing/download_protection_service.h

Issue 2128583005: DownloadProtection: Add more graceful handling of verdict=UNKNOWN (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // Helper class which handles communication with the SafeBrowsing servers for 5 // Helper class which handles communication with the SafeBrowsing servers for
6 // improved binary download protection. 6 // improved binary download protection.
7 7
8 #ifndef CHROME_BROWSER_SAFE_BROWSING_DOWNLOAD_PROTECTION_SERVICE_H_ 8 #ifndef CHROME_BROWSER_SAFE_BROWSING_DOWNLOAD_PROTECTION_SERVICE_H_
9 #define CHROME_BROWSER_SAFE_BROWSING_DOWNLOAD_PROTECTION_SERVICE_H_ 9 #define CHROME_BROWSER_SAFE_BROWSING_DOWNLOAD_PROTECTION_SERVICE_H_
10 10
(...skipping 169 matching lines...) Expand 10 before | Expand all | Expand 10 after
180 REASON_DOWNLOAD_NOT_SUPPORTED = 17, 180 REASON_DOWNLOAD_NOT_SUPPORTED = 17,
181 REASON_INVALID_RESPONSE_VERDICT = 18, 181 REASON_INVALID_RESPONSE_VERDICT = 18,
182 REASON_ARCHIVE_WITHOUT_BINARIES = 19, 182 REASON_ARCHIVE_WITHOUT_BINARIES = 19,
183 REASON_DOWNLOAD_DANGEROUS_HOST = 20, 183 REASON_DOWNLOAD_DANGEROUS_HOST = 20,
184 REASON_DOWNLOAD_POTENTIALLY_UNWANTED = 21, 184 REASON_DOWNLOAD_POTENTIALLY_UNWANTED = 21,
185 REASON_UNSUPPORTED_URL_SCHEME = 22, 185 REASON_UNSUPPORTED_URL_SCHEME = 22,
186 REASON_MANUAL_BLACKLIST = 23, 186 REASON_MANUAL_BLACKLIST = 23,
187 REASON_LOCAL_FILE = 24, 187 REASON_LOCAL_FILE = 24,
188 REASON_REMOTE_FILE = 25, 188 REASON_REMOTE_FILE = 25,
189 REASON_SAMPLED_UNSUPPORTED_FILE = 26, 189 REASON_SAMPLED_UNSUPPORTED_FILE = 26,
190 REASON_VERDICT_UNKNOWN = 27,
190 REASON_MAX // Always add new values before this one. 191 REASON_MAX // Always add new values before this one.
191 }; 192 };
192 193
193 private: 194 private:
194 class CheckClientDownloadRequest; 195 class CheckClientDownloadRequest;
195 class PPAPIDownloadRequest; 196 class PPAPIDownloadRequest;
196 friend class DownloadProtectionServiceTest; 197 friend class DownloadProtectionServiceTest;
197 friend class DownloadDangerPromptTest; 198 friend class DownloadDangerPromptTest;
198 199
199 FRIEND_TEST_ALL_PREFIXES(DownloadProtectionServiceTest, 200 FRIEND_TEST_ALL_PREFIXES(DownloadProtectionServiceTest,
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
300 std::set<std::string> manual_blacklist_hashes_; 301 std::set<std::string> manual_blacklist_hashes_;
301 302
302 // Rate of whitelisted downloads we sample to send out download ping. 303 // Rate of whitelisted downloads we sample to send out download ping.
303 double whitelist_sample_rate_; 304 double whitelist_sample_rate_;
304 305
305 DISALLOW_COPY_AND_ASSIGN(DownloadProtectionService); 306 DISALLOW_COPY_AND_ASSIGN(DownloadProtectionService);
306 }; 307 };
307 } // namespace safe_browsing 308 } // namespace safe_browsing
308 309
309 #endif // CHROME_BROWSER_SAFE_BROWSING_DOWNLOAD_PROTECTION_SERVICE_H_ 310 #endif // CHROME_BROWSER_SAFE_BROWSING_DOWNLOAD_PROTECTION_SERVICE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698