Use setSequentialFocusNavigationStartingPoint in ChromeVox

third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp

 /*
  * Copyright (C) 2012, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 2151 matching lines @@
 void AXNodeObject::decrement() {
   UserGestureIndicator gestureIndicator(
       UserGestureToken::create(UserGestureToken::NewGesture));
   alterSliderValue(false);
 }
 
 void AXNodeObject::setSequentialFocusNavigationStartingPoint() {
   if (!getNode())
     return;
 
+  getNode()->document().clearFocusedElement();

[David Tseng, 2016/10/24 21:51:32]

Will this result in any ax events?

[dmazzoni, 2016/10/26 16:20:22]

Yes, but the existing mechanism you added suppresses it from
speaking.

[David Tseng, 2016/10/26 17:03:26]

Great, so you didn't get any repeated text/odd focus syncing while navigating
with ChromeVox.

   getNode()->document().setSequentialFocusNavigationStartingPoint(getNode());
 }
 
 void AXNodeObject::childrenChanged() {
   // This method is meant as a quick way of marking a portion of the
   // accessibility tree dirty.
   if (!getNode() && !getLayoutObject())
     return;
 
   // If this is not part of the accessibility tree because an ancestor
@@ skipping 779 matching lines @@
   }
   return placeholder;
 }
 
 DEFINE_TRACE(AXNodeObject) {
   visitor->trace(m_node);
   AXObject::trace(visitor);
 }
 
 }  // namespace blink