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

Unified Diff: chrome/browser/ui/views/download/download_shelf_view.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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/download/download_shelf_view.h
diff --git a/chrome/browser/ui/views/download/download_shelf_view.h b/chrome/browser/ui/views/download/download_shelf_view.h
index 43e4fda185f4d9682bd0519e6e06740608d19ff8..a0942cecb265b861016e96daed499b48db48e309 100644
--- a/chrome/browser/ui/views/download/download_shelf_view.h
+++ b/chrome/browser/ui/views/download/download_shelf_view.h
@@ -50,7 +50,10 @@ class DownloadShelfView : public views::AccessiblePaneView,
~DownloadShelfView() override;
// Sent from the DownloadItemView when the user opens an item.
- void OpenedDownload();
+ void OnDownloadOpened();
+
+ // Sent from the DownloadItemView when the user shows an item in shell.
+ void OnDownloadShown();
// Returns the relevant containing object that can load pages.
// i.e. the |browser_|.
@@ -128,6 +131,9 @@ class DownloadShelfView : public views::AccessiblePaneView,
// the shelf have been opened.
bool CanAutoClose();
+ // Starts mouse watcher if we can auto close.
+ void TriggerAutoClose();
+
// Returns the color of text for the shelf (used for deriving icon color).
SkColor GetTextColorForIconMd();
« no previous file with comments | « chrome/browser/ui/views/download/download_item_view.cc ('k') | chrome/browser/ui/views/download/download_shelf_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698