| Index: chrome/browser/safe_browsing/download_feedback_service.cc
|
| diff --git a/chrome/browser/safe_browsing/download_feedback_service.cc b/chrome/browser/safe_browsing/download_feedback_service.cc
|
| index 656c80402ce1ae8e200fe6a355bc33149169ce77..adebf92cbb9fffdccac79e6d0dbe97d6c64e79e9 100644
|
| --- a/chrome/browser/safe_browsing/download_feedback_service.cc
|
| +++ b/chrome/browser/safe_browsing/download_feedback_service.cc
|
| @@ -91,19 +91,19 @@ DownloadFeedbackService::~DownloadFeedbackService() {
|
| // static
|
| void DownloadFeedbackService::MaybeStorePingsForDownload(
|
| DownloadProtectionService::DownloadCheckResult result,
|
| + bool upload_requested,
|
| content::DownloadItem* download,
|
| const std::string& ping,
|
| const std::string& response) {
|
| - switch (result) {
|
| - case DownloadProtectionService::UNKNOWN:
|
| - case DownloadProtectionService::SAFE:
|
| - case DownloadProtectionService::DANGEROUS:
|
| - return;
|
| - case DownloadProtectionService::UNCOMMON:
|
| - case DownloadProtectionService::DANGEROUS_HOST:
|
| - case DownloadProtectionService::POTENTIALLY_UNWANTED:
|
| - break; // Fall through.
|
| - }
|
| + // We never upload SAFE files.
|
| + if (result == DownloadProtectionService::SAFE)
|
| + return;
|
| +
|
| + UMA_HISTOGRAM_BOOLEAN("SBDownloadFeedback.UploadRequestedByServer",
|
| + upload_requested);
|
| + if (!upload_requested)
|
| + return;
|
| +
|
| UMA_HISTOGRAM_COUNTS("SBDownloadFeedback.SizeEligibleKB",
|
| download->GetReceivedBytes() / 1024);
|
| if (download->GetReceivedBytes() > DownloadFeedback::kMaxUploadSize)
|
|
|