Don't set the STATE_SYSTEM_HOTTRACKED state in tests.

content/browser/accessibility/browser_accessibility_state_impl.h

Index: content/browser/accessibility/browser_accessibility_state_impl.h
diff --git a/content/browser/accessibility/browser_accessibility_state_impl.h b/content/browser/accessibility/browser_accessibility_state_impl.h
index 4c8884373d6c1ef25ed04c62d47369d1ccb310da..77249163f402dee8dd1286e1f8180ae91777c052 100644
--- a/content/browser/accessibility/browser_accessibility_state_impl.h
+++ b/content/browser/accessibility/browser_accessibility_state_impl.h
@@ -60,6 +60,18 @@ class CONTENT_EXPORT BrowserAccessibilityStateImpl
   // removed.
   void RemoveAccessibilityMode(AccessibilityMode mode);
 
+  // Accessibility objects can have the "hot tracked" state set when

[David Tseng, 2015/02/03 02:19:42]

I wonder if an easier fix to this is to just move the mouse to some known
position programmatically as part of a test's setup?

[dmazzoni, 2015/02/03 06:54:11]

I think that'd be annoying for anyone who runs content_browsertests locally.
Also I think it sets a bad precedent because tests run in parallel but there's
only one mouse. If I set it to 0,0 and another test set it to 9999, 9999 they
could mess with each other and cause more flakiness. I think it's better to not
have tests depend on the local machine.

The exception is interactive_ui_tests - those tests are allowed to do things
like move the mouse, but I don't see any reason to migrate these tests there -
they'd run much more slowly since those tests can't be parallelized.

Other options would include:
* Setting the flag as a static bool in BrowserAccessibility or
BrowserAccessibilityWin instead
* Ignoring STATE_SYSTEM_HOTTRACKED when checking states in test code, rather
than changing the code being tested - this has the advantage of not messing with
the real code, but it requires remembering to do that in every test.

+  // the mouse is hovering over them, but this makes tests flaky because
+  // the test behaves differently when the mouse happens to be over an
+  // element.  This is a global switch to not use the "hot tracked" state
+  // in a test.
+  void set_disable_hot_tracking_for_testing(bool disable_hot_tracking) {
+    disable_hot_tracking_ = disable_hot_tracking;
+  }
+  bool disable_hot_tracking_for_testing() const {
+    return disable_hot_tracking_;
+  }
+
  private:
   friend class base::RefCountedThreadSafe<BrowserAccessibilityStateImpl>;
   friend struct DefaultSingletonTraits<BrowserAccessibilityStateImpl>;
@@ -84,6 +96,8 @@ class CONTENT_EXPORT BrowserAccessibilityStateImpl
 
   std::vector<base::Closure> histogram_callbacks_;
 
+  bool disable_hot_tracking_;
+
   DISALLOW_COPY_AND_ASSIGN(BrowserAccessibilityStateImpl);
 };