ash/chromeos: "Search" key to toggle app list.

ash/accelerators/accelerator_controller.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/accelerators/accelerator_controller.h"
 
 #include "ash/accelerators/accelerator_table.h"
 #include "ash/ash_switches.h"
 #include "ash/caps_lock_delegate.h"
 #include "ash/desktop_background/desktop_background_controller.h"
@@ skipping 316 matching lines @@
       }
       // Return true to prevent propagation of the key event.
       return true;
     case TAKE_PARTIAL_SCREENSHOT:
       if (screenshot_delegate_.get())
         ash::Shell::GetInstance()->delegate()->
             StartPartialScreenshot(screenshot_delegate_.get());
       // Return true to prevent propagation of the key event because
       // this key combination is reserved for partial screenshot.
       return true;
     case SEARCH_KEY:

[mazda, 2012/06/08 03:43:57]

How about removing SEARCH_KEY instead of TOGGLE_APP_LIST?
The Enum name should be named after the invoked action rather than the
accelerators that trigger the action.

[xiyuan, 2012/06/08 16:37:13]

Done.

-      ash::Shell::GetInstance()->delegate()->Search();
-      return true;
-    case TOGGLE_APP_LIST:
       ash::Shell::GetInstance()->ToggleAppList();
       return true;
     case TOGGLE_CAPS_LOCK:
       if (caps_lock_delegate_.get())
         return caps_lock_delegate_->HandleToggleCapsLock();
       break;
     case BRIGHTNESS_DOWN:
       if (brightness_control_delegate_.get())
         return brightness_control_delegate_->HandleBrightnessDown(accelerator);
       break;
@@ skipping 210 matching lines @@
     // Then set this one as active.
     Shell::GetInstance()->launcher()->ActivateLauncherItem(found_index);
   }
 }
 
 bool AcceleratorController::CanHandleAccelerators() const {
   return true;
 }
 
 }  // namespace ash