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 288291 d80794ec133731287eeadff04797d4cd93641509
parent 288290 c8b421423059068586d436fb7fbab9be5059faf5
child 288292 4916251eb38968263c69c633fee264d129570768
push id30079
push userryanvm@gmail.com
push dateSat, 12 Mar 2016 20:24:19 +0000
treeherdermozilla-central@d1d47ba19ce9 [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) {