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

Unified Diff: ui/aura_shell/app_list.cc

Issue 8747021: [Aura] Polish app list. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 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: ui/aura_shell/app_list.cc
diff --git a/ui/aura_shell/app_list.cc b/ui/aura_shell/app_list.cc
index 0218713b9e3a8419a54a09ae33229d2946bd726c..d6f6f760178265fae7295bc6d473f22f78d9c1bd 100644
--- a/ui/aura_shell/app_list.cc
+++ b/ui/aura_shell/app_list.cc
@@ -62,6 +62,7 @@ void AppList::SetVisible(bool visible) {
ScheduleAnimation();
} else if (is_visible_ && !set_widget_factory_.HasWeakPtrs()) {
Shell::GetInstance()->delegate()->RequestAppListWidget(
+ GetPreferredBounds(false),
base::Bind(&AppList::SetWidget, set_widget_factory_.GetWeakPtr()));
}
}
@@ -88,7 +89,6 @@ void AppList::SetWidget(views::Widget* widget) {
ScheduleAnimation();
widget_->Show();
- widget_->Activate();
} else {
widget->Close();
}
@@ -162,6 +162,8 @@ void AppList::OnLayerAnimationScheduled(
void AppList::OnWidgetClosing(views::Widget* widget) {
DCHECK(widget_ == widget);
+ if (is_visible_)
+ SetVisible(false);
ResetWidget();
}

Powered by Google App Engine
This is Rietveld 408576698