OLD | NEW |
---|---|
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "chrome/browser/download/download_shelf_context_menu.h" | 5 #include "chrome/browser/download/download_shelf_context_menu.h" |
6 | 6 |
7 #include "chrome/browser/browser_process.h" | 7 #include "chrome/browser/browser_process.h" |
8 #include "chrome/browser/download/download_crx_util.h" | 8 #include "chrome/browser/download/download_crx_util.h" |
9 #include "chrome/browser/download/download_item_model.h" | 9 #include "chrome/browser/download/download_item_model.h" |
10 #include "chrome/browser/download/download_prefs.h" | 10 #include "chrome/browser/download/download_prefs.h" |
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
122 prefs->DisableAutoOpenBasedOnExtension(path); | 122 prefs->DisableAutoOpenBasedOnExtension(path); |
123 break; | 123 break; |
124 } | 124 } |
125 case CANCEL: | 125 case CANCEL: |
126 download_item_->Cancel(true /* Cancelled by user */); | 126 download_item_->Cancel(true /* Cancelled by user */); |
127 break; | 127 break; |
128 case TOGGLE_PAUSE: | 128 case TOGGLE_PAUSE: |
129 // It is possible for the download to complete before the user clicks the | 129 // It is possible for the download to complete before the user clicks the |
130 // menu item, recheck if the download is in progress state before toggling | 130 // menu item, recheck if the download is in progress state before toggling |
131 // pause. | 131 // pause. |
132 if (download_item_->IsPartialDownload()) { | 132 if (download_item_->GetState() == DownloadItem::IN_PROGRESS) { |
133 if (download_item_->IsPaused()) | 133 if (download_item_->IsPaused()) |
134 download_item_->Resume(); | 134 download_item_->Resume(); |
135 else | 135 else |
136 download_item_->Pause(); | 136 download_item_->Pause(); |
137 } | 137 } |
138 break; | 138 break; |
139 case DISCARD: | 139 case DISCARD: |
140 download_item_->Remove(); | 140 download_item_->Remove(); |
141 break; | 141 break; |
142 case KEEP: | 142 case KEEP: |
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
176 | 176 |
177 bool DownloadShelfContextMenu::IsItemForCommandIdDynamic(int command_id) const { | 177 bool DownloadShelfContextMenu::IsItemForCommandIdDynamic(int command_id) const { |
178 return command_id == TOGGLE_PAUSE; | 178 return command_id == TOGGLE_PAUSE; |
179 } | 179 } |
180 | 180 |
181 string16 DownloadShelfContextMenu::GetLabelForCommandId(int command_id) const { | 181 string16 DownloadShelfContextMenu::GetLabelForCommandId(int command_id) const { |
182 switch (static_cast<ContextMenuCommands>(command_id)) { | 182 switch (static_cast<ContextMenuCommands>(command_id)) { |
183 case SHOW_IN_FOLDER: | 183 case SHOW_IN_FOLDER: |
184 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_SHOW); | 184 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_SHOW); |
185 case OPEN_WHEN_COMPLETE: | 185 case OPEN_WHEN_COMPLETE: |
186 if (download_item_ && | 186 if (download_item_ && download_item_->IsPartialDownload()) |
Randy Smith (Not in Mondays)
2013/06/05 20:21:44
Do we persist the open state across browser restar
asanka
2013/06/05 21:33:47
We persist the "opened" state, but not the "will o
| |
187 download_item_->GetState() == DownloadItem::IN_PROGRESS) | |
188 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_OPEN_WHEN_COMPLETE); | 187 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_OPEN_WHEN_COMPLETE); |
189 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_OPEN); | 188 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_OPEN); |
190 case ALWAYS_OPEN_TYPE: | 189 case ALWAYS_OPEN_TYPE: |
191 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_ALWAYS_OPEN_TYPE); | 190 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_ALWAYS_OPEN_TYPE); |
192 case CANCEL: | 191 case CANCEL: |
193 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_CANCEL); | 192 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_CANCEL); |
194 case TOGGLE_PAUSE: | 193 case TOGGLE_PAUSE: |
195 if (download_item_ && download_item_->IsPaused()) | 194 if (download_item_ && download_item_->IsPaused()) |
196 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_RESUME_ITEM); | 195 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_RESUME_ITEM); |
197 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_PAUSE_ITEM); | 196 return l10n_util::GetStringUTF16(IDS_DOWNLOAD_MENU_PAUSE_ITEM); |
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
292 malicious_download_menu_model_->AddItemWithStringId( | 291 malicious_download_menu_model_->AddItemWithStringId( |
293 DISCARD, IDS_DOWNLOAD_MENU_DISCARD); | 292 DISCARD, IDS_DOWNLOAD_MENU_DISCARD); |
294 malicious_download_menu_model_->AddItemWithStringId( | 293 malicious_download_menu_model_->AddItemWithStringId( |
295 KEEP, IDS_DOWNLOAD_MENU_KEEP); | 294 KEEP, IDS_DOWNLOAD_MENU_KEEP); |
296 malicious_download_menu_model_->AddSeparator(ui::NORMAL_SEPARATOR); | 295 malicious_download_menu_model_->AddSeparator(ui::NORMAL_SEPARATOR); |
297 malicious_download_menu_model_->AddItemWithStringId( | 296 malicious_download_menu_model_->AddItemWithStringId( |
298 LEARN_MORE_SCANNING, IDS_DOWNLOAD_MENU_LEARN_MORE_SCANNING); | 297 LEARN_MORE_SCANNING, IDS_DOWNLOAD_MENU_LEARN_MORE_SCANNING); |
299 | 298 |
300 return malicious_download_menu_model_.get(); | 299 return malicious_download_menu_model_.get(); |
301 } | 300 } |
OLD | NEW |