Split Shell::ToggleAppList() into ShowAppList and DismissAppList.

chrome/browser/ui/ash/app_list/app_list_service_ash.cc

Index: chrome/browser/ui/ash/app_list/app_list_service_ash.cc
diff --git a/chrome/browser/ui/ash/app_list/app_list_service_ash.cc b/chrome/browser/ui/ash/app_list/app_list_service_ash.cc
index 58eb99b5d10b8037589551f9d1f34ca2447f8040..59f599a75d02236193745cc47178f8f7d55eda14 100644
--- a/chrome/browser/ui/ash/app_list/app_list_service_ash.cc
+++ b/chrome/browser/ui/ash/app_list/app_list_service_ash.cc
@@ -35,8 +35,7 @@ void AppListServiceAsh::ShowForProfile(Profile* default_profile) {
   // This may not work correctly if the profile passed in is different from the
   // one the ash Shell is currently using.
   // TODO(ananta): Handle profile changes correctly when !defined(OS_CHROMEOS).
-  if (!ash::Shell::GetInstance()->GetAppListTargetVisibility())
-    ash::Shell::GetInstance()->ToggleAppList(NULL);
+  ash::Shell::GetInstance()->ShowAppList(NULL);
 }
 
 bool AppListServiceAsh::IsAppListVisible() const {

[Matt Giuca, 2014/08/14 04:17:07]

You can delete this now.

[calamity, 2014/08/14 05:00:59]

It's an override.

[Matt Giuca, 2014/08/14 05:05:36]

Acknowledged.

@@ -44,8 +43,7 @@ bool AppListServiceAsh::IsAppListVisible() const {
 }
 
 void AppListServiceAsh::DismissAppList() {
-  if (IsAppListVisible())
-    ash::Shell::GetInstance()->ToggleAppList(NULL);
+  ash::Shell::GetInstance()->DismissAppList();
 }
 
 void AppListServiceAsh::EnableAppList(Profile* initial_profile,