Set keys for SwitchAccessEventHandler to capture using accessibiltyPrivate API

chrome/browser/accessibility/accessibility_extension_api.cc

Index: chrome/browser/accessibility/accessibility_extension_api.cc
diff --git a/chrome/browser/accessibility/accessibility_extension_api.cc b/chrome/browser/accessibility/accessibility_extension_api.cc
index 62abc25566768286c5c722d2896334b09bbf0b2b..e42a61d7efb49819c03a58830897bb9d3b54b19e 100644
--- a/chrome/browser/accessibility/accessibility_extension_api.cc
+++ b/chrome/browser/accessibility/accessibility_extension_api.cc
@@ -27,6 +27,7 @@
 #include "extensions/common/error_utils.h"
 #include "extensions/common/image_util.h"
 #include "extensions/common/manifest_handlers/background_info.h"
+#include "ui/events/keycodes/keyboard_codes.h"
 
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
@@ -164,3 +165,29 @@ AccessibilityPrivateDarkenScreenFunction::Run() {
 
   return RespondNow(Error(kErrorNotSupported));
 }
+
+ExtensionFunction::ResponseAction
+AccessibilityPrivateSetSwitchAccessKeysFunction::Run() {
+#if defined(OS_CHROMEOS)
+  base::ListValue* key_codes_list = NULL;
+  EXTENSION_FUNCTION_VALIDATE(args_->GetSize() == 1);
+  EXTENSION_FUNCTION_VALIDATE(args_->GetList(0, &key_codes_list));
+
+  // For now, only accept key code if it represents an alphanumeric character.
+  std::unordered_set<int> key_codes;
+  for (size_t i = 0; i < key_codes_list->GetSize(); ++i) {
+    int key_code;
+    key_codes_list->GetInteger(i, &key_code);

[dmazzoni, 2017/05/30 15:09:13]

GetInteger returns true on success, so put EXTENSION_FUNCTION_VALIDATE around
this so that it fails if you try to pass something other than an integer

[elichtenberg, 2017/05/30 16:45:33]

Done.

+    if (key_code >= ui::VKEY_0 && key_code <= ui::VKEY_Z)
+      key_codes.insert(key_code);
+  }
+
+  chromeos::AccessibilityManager* manager =
+      chromeos::AccessibilityManager::Get();
+  if (manager)
+    manager->SetSwitchAccessKeys(key_codes);
+  return RespondNow(NoArguments());
+#endif  // defined (OS_CHROMEOS)
+
+  return RespondNow(Error(kErrorNotSupported));
+}