Changing focus ring color by node. Focus ring can now change color without changing bounds

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

 // Copyright 2017 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.
 
 /**
  * Class to manage interactions with the accessibility tree, including moving
  * to and selecting nodes.
  *
  * @constructor
  */
@@ skipping 31 matching lines @@
   /**
    * Moves to the appropriate node in the accessibility tree.
    *
    * @private {AutomationTreeWalker}
    */
   this.treeWalker_ = null;
 
   this.init_();
 };
 
+/**
+ * Highlight colors for the focus ring to distinguish between different types
+ * of nodes.
+ *
+ * @const
+ */
+AutomationManager.Color = {
+  SCOPE: '#de742f', // dark orange
+  GROUP: '#ffbb33', // light orange
+  LEAF: '#78e428' //light green
+};
+
 AutomationManager.prototype = {
   /**
    * Set this.node_, this.root_, and this.desktop_ to the desktop node, and
    * creates an initial tree walker.
    *
    * @private
    */
   init_: function() {
     chrome.automation.getDesktop(function(desktop) {
       this.node_ = desktop;
@@ skipping 14 matching lines @@
    * @param {boolean} doNext
    */
   moveToNode: function(doNext) {
     if (!this.treeWalker_)
       return;
 
     let node = this.treeWalker_.moveToNode(doNext);
     if (node) {
       this.node_ = node;
       this.printNode_(this.node_);
-      chrome.accessibilityPrivate.setFocusRing([this.node_.location]);
+
+      let color;
+      if (this.node_ === this.scope_)
+        color = AutomationManager.Color.SCOPE;
+      else if (AutomationPredicate.isInteresting(this.node_))
+        color = AutomationManager.Color.LEAF;
+      else
+        color = AutomationManager.Color.GROUP;
+      chrome.accessibilityPrivate.setFocusRing([this.node_.location], color);
     }
   },
 
   /**
    * Select the currently highlighted node. If the node is the current scope,
    * go back to the previous scope (i.e., create a new tree walker rooted at
    * the previous scope). If the node is a group other than the current scope,
    * create a new tree walker for the new subtree the user is scanning through.
    * Otherwise, meaning the node is interesting, perform the default action on
    * it.
@@ skipping 10 matching lines @@
       // Find a previous scope that is still valid.
       let oldScope;
       do {
         oldScope = this.scopeStack_.pop();
       } while (oldScope && !oldScope.role);
 
       // oldScope will always be valid here, so this will always be true.
       if (oldScope) {
         this.scope_ = oldScope;
         this.treeWalker_ = this.createTreeWalker_(this.scope_, this.node_);
+        chrome.accessibilityPrivate.setFocusRing(
+            [this.node_.location], AutomationManager.Color.GROUP);
       }
       return;
     }
 
     if (AutomationPredicate.isGroup(this.node_, this.scope_)) {
       this.scopeStack_.push(this.scope_);
       this.scope_ = this.node_;
       this.treeWalker_ = this.createTreeWalker_(this.scope_);
       this.moveToNode(true);
       return;
@@ skipping 102 matching lines @@
    */
   debugMoveToParent: function() {
     let parent = this.treeWalker_.debugMoveToParent(this.node_);
     if (parent) {
       this.node_ = parent;
       this.printNode_(this.node_);
       chrome.accessibilityPrivate.setFocusRing([this.node_.location]);
     }
   }
 };