Add UMA metrics for the frequency of closing dangerous download notifications.

chrome/browser/download/notification/download_item_notification.cc

 // Copyright 2015 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/download/notification/download_item_notification.h"
 
 #include "base/files/file_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/download/download_crx_util.h"
@@ skipping 136 matching lines @@
       content::RecordAction(
           UserMetricsAction("DownloadNotification.Button_Pause"));
       break;
     case DownloadCommands::RESUME:
       content::RecordAction(
           UserMetricsAction("DownloadNotification.Button_Resume"));
       break;
   }
 }
 
+struct DownloadItemNotificationCommandEntry {
+  DownloadCommands::Command command;
+  int id;
+};
+
+DownloadItemNotificationCommandEntry kDownloadNotificationCommand[] = {
+    {DownloadCommands::OPEN_WHEN_COMPLETE,
+     IDS_DOWNLOAD_STATUS_OPEN_WHEN_COMPLETE},
+    {DownloadCommands::PAUSE, IDS_DOWNLOAD_LINK_PAUSE},
+    {DownloadCommands::RESUME, IDS_DOWNLOAD_LINK_RESUME},
+    {DownloadCommands::SHOW_IN_FOLDER, IDS_DOWNLOAD_LINK_SHOW},
+    {DownloadCommands::DISCARD, IDS_DISCARD_DOWNLOAD},
+    {DownloadCommands::KEEP, IDS_CONFIRM_DOWNLOAD},
+    {DownloadCommands::CANCEL, IDS_DOWNLOAD_LINK_CANCEL},
+    {DownloadCommands::LEARN_MORE_SCANNING,
+     IDS_DOWNLOAD_LINK_LEARN_MORE_SCANNING},
+};
+
 }  // anonymous namespace
 
 DownloadItemNotification::DownloadItemNotification(
     content::DownloadItem* item,
     DownloadNotificationManagerForProfile* manager)
     : item_(item),
       weak_factory_(this) {
   ui::ResourceBundle& bundle = ui::ResourceBundle::GetSharedInstance();
 
   message_center::RichNotificationData data;
@@ skipping 25 matching lines @@
     // Dangerous notifications don't have a click handler.
     return false;
   }
   return true;
 }
 
 void DownloadItemNotification::OnNotificationClose() {
   visible_ = false;
 
   if (item_ && item_->IsDangerous() && !item_->IsDone()) {
-    // TODO(yoshiki): Add metrics.
+    content::RecordAction(
+        UserMetricsAction("DownloadNotification.Close_Dangerous"));
     item_->Cancel(true /* by_user */);
     return;
   }
 
   if (image_decode_status_ == IN_PROGRESS) {
     image_decode_status_ = NOT_STARTED;
     ImageDecoder::Cancel(this);
   }
 }
 
@@ skipping 423 matching lines @@
           IDS_DOWNLOAD_STATUS_DOWNLOAD_FAILED_TITLE, file_name);
       break;
     case content::DownloadItem::MAX_DOWNLOAD_STATE:
       NOTREACHED();
   }
   return title_text;
 }
 
 base::string16 DownloadItemNotification::GetCommandLabel(
     DownloadCommands::Command command) const {
-  int id = -1;
-  switch (command) {
-    case DownloadCommands::OPEN_WHEN_COMPLETE:
-      if (item_ && !item_->IsDone())
-        id = IDS_DOWNLOAD_STATUS_OPEN_WHEN_COMPLETE;
-      else
-        id = IDS_DOWNLOAD_STATUS_OPEN_WHEN_COMPLETE;
-      break;
-    case DownloadCommands::PAUSE:
-      // Only for non menu.
-      id = IDS_DOWNLOAD_LINK_PAUSE;
-      break;
-    case DownloadCommands::RESUME:
-      // Only for non menu.
-      id = IDS_DOWNLOAD_LINK_RESUME;
-      break;
-    case DownloadCommands::SHOW_IN_FOLDER:
-      id = IDS_DOWNLOAD_LINK_SHOW;
-      break;
-    case DownloadCommands::DISCARD:
-      id = IDS_DISCARD_DOWNLOAD;
-      break;
-    case DownloadCommands::KEEP:
-      id = IDS_CONFIRM_DOWNLOAD;
-      break;
-    case DownloadCommands::CANCEL:
-      id = IDS_DOWNLOAD_LINK_CANCEL;
-      break;
-    case DownloadCommands::LEARN_MORE_SCANNING:
-      id = IDS_DOWNLOAD_LINK_LEARN_MORE_SCANNING;
-      break;
-    case DownloadCommands::ALWAYS_OPEN_TYPE:
-    case DownloadCommands::PLATFORM_OPEN:
-    case DownloadCommands::LEARN_MORE_INTERRUPTED:
-      // Only for menu.
-      NOTREACHED();
-      return base::string16();
+  for (const DownloadItemNotificationCommandEntry& entry :
+       kDownloadNotificationCommand) {
+    if (entry.command == command) {
+      return l10n_util::GetStringUTF16(entry.id);
+    }
   }
-  CHECK(id != -1);
-  return l10n_util::GetStringUTF16(id);
+  NOTREACHED();
+  return base::string16();
 }
 
 base::string16 DownloadItemNotification::GetWarningStatusString() const {
   // Should only be called if IsDangerous().
   DCHECK(item_->IsDangerous());
   base::string16 elided_filename =
       item_->GetFileNameToReportUser().LossyDisplayName();
   switch (item_->GetDangerType()) {
     case content::DOWNLOAD_DANGER_TYPE_DANGEROUS_URL: {
       return l10n_util::GetStringUTF16(IDS_PROMPT_MALICIOUS_DOWNLOAD_URL);
@@ skipping 157 matching lines @@
                                                     profile_id);
 
   message_center::NotificationList::Notifications visible_notifications =
       g_browser_process->message_center()->GetVisibleNotifications();
   for (const auto& notification : visible_notifications) {
     if (notification->id() == notification_id_in_message_center)
       return true;
   }
   return false;
 }