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

Unified Diff: ash/app_list/app_list_presenter_delegate.cc

Issue 2898743002: Draggable peeking/fullscreen launcher with transparent background. (Closed)
Patch Set: Migrated from wm_shelf to shelf and addressed comments! Created 3 years, 7 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
« no previous file with comments | « no previous file | ash/app_list/app_list_presenter_delegate_unittest.cc » ('j') | ash/shelf/shelf.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 258a88d4903e4a7b8a6073e9559ac0ccb668cbca..10833f24eba4e6a96b85a9c6dad28b86e7ccb189 100644
--- a/ash/app_list/app_list_presenter_delegate.cc
+++ b/ash/app_list/app_list_presenter_delegate.cc
@@ -129,6 +129,12 @@ void AppListPresenterDelegate::OnShown(int64_t display_id) {
Shelf::ForWindow(root_window)->shelf_widget()->GetAppListButton();
if (app_list_button)
app_list_button->OnAppListShown();
+
+ // Notify the shelf that the app list is active so the backgrounds do not
+ // overlap.
+ Shelf* shelf = Shelf::ForWindow(root_window);
vadimt 2017/05/26 01:27:56 You can move this inside if.
newcomer 2017/05/26 17:11:27 Done.
+ if (app_list::features::IsFullscreenAppListEnabled())
+ shelf->OnAppListIsActive(true);
}
void AppListPresenterDelegate::OnDismissed() {
@@ -142,6 +148,11 @@ void AppListPresenterDelegate::OnDismissed() {
AppListButton* app_list_button = shelf->shelf_widget()->GetAppListButton();
if (app_list_button)
app_list_button->OnAppListDismissed();
+ if (app_list::features::IsFullscreenAppListEnabled()) {
+ // Notify the shelf that the app list is being dismissed so it can
+ // re-enable its transparent background.
+ shelf->OnAppListIsActive(false);
+ }
}
void AppListPresenterDelegate::UpdateBounds() {
« no previous file with comments | « no previous file | ash/app_list/app_list_presenter_delegate_unittest.cc » ('j') | ash/shelf/shelf.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698