Fix Android hit testing in a totally static page.

content/browser/accessibility/browser_accessibility_manager_android.cc

Index: content/browser/accessibility/browser_accessibility_manager_android.cc
diff --git a/content/browser/accessibility/browser_accessibility_manager_android.cc b/content/browser/accessibility/browser_accessibility_manager_android.cc
index 0259faf4ac615a586215909a8b0c1dc22c1742e7..5b4541f56ed7c2ed428311ca5608f85250960971 100644
--- a/content/browser/accessibility/browser_accessibility_manager_android.cc
+++ b/content/browser/accessibility/browser_accessibility_manager_android.cc
@@ -416,7 +416,7 @@ void BrowserAccessibilityManagerAndroid::HandleHoverEvent(
 
   BrowserAccessibilityAndroid* ancestor =
       static_cast<BrowserAccessibilityAndroid*>(node->GetParent());
-  while (ancestor) {
+  while (ancestor && ancestor != GetRoot()) {

[aboxhall, 2014/10/16 21:33:34]

Would it be worthwhile pulling the result of GetRoot() out into a local pointer
to avoid calling the method repeatedly?

[dmazzoni, 2014/10/16 21:47:24]

The compiler ought to be smart enough to do this. Still might be a good idea if
GetRoot was known to be expensive, just in case - but it's cheap so I think it's
more concise like this.

     if (ancestor->PlatformIsLeaf() ||
         (ancestor->IsFocusable() && !ancestor->HasFocusableChild())) {
       node = ancestor;