Do not toggle Applist with Search key press when accessibility is enabled.

ash/shell/shell_delegate_impl.cc

Index: ash/shell/shell_delegate_impl.cc
diff --git a/ash/shell/shell_delegate_impl.cc b/ash/shell/shell_delegate_impl.cc
index 661aa515b9ee2a741b17e05e7f29d17b01df8e94..bc3f30d6a9a6f5e8ae66c6773cd1d768ebdc79ef 100644
--- a/ash/shell/shell_delegate_impl.cc
+++ b/ash/shell/shell_delegate_impl.cc
@@ -18,7 +18,8 @@ namespace shell {
 ShellDelegateImpl::ShellDelegateImpl()
     : watcher_(NULL),
       launcher_delegate_(NULL),
-      locked_(false) {
+      locked_(false),
+      spoken_feedback_enabled_(false) {
 }
 
 ShellDelegateImpl::~ShellDelegateImpl() {
@@ -93,6 +94,11 @@ content::BrowserContext* ShellDelegateImpl::GetCurrentBrowserContext() {
 }
 
 void ShellDelegateImpl::ToggleSpokenFeedback() {
+  spoken_feedback_enabled_ = !spoken_feedback_enabled_;

[dmazzoni, 2012/06/14 16:45:25]

This worries me slightly, I'm afraid it could get out of sync.

Spoken feedback is turned on in a couple of different places - not only via the
global shortcut but also by toggling the checkbox in the Options. It's also
persistent - i.e. it will turn on automatically on boot if the preference is
set.

The main toggle function is in
src/chrome/browser/chromeos/accessibility/accessibility_util.cc - I think you
need to hook into there, or read the preference directly in order to be 100%
sure that this state will always be the same.

[Yusuke Sato, 2012/06/14 16:57:15]

If I understand correctly, ash/shell/* files are just for building test
environment for Ash developers, and are not included in a production Chrome OS
image.

The real implementation of ShellDelegate::ToggleSpokenFeedback() and
ShellDelegate::IsSpokenFeedbackEnabled() are in
chrome/browser/ui/views/ash/chrome_shell_delegate.cc. As you can see in
https://chromiumcodereview.appspot.com/10543158/diff/7001/chrome/browser/ui/v...
, the ShellDelegate functions in the file correctly use
chromeos::accessibility:: utility functions as you suggested.

On 2012/06/14 16:45:25, Dominic Mazzoni wrote:

[dmazzoni, 2012/06/14 17:05:00]

Ah! Thanks, I didn't realize that. Looks good.

- Dominic

+}
+
+bool ShellDelegateImpl::IsSpokenFeedbackEnabled() const {
+  return spoken_feedback_enabled_;
 }
 
 app_list::AppListViewDelegate* ShellDelegateImpl::CreateAppListViewDelegate() {