Avoid getting stuck when calling focus() on iframe nodes.

chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js

Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
index 67b261f3cb7cbf9b8ea1f12397f8003f0e9a0343..575dfe720d084db11fa5eac8c36ecfa06799ce27 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js
@@ -500,7 +500,11 @@ Background.prototype = {
       var actionNode = current.start.node;
       if (actionNode.role == RoleType.inlineTextBox)
         actionNode = actionNode.parent;
-      actionNode.focus();
+
+      // Iframes, when focused, causes the child webArea to fire focus event.
+      // This can result in getting stuck when navigating backward.
+      if (!actionNode.role == RoleType.iframe && actionNode.state.focused)

[dmazzoni, 2016/01/23 00:01:38]

The order of precedence isn't obvious to me here, could you either put the thing
being negated in parens, or use a != operator?

[David Tseng, 2016/01/25 20:13:16]

Done.

+        actionNode.focus();
 
       var prevRange = this.currentRange_;
       this.setCurrentRange(current);