[Download Notification] Use NotificationUIManager instead of MessageCenter

chrome/browser/download/download_commands.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/download_commands.h"
 
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/download/download_crx_util.h"
 #include "chrome/browser/download/download_item_model.h"
 #include "chrome/browser/download/download_prefs.h"
@@ skipping 24 matching lines @@
       return IDR_DOWNLOAD_NOTIFICATION_MENU_PAUSE;
     case RESUME:
       return IDR_DOWNLOAD_NOTIFICATION_MENU_RESUME;
     case SHOW_IN_FOLDER:
       return IDR_DOWNLOAD_NOTIFICATION_MENU_FOLDER;
     case RETRY:
     case KEEP:
       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;
   }
   NOTREACHED();
   return -1;
 }
 
 gfx::Image DownloadCommands::GetCommandIcon(Command command) {
   ResourceBundle& bundle = ResourceBundle::GetSharedInstance();
@@ skipping 173 matching lines @@
     is_adobe_pdf_reader_up_to_date =
         DownloadTargetDeterminer::IsAdobeReaderUpToDate();
   }
   return IsDownloadPdf() &&
          (IsAdobeReaderDefaultPDFViewer() ? is_adobe_pdf_reader_up_to_date
                                           : true);
 #elif defined(OS_MACOSX) || defined(OS_LINUX)
   return IsDownloadPdf();
 #endif
 }