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

Unified Diff: chrome/browser/ui/views/download/download_shelf_view.cc

Issue 2688413012: Don't animate the download shelf when entering/exiting fullscreen. (Closed)
Patch Set: Turn on animation for all tests except the one that was flaky due to animation. Created 3 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/views/download/download_shelf_view.cc
diff --git a/chrome/browser/ui/views/download/download_shelf_view.cc b/chrome/browser/ui/views/download/download_shelf_view.cc
index 1e119e9d3959fbaafbb2a72af0634e83a2a4695f..23b32978b9c7f763f8fb1da9c346a56a417695ee 100644
--- a/chrome/browser/ui/views/download/download_shelf_view.cc
+++ b/chrome/browser/ui/views/download/download_shelf_view.cc
@@ -367,7 +367,7 @@ bool DownloadShelfView::IsClosing() const {
return shelf_animation_.IsClosing();
}
-void DownloadShelfView::DoShow() {
+void DownloadShelfView::DoOpen() {
SetVisible(true);
shelf_animation_.Show();
}
@@ -384,6 +384,18 @@ void DownloadShelfView::DoClose(CloseReason reason) {
shelf_animation_.Hide();
}
+void DownloadShelfView::DoHide() {
+ SetVisible(false);
+ parent_->ToolbarSizeChanged(false);
+ parent_->SetDownloadShelfVisible(false);
+}
+
+void DownloadShelfView::DoUnhide() {
+ SetVisible(true);
+ parent_->ToolbarSizeChanged(true);
+ parent_->SetDownloadShelfVisible(true);
+}
+
Browser* DownloadShelfView::browser() const {
return browser_;
}
« no previous file with comments | « chrome/browser/ui/views/download/download_shelf_view.h ('k') | chrome/browser/ui/views/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698