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

Unified Diff: ash/app_list/app_list_presenter_delegate.cc

Issue 2534953006: Fix shelf auto-hide calculation for app-list visibility. (Closed)
Patch Set: try another approach. Created 4 years 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
« no previous file with comments | « no previous file | ash/common/shelf/app_list_button.h » ('j') | ash/common/shelf/app_list_button.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/app_list/app_list_presenter_delegate.cc
diff --git a/ash/app_list/app_list_presenter_delegate.cc b/ash/app_list/app_list_presenter_delegate.cc
index 976dc33f8aa29ed69cafca086f0b9cd3a3bea141..b4d2c3ab28bf698a6e411e1dffd462c1a25dba32 100644
--- a/ash/app_list/app_list_presenter_delegate.cc
+++ b/ash/app_list/app_list_presenter_delegate.cc
@@ -164,13 +164,9 @@ void AppListPresenterDelegate::OnDismissed() {
is_visible_ = false;
- // App list needs to know the new shelf layout in order to calculate its
- // UI layout when AppListView visibility changes.
+ // Update applist button status when app list visibility is changed.
WmShelf* shelf = WmShelf::ForWindow(
WmLookup::Get()->GetWindowForWidget(view_->GetWidget()));
- shelf->UpdateAutoHideState();
-
- // Update applist button status when app list visibility is changed.
shelf->shelf_widget()->GetAppListButton()->OnAppListDismissed();
}
« no previous file with comments | « no previous file | ash/common/shelf/app_list_button.h » ('j') | ash/common/shelf/app_list_button.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698