Initial Implementation of Download Notification

chrome/browser/download/download_commands.h

+// 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.
+
+#ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_COMMANDS_H_
+#define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_COMMANDS_H_
+
+#include "base/strings/string16.h"
+
+#include "content/public/browser/download_item.h"
+#include "content/public/browser/page_navigator.h"
+#include "ui/gfx/image/image.h"
+
+class DownloadCommands {
+ public:
+  enum Command {
+    SHOW_IN_FOLDER = 1,    // Open a folder view window with the item selected.
+    OPEN_WHEN_COMPLETE,    // Open the download when it's finished.
+    ALWAYS_OPEN_TYPE,      // Default this file extension to always open.
+    PLATFORM_OPEN,         // Open using platform handler.
+    CANCEL,                // Cancel the download.
+    PAUSE,                 // Pause a download.
+    RESUME,                // Resume a download.
+    DISCARD,               // Discard the malicious download.
+    KEEP,                  // Keep the malicious download.
+    RETRY,                 // Retry the download.
+    LEARN_MORE_SCANNING,   // Show information about download scanning.
+    LEARN_MORE_INTERRUPTED,// Show information about interrupted downloads.
+  };
+
+  explicit DownloadCommands(content::DownloadItem* download_item);
+  virtual ~DownloadCommands() {}
+
+  gfx::Image GetCommandIcon(Command command);
+
+  base::string16 GetCommandString(Command command) const;

[asanka, 2015/02/13 21:17:05]

Could you explain the intended difference between these two methods?

[yoshiki, 2015/02/19 15:07:01]

(however I removed these methods)
We show longer messages for notification than ones for menu, hence there are 2
methods (one for menu, one for notifications) to get the strings.

+  base::string16 GetCommandStringForMenu(Command command) const;
+
+  bool IsCommandEnabled(Command command) const;
+  bool IsCommandChecked(Command command) const;
+  bool IsCommandVisible(Command command) const;
+  void ExecuteCommand(Command command, content::PageNavigator* navigator);
+
+ private:
+#if defined(OS_WIN) || defined(OS_LINUX) || \
+    (defined(OS_MACOSX) && !defined(OS_IOS))
+  bool IsDownloadPdf() const;
+  bool CanOpenPdfInSystemViewer() const;
+#endif
+
+  int GetCommandStringIdInternal(Command command, bool for_menu) const;
+  int GetCommandIconId(Command command);
+
+  int GetAlwaysOpenStringId() const;
+
+  content::DownloadItem* download_item_;
+};
+
+#endif  // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_COMMANDS_H_