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

Unified Diff: ui/app_list/presenter/app_list_presenter_impl.cc

Issue 2898743002: Draggable peeking/fullscreen launcher with transparent background. (Closed)
Patch Set: Fixed case where Esc key was not closing app list. Created 3 years, 6 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: ui/app_list/presenter/app_list_presenter_impl.cc
diff --git a/ui/app_list/presenter/app_list_presenter_impl.cc b/ui/app_list/presenter/app_list_presenter_impl.cc
index 9fa5613fca946acea340239af1c1cd85735af108..c4ef309215f59c14134c09e053aa685c4a643e51 100644
--- a/ui/app_list/presenter/app_list_presenter_impl.cc
+++ b/ui/app_list/presenter/app_list_presenter_impl.cc
@@ -59,8 +59,10 @@ void AppListPresenterImpl::Show(int64_t display_id) {
return;
is_visible_ = true;
- if (app_list_)
+ if (app_list_) {
app_list_->OnTargetVisibilityChanged(GetTargetVisibility());
+ app_list_->OnVisibilityChanged(GetTargetVisibility(), display_id);
+ }
if (view_) {
ScheduleAnimation();
@@ -86,9 +88,10 @@ void AppListPresenterImpl::Dismiss() {
DCHECK(view_);
is_visible_ = false;
- if (app_list_)
+ if (app_list_) {
app_list_->OnTargetVisibilityChanged(GetTargetVisibility());
-
+ app_list_->OnVisibilityChanged(GetTargetVisibility(), GetDisplayId());
+ }
// The dismissal may have occurred in response to the app list losing
// activation. Otherwise, our widget is currently active. When the animation
// completes we'll hide the widget, changing activation. If a menu is shown

Powered by Google App Engine
This is Rietveld 408576698