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

Side by Side Diff: chrome/browser/ui/cocoa/download/download_shelf_controller.h

Issue 209613002: Download shelf autohides on showing in shell, just same as regular open Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added browser and unit tests. Renamed 'UserActed' to 'OpenedOrShown'. Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
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 #import <Cocoa/Cocoa.h> 5 #import <Cocoa/Cocoa.h>
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/mac/scoped_nsobject.h" 9 #include "base/mac/scoped_nsobject.h"
10 #import "chrome/browser/ui/cocoa/has_weak_browser_pointer.h" 10 #import "chrome/browser/ui/cocoa/has_weak_browser_pointer.h"
(...skipping 102 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 // Add a new download item to the leftmost position of the download shelf. The 113 // Add a new download item to the leftmost position of the download shelf. The
114 // item should not have been already added to this shelf. 114 // item should not have been already added to this shelf.
115 - (void)addDownloadItem:(content::DownloadItem*)downloadItem; 115 - (void)addDownloadItem:(content::DownloadItem*)downloadItem;
116 116
117 // Similar to addDownloadItem above, but adds a DownloadItemController. 117 // Similar to addDownloadItem above, but adds a DownloadItemController.
118 - (void)add:(DownloadItemController*)download; 118 - (void)add:(DownloadItemController*)download;
119 119
120 // Remove a download, possibly via clearing browser data. 120 // Remove a download, possibly via clearing browser data.
121 - (void)remove:(DownloadItemController*)download; 121 - (void)remove:(DownloadItemController*)download;
122 122
123 // Called by individual item controllers when their downloads are opened. 123 // Called by individual item controllers when their downloads are opened or
124 - (void)downloadWasOpened:(DownloadItemController*)download; 124 // shown.
125 - (void)downloadWasOpenedOrShown:(DownloadItemController*)download;
125 126
126 // Return the height of the download shelf. 127 // Return the height of the download shelf.
127 - (float)height; 128 - (float)height;
128 129
129 // Re-layouts all download items based on their current state. 130 // Re-layouts all download items based on their current state.
130 - (void)layoutItems; 131 - (void)layoutItems;
131 132
132 @end 133 @end
OLDNEW
« no previous file with comments | « chrome/browser/ui/cocoa/download/download_item_mac.mm ('k') | chrome/browser/ui/cocoa/download/download_shelf_controller.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698