15252 2011-12-17 01:28:12 +0000 Documentation error - TreeWalker.nextNode() 2011-12-17 11:01:36 +0000 1 1 1 Unclassified WebAppsWG DOM unspecified PC Windows NT RESOLVED WORKSFORME P2 minor --- 1 brandonyoyoslade annevk mike www-dom public-webapps-bugzilla oldest_to_newest 61730 0 brandonyoyoslade 2011-12-17 01:28:12 +0000 The current instructions for implementing this method can be found at http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#dom-treewalker-nextnode. Step 3.2 states that implementors should: "If a node is following node and is not following root, set node to the first such node. Otherwise, run these substeps again." Earlier in the steps, you were instructed to set node to currentNode. Since currentNode is always a descendant of root, this means that the first "if" statement is never true - the "otherwise" or "else" clause is executed immediately. After some thought, I believe that you meant the following instead: "If a node is following node and is A DESCENDANT OF root, set node to the first such node. Otherwise, run these substeps again." If this is what you meant, then please change it. If not, then please clear up exactly what you meant. Also, there is a step 3.3, so consider changing the second sentence of 3.2 to "Otherwise, SKIP THE REMAINING SUBSTEPS AND THEN run them again." or "Otherwise, SKIP STEP 3 AND THEN run these substeps again." 61738 1 annevk 2011-12-17 10:45:53 +0000 currentNode can be set to anything so it does not have to be a descendant. I have made the clarification in the end. 61741 2 annevk 2011-12-17 11:01:36 +0000 https://bitbucket.org/ms2ger/dom-core/changeset/c5238c17e1f4