Setting CommandLabel 'IDS_DOWNLOAD_NOTIFICATION_LABEL_OPEN" for finished Download Notification.

chrome/browser/download/notification/download_item_notification.cc

Index: chrome/browser/download/notification/download_item_notification.cc
diff --git a/chrome/browser/download/notification/download_item_notification.cc b/chrome/browser/download/notification/download_item_notification.cc
index e2b58bfbce543c18e3fbfc93e9097e07863a4f81..b57699beb451f0bf4cdfd840c3f656660fc6d67d 100644
--- a/chrome/browser/download/notification/download_item_notification.cc
+++ b/chrome/browser/download/notification/download_item_notification.cc
@@ -682,7 +682,7 @@ base::string16 DownloadItemNotification::GetCommandLabel(
       if (item_ && !item_->IsDone())
         id = IDS_DOWNLOAD_STATUS_OPEN_WHEN_COMPLETE;
       else
-        id = IDS_DOWNLOAD_STATUS_OPEN_WHEN_COMPLETE;
+        id = IDS_DOWNLOAD_STATUS_OPEN;

[asanka, 2015/09/25 03:18:47]

yoshiki: Aren't these labels for the command buttons?

The text "Opening when complete" isn't grammatically correct for the open
button. The IDS_DOWNLOAD_STATUS_* strings are for indicating the status of the
download. If the user elects to open the download when it completes, then the
status says "Opening when complete" or "Opening when done". But that text isn't
suited for use as a command label.

A more appropriate string would be something closer to the
IDS_DOWNLOAD_MENU_OPEN_WHEN_COMPLETE and IDS_DOWNLOAD_MENU_OPEN.

[Deepak, 2015/09/25 16:09:33]

In DownloadShelfContextMenu::GetLabelForCommandId()
it is using IDS_DOWNLOAD_MENU_OPEN_WHEN_COMPLETE and IDS_DOWNLOAD_MENU_OPEN
But here all the labels have IDS_DOWNLOAD_MENU_* strings that is menu strings.
It appears that DownloadItemNotification::GetCommandLabel() is for non menu
items. So I have introduced IDS_DOWNLOAD_STATUS_OPEN, on similar lines.
This is as per my understanding. Please correct me if I misunderstood.
I would like to hear more on this from @yoshiki.
Thanks

[asanka, 2015/09/25 17:09:35]

_DOWNLOAD_STATUS_ is for download status strings. They shouldn't be used to
describe actions.

[Deepak, 2015/09/27 06:19:57]

I got your point. Changes done.
Thanks

       break;
     case DownloadCommands::PAUSE:
       // Only for non menu.