[Download Notification] Use NotificationUIManager instead of MessageCenter

chrome/browser/download/download_commands.cc

Index: chrome/browser/download/download_commands.cc
diff --git a/chrome/browser/download/download_commands.cc b/chrome/browser/download/download_commands.cc
index 15b6d6bac73385ceffd2be5711d07b9900086de9..f4ca401501fa888de15dbc2d6b3187d47de25c41 100644
--- a/chrome/browser/download/download_commands.cc
+++ b/chrome/browser/download/download_commands.cc
@@ -42,10 +42,12 @@ int DownloadCommands::GetCommandIconId(Command command) {
       return IDR_DOWNLOAD_NOTIFICATION_MENU_DOWNLOAD;
     case DISCARD:
       return IDR_DOWNLOAD_NOTIFICATION_MENU_DELETE;
+    case CANCEL:
+      // TODO(yoshiki): This is temporary. Change the asset when prepared.

[Randy Smith (Not in Mondays), 2015/04/10 21:43:19]

Could you give me context for this comment, preferably by fleshing it out?  If I
ran into this in landed source, I would have no idea what it meant.

[yoshiki, 2015/04/13 15:51:53]

I changed the context in the comment. The Download Notification feature is
currently under development and behind the flag.

Currently, we are waiting for the proper asset images from the UX guys.
Meanwhile we'd like to use temporary images to develop and improve the
functionality. I will replace the temporary with them soon after I get them.

+      return IDR_DOWNLOAD_NOTIFICATION_MENU_DELETE;
     case OPEN_WHEN_COMPLETE:
     case ALWAYS_OPEN_TYPE:
     case PLATFORM_OPEN:
-    case CANCEL:
     case LEARN_MORE_SCANNING:
     case LEARN_MORE_INTERRUPTED:
       return -1;