Bug 1249730 - make TreeWalker bi-directional, follow up fix, r=yzen
authorAlexander Surkov <surkov.alexander@gmail.com>
Fri, 11 Mar 2016 11:14:29 -0500
changeset 288261 d80794ec133731287eeadff04797d4cd93641509
parent 288260 c8b421423059068586d436fb7fbab9be5059faf5
child 288262 4916251eb38968263c69c633fee264d129570768
push id73366
push usersurkov.alexander@gmail.com
push dateFri, 11 Mar 2016 16:35:17 +0000
treeherdermozilla-inbound@d80794ec1337 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersyzen
bugs1249730
milestone48.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1249730 - make TreeWalker bi-directional, follow up fix, r=yzen
accessible/base/TreeWalker.cpp
--- a/accessible/base/TreeWalker.cpp
+++ b/accessible/base/TreeWalker.cpp
@@ -233,17 +233,17 @@ TreeWalker::Prev()
       bool skipSubtree = false;
       Accessible* child = AccessibleFor(childNode, eWalkCache, &skipSubtree);
       if (child) {
         return child;
       }
 
       // Walk down into subtree to find accessibles.
       if (!skipSubtree && childNode->IsElement()) {
-        top = PushState(childNode, true);
+        top = PushState(childNode, false);
       }
     }
     top = PopState();
   }
 
   // Move to a previous node relative the anchor node within the context
   // subtree if asked.
   if (mFlags != eWalkContextTree) {