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

chrome/browser/resources/chromeos/switch_access/tree_walker_unittest.gtestjs

Index: chrome/browser/resources/chromeos/switch_access/tree_walker_unittest.gtestjs
diff --git a/chrome/browser/resources/chromeos/switch_access/switch_access_unittest.gtestjs b/chrome/browser/resources/chromeos/switch_access/tree_walker_unittest.gtestjs
similarity index 61%
rename from chrome/browser/resources/chromeos/switch_access/switch_access_unittest.gtestjs
rename to chrome/browser/resources/chromeos/switch_access/tree_walker_unittest.gtestjs
index 4536781df8d97ed45b1d8429eb6d7f9f8a4523fa..f305d28aff865a74f46a0e88a30ab3acebf3bcf8 100644
--- a/chrome/browser/resources/chromeos/switch_access/switch_access_unittest.gtestjs
+++ b/chrome/browser/resources/chromeos/switch_access/tree_walker_unittest.gtestjs
@@ -3,23 +3,21 @@
 // found in the LICENSE file.
 
 /**
- * Test fixture for switch_access.js.
+ * Test fixture for tree_walker.js.
  * @constructor
  * @extends {testing.Test}
  */
-function SwitchAccessUnitTest () {
+function TreeWalkerUnitTest () {

[David Tseng, 2017/03/29 21:57:21]

AutomationTreeWalkerUnitTest

[elichtenberg, 2017/03/30 21:46:56]

Done.

   testing.Test.call(this);
 };
 
-SwitchAccessUnitTest.prototype = {
+TreeWalkerUnitTest.prototype = {
   __proto__: testing.Test.prototype,
 
   /** @override */
   extraLibraries: [
-    'test_support.js',
-    'prefs.js',
-    'switch_access.js',
-    'testable_switch_access.js',
+    'tree_walker.js',
+    'testable_tree_walker.js',
   ],
 
   getSampleTree: function() {
@@ -78,9 +76,9 @@ SwitchAccessUnitTest.prototype = {
   }
 };
 
-TEST_F('SwitchAccessUnitTest', 'GetNextNode', function() {
+TEST_F('TreeWalkerUnitTest', 'GetNextNode', function() {
   let tree = this.getSampleTree();
-  let tsa = new TestableSwitchAccess();
+  let treeWalker = new TestableTreeWalker();
 
   let order =
       [tree.root, tree.middle1, tree.leaf1, tree.leaf2, tree.leaf3,
@@ -88,14 +86,14 @@ TEST_F('SwitchAccessUnitTest', 'GetNextNode', function() {
   let node = tree.root;
   for (let i = 0; i < order.length; i++) {
     assertEquals(order[i], node);
-    node = tsa.getNextNode(node);
+    node = treeWalker.getNextNode(node);
   }
   assertEquals(null, node);
 });
 
-TEST_F('SwitchAccessUnitTest', 'GetPreviousNode', function() {
+TEST_F('TreeWalkerUnitTest', 'GetPreviousNode', function() {
   let tree = this.getSampleTree();
-  let tsa = new TestableSwitchAccess();
+  let treeWalker = new TestableTreeWalker();
 
   let order =
       [tree.leaf5, tree.leaf4, tree.middle2, tree.leaf3, tree.leaf2,
@@ -103,34 +101,34 @@ TEST_F('SwitchAccessUnitTest', 'GetPreviousNode', function() {
   let node = tree.leaf5;
   for (let i = 0; i < order.length; i++) {
     assertEquals(order[i], node);
-    node = tsa.getPreviousNode(node);
+    node = treeWalker.getPreviousNode(node);
   }
   assertEquals(null, node);
 });
 
-TEST_F('SwitchAccessUnitTest', 'GetYoungestDescendant', function() {
+TEST_F('TreeWalkerUnitTest', 'GetYoungestDescendant', function() {
   let tree = this.getSampleTree();
-  let tsa = new TestableSwitchAccess();
-
-  assertEquals(tree.leaf5, tsa.getYoungestDescendant(tree.root));
-  assertEquals(tree.leaf3, tsa.getYoungestDescendant(tree.middle1));
-  assertEquals(tree.leaf5, tsa.getYoungestDescendant(tree.middle2));
-  assertEquals(null, tsa.getYoungestDescendant(tree.leaf1));
-  assertEquals(null, tsa.getYoungestDescendant(tree.leaf2));
-  assertEquals(null, tsa.getYoungestDescendant(tree.leaf3));
-  assertEquals(null, tsa.getYoungestDescendant(tree.leaf4));
-  assertEquals(null, tsa.getYoungestDescendant(tree.leaf5));
+  let treeWalker = new TestableTreeWalker();
+
+  assertEquals(tree.leaf5, treeWalker.getYoungestDescendant(tree.root));
+  assertEquals(tree.leaf3, treeWalker.getYoungestDescendant(tree.middle1));
+  assertEquals(tree.leaf5, treeWalker.getYoungestDescendant(tree.middle2));
+  assertEquals(null, treeWalker.getYoungestDescendant(tree.leaf1));
+  assertEquals(null, treeWalker.getYoungestDescendant(tree.leaf2));
+  assertEquals(null, treeWalker.getYoungestDescendant(tree.leaf3));
+  assertEquals(null, treeWalker.getYoungestDescendant(tree.leaf4));
+  assertEquals(null, treeWalker.getYoungestDescendant(tree.leaf5));
 });
 
-TEST_F('SwitchAccessUnitTest', 'IsInteresting', function() {
+TEST_F('TreeWalkerUnitTest', 'IsInteresting', function() {
   let node1 = {};
   let node2 = {state: {}};
   let node3 = {state: {focusable: false}};
   let node4 = {state: {focusable: true}};
-  let tsa = new TestableSwitchAccess();
+  let treeWalker = new TestableTreeWalker();
 
-  assertTrue(tsa.isInteresting(node1) === undefined);
-  assertTrue(tsa.isInteresting(node2) === undefined);
-  assertFalse(tsa.isInteresting(node3));
-  assertTrue(tsa.isInteresting(node4));
+  assertTrue(treeWalker.isInteresting(node1) === undefined);
+  assertTrue(treeWalker.isInteresting(node2) === undefined);
+  assertFalse(treeWalker.isInteresting(node3));
+  assertTrue(treeWalker.isInteresting(node4));
 });