Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6043)

Unified Diff: chrome/browser/ui/cocoa/download/download_shelf_controller.mm

Issue 966983002: downloads: clicking "remove" on chrome://downloads should also hide shelf item. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/download/download_shelf_controller.mm
diff --git a/chrome/browser/ui/cocoa/download/download_shelf_controller.mm b/chrome/browser/ui/cocoa/download/download_shelf_controller.mm
index ebebb03f18c96560117ed7c28b07ca535c560912..29b6398c091401c2fa3569578719eaf8d389240a 100644
--- a/chrome/browser/ui/cocoa/download/download_shelf_controller.mm
+++ b/chrome/browser/ui/cocoa/download/download_shelf_controller.mm
@@ -309,6 +309,17 @@ const NSSize kHoverCloseButtonDefaultSize = { 18, 18 };
[self add:controller.get()];
}
+- (void)removeDownloadItem:(DownloadItem*)downloadItem {
+ for (NSUInteger i = 0; i < [downloadItemControllers_ count]; ++i) {
+ DownloadItemController* itemController =
+ [downloadItemControllers_ objectAtIndex:i];
+ if ([itemController download] == downloadItem) {
+ [self remove:itemController];
+ return;
+ }
+ }
+}
+
- (void)add:(DownloadItemController*)controller {
DCHECK([NSThread isMainThread]);
[self cancelAutoClose];

Powered by Google App Engine
This is Rietveld 408576698