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

ash/accelerators/accelerator_controller.cc

Index: ash/accelerators/accelerator_controller.cc
diff --git a/ash/accelerators/accelerator_controller.cc b/ash/accelerators/accelerator_controller.cc
index 289c4d56df089f3f6a4fee06b14fa9333e01bb47..1963c199311f408ea580cc6652c2965fd5d03c8a 100644
--- a/ash/accelerators/accelerator_controller.cc
+++ b/ash/accelerators/accelerator_controller.cc
@@ -485,7 +485,10 @@ bool HandleToggleAppList(ui::KeyboardCode key_code,
   if (Shell::GetInstance()->accessibility_delegate()->
       IsSpokenFeedbackEnabled())
     return false;
-  ash::Shell::GetInstance()->ToggleAppList(NULL);
+  if (ash::Shell::GetInstance()->GetAppListTargetVisibility())

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

I think it would be good to preserve ToggleAppList as a wrapper around
ShowAppList and DismissAppList, so you don't need this four-line logic
duplicated in potentially multiple places (I count 2 in this CL, but more may
follow).

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

Done.

+    ash::Shell::GetInstance()->DismissAppList();
+  else
+    ash::Shell::GetInstance()->ShowAppList(NULL);
   return true;
 }