Set keys for SwitchAccessEventHandler to capture using accessibiltyPrivate API

chrome/browser/accessibility/accessibility_extension_api.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 "chrome/browser/accessibility/accessibility_extension_api.h"
 
 #include <stddef.h>
 
 #include "base/json/json_writer.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 146 matching lines @@
 
   // Called twice to ensure the cros end of the dbus message is in a good
   // state.
   client->SetBacklightsForcedOff(!darken);
   client->SetBacklightsForcedOff(darken);
   return RespondNow(NoArguments());
 #endif  // defined OS_CHROMEOS
 
   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::vector<int> key_codes;

[David Tseng, 2017/05/26 22:10:31]

Is this supposed to be ordered or are you maybe looking for a set?

[elichtenberg, 2017/05/27 00:13:04]

Yep, I did mean to do a set. Thanks!

+  for (size_t i = 0; i < key_codes_list->GetSize(); ++i) {
+    int key_code;
+    key_codes_list->GetInteger(i, &key_code);
+    if (key_code >= 48 && key_code <= 90)

[David Tseng, 2017/05/26 22:10:31]

You should be able to grab key codes from a number of headers in the Chrome
source.

[elichtenberg, 2017/05/27 00:13:04]

Done.

+      key_codes.push_back(key_code);
+  }
+
+  chromeos::AccessibilityManager* manager =
+      chromeos::AccessibilityManager::Get();

[David Tseng, 2017/05/26 22:10:31]

Can be null.

[elichtenberg, 2017/05/27 00:13:04]

Done.

+  manager->SetSwitchAccessKeys(key_codes);
+  return RespondNow(NoArguments());

[David Tseng, 2017/05/26 22:10:31]

You should probably respond with an error for various cases above.

[elichtenberg, 2017/05/27 00:13:04]

What cases are you thinking of? Like if the key codes list is empty or has
non-alphanumeric characters? Maybe we should discuss this offline.

+#endif  // defined (OS_CHROMEOS)
+
+  return RespondNow(Error(kErrorNotSupported));
+}