[Aura] Polish app list.

- Remove search box until we have decided how to approach it;
- Show apps pagination dots;
- Dismiss app list UI on escape or click on non-app and non nav dots;
- Defer widget showing further until apps are loaded by webui;
- Add a container div to limit apps page height so that the gap between apps icons and nav dots are not too big;
- Passing bounds when requesting app list widget to avoid resizing;

BUG=98308,105794,106451
TEST=Verify search box is gone, nav gots show up on bottom and could switch pages by clicking on it;


Review URL: http://codereview.chromium.org/8747021

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113156 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 1 deletions

diff --git a/ui/aura_shell/app_list.cc b/ui/aura_shell/app_list.cc
index 0218713..d6f6f76 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();
 }