Added auto-scan, made some small changes to how prefs are stored, refactored.

chrome/browser/resources/chromeos/switch_access/closure_support.js

Index: chrome/browser/resources/chromeos/switch_access/closure_support.js
diff --git a/chrome/browser/resources/chromeos/switch_access/test_support.js b/chrome/browser/resources/chromeos/switch_access/closure_support.js
similarity index 57%
rename from chrome/browser/resources/chromeos/switch_access/test_support.js
rename to chrome/browser/resources/chromeos/switch_access/closure_support.js
index 51f0e823acbd3b1f74764e7d1833241567d4e5ce..9ea49974cf24b2218b085b412a477da99e5d7c93 100644
--- a/chrome/browser/resources/chromeos/switch_access/test_support.js
+++ b/chrome/browser/resources/chromeos/switch_access/closure_support.js
@@ -3,7 +3,6 @@
 // found in the LICENSE file.
 
 /**
- * Stubs out extension API types and functions, and DOM elements, so
- * SwitchAccessUnitTest can load.
+ * Shortcuts for variable type declarations in closure compiler.
  */
-chrome.automation = {};
+let AutomationNode = chrome.automation.AutomationNode;

[dmazzoni, 2017/03/29 05:52:20]

This doesn't look like a shortcut for the Closure Compiler anymore.
I think it'd be better to delete this file and put this in any scoped block
where it'd be useful for readability.

[elichtenberg, 2017/03/29 19:15:11]

Done. Creating scopes caused some issues with closure, so just used full type
name in annotations.