Opt out of download feedback from settings

chrome/browser/ui/views/download/download_item_view.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/download/download_item_view.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 73 matching lines @@
 // How long we keep the item disabled after the user clicked it to open the
 // downloaded item.
 static const int kDisabledOnOpenDuration = 3000;
 
 // Darken light-on-dark download status text by 20% before drawing, thus
 // creating a "muted" version of title text for both dark-on-light and
 // light-on-dark themes.
 static const double kDownloadItemLuminanceMod = 0.8;
 
 using content::DownloadItem;
+using safe_browsing::DownloadFeedbackService;
 
 DownloadItemView::DownloadItemView(DownloadItem* download_item,
     DownloadShelfView* parent)
   : warning_icon_(NULL),
     shelf_(parent),
     status_text_(l10n_util::GetStringUTF16(IDS_DOWNLOAD_STATUS_STARTING)),
     body_state_(NORMAL),
     drop_down_state_(NORMAL),
     mode_(NORMAL_MODE),
     progress_angle_(DownloadShelf::kStartAngleDegrees),
@@ skipping 446 matching lines @@
   // WARNING: all end states after this point delete |this|.
   DCHECK_EQ(discard_button_, sender);
   if (model_.IsMalicious()) {
     UMA_HISTOGRAM_LONG_TIMES("clickjacking.dismiss_download", warning_duration);
     shelf_->RemoveDownloadView(this);
     return;
   }
   UMA_HISTOGRAM_LONG_TIMES("clickjacking.discard_download", warning_duration);
   if (model_.ShouldAllowDownloadFeedback() &&
       !shelf_->browser()->profile()->IsOffTheRecord()) {
-    DownloadFeedbackDialogView::DownloadReportingStatus pref_value =
-        static_cast<DownloadFeedbackDialogView::DownloadReportingStatus>(
+    DownloadFeedbackService::DownloadReportingStatus pref_value =
+        static_cast<DownloadFeedbackService::DownloadReportingStatus>(
             shelf_->browser()->profile()->GetPrefs()->GetInteger(
                 prefs::kSafeBrowsingDownloadReportingEnabled));
     switch (pref_value) {
-      case DownloadFeedbackDialogView::kDialogNotYetShown:
+      case DownloadFeedbackService::kDialogNotYetShown:
         DownloadFeedbackDialogView::Show(
             shelf_->get_parent()->GetNativeWindow(),
             shelf_->browser()->profile(),
             base::Bind(
                 &DownloadItemView::PossiblySubmitDownloadToFeedbackService,
                 weak_ptr_factory_.GetWeakPtr()));
         break;
 
-      case DownloadFeedbackDialogView::kDownloadReportingEnabled:
-      case DownloadFeedbackDialogView::kDownloadReportingDisabled:
+      case DownloadFeedbackService::kDownloadReportingEnabled:
+      case DownloadFeedbackService::kDownloadReportingDisabled:
         PossiblySubmitDownloadToFeedbackService(pref_value);
         break;
 
-      case DownloadFeedbackDialogView::kMaxValue:
+      case DownloadFeedbackService::kMaxValue:
         NOTREACHED();
     }
     return;
   }
   download()->Remove();
 }
 
 void DownloadItemView::AnimationProgressed(const gfx::Animation* animation) {
   // We don't care if what animation (body button/drop button/complete),
   // is calling back, as they all have to go through the same paint call.
@@ skipping 341 matching lines @@
       download());
   // WARNING: we are deleted at this point.  Don't access 'this'.
   return true;
 #else
   NOTREACHED();
   return false;
 #endif
 }
 
 void DownloadItemView::PossiblySubmitDownloadToFeedbackService(
-    DownloadFeedbackDialogView::DownloadReportingStatus status) {
-  if (status != DownloadFeedbackDialogView::kDownloadReportingEnabled ||
+    DownloadFeedbackService::DownloadReportingStatus status) {
+  if (status != DownloadFeedbackService::kDownloadReportingEnabled ||
       !SubmitDownloadToFeedbackService()) {
     download()->Remove();
   }
   // WARNING: 'this' is deleted at this point. Don't access 'this'.
 }
 
 void DownloadItemView::LoadIcon() {
   IconManager* im = g_browser_process->icon_manager();
   last_download_item_path_ = download()->GetTargetFilePath();
   im->LoadIcon(last_download_item_path_,
@@ skipping 196 matching lines @@
 
   TooltipTextChanged();
 }
 
 void DownloadItemView::ShowWarningDialog() {
   DCHECK(mode_ != DANGEROUS_MODE && mode_ != MALICIOUS_MODE);
   time_download_warning_shown_ = base::Time::Now();
   content::DownloadDangerType danger_type = download()->GetDangerType();
   RecordDangerousDownloadWarningShown(danger_type);
 #if defined(FULL_SAFE_BROWSING)
-  if (model_.ShouldAllowDownloadFeedback()) {
-    safe_browsing::DownloadFeedbackService::RecordEligibleDownloadShown(
-        danger_type);
-  }
+  if (model_.ShouldAllowDownloadFeedback())
+    DownloadFeedbackService::RecordEligibleDownloadShown(danger_type);
 #endif
   mode_ = model_.MightBeMalicious() ? MALICIOUS_MODE : DANGEROUS_MODE;
 
   body_state_ = NORMAL;
   drop_down_state_ = NORMAL;
   if (mode_ == DANGEROUS_MODE) {
     save_button_ = new views::LabelButton(
         this, model_.GetWarningConfirmButtonText());
     save_button_->SetStyle(views::Button::STYLE_BUTTON);
     AddChildView(save_button_);
@@ skipping 176 matching lines @@
 void DownloadItemView::AnimateStateTransition(State from, State to,
                                               gfx::SlideAnimation* animation) {
   if (from == NORMAL && to == HOT) {
     animation->Show();
   } else if (from == HOT && to == NORMAL) {
     animation->Hide();
   } else if (from != to) {
     animation->Reset((to == HOT) ? 1.0 : 0.0);
   }
 }