Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(845)

Unified Diff: Source/core/dom/NodeTraversal.h

Issue 420603012: Remove ContainerNode overload for NodeTraversal::nextSkippingChildren() (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Remove the one from ElementTraversal as well Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/dom/ElementTraversal.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/dom/NodeTraversal.h
diff --git a/Source/core/dom/NodeTraversal.h b/Source/core/dom/NodeTraversal.h
index b27d12d499e6a2c8fcc5d5914cdc55e6149d06b9..a3dbe6ed57f75c3530b24ad318694325a7c1d3c9 100644
--- a/Source/core/dom/NodeTraversal.h
+++ b/Source/core/dom/NodeTraversal.h
@@ -42,10 +42,8 @@ public:
static Node* next(const ContainerNode& current, const Node* stayWithin) { return traverseNextTemplate(current, stayWithin); }
// Like next, but skips children and starts with the next sibling.
- static Node* nextSkippingChildren(const Node& current) { return traverseNextSkippingChildrenTemplate(current); }
- static Node* nextSkippingChildren(const ContainerNode& current) { return traverseNextSkippingChildrenTemplate(current); }
- static Node* nextSkippingChildren(const Node& current, const Node* stayWithin) { return traverseNextSkippingChildrenTemplate(current, stayWithin); }
- static Node* nextSkippingChildren(const ContainerNode& current, const Node* stayWithin) { return traverseNextSkippingChildrenTemplate(current, stayWithin); }
+ static Node* nextSkippingChildren(const Node&);
+ static Node* nextSkippingChildren(const Node&, const Node* stayWithin);
// Does a reverse pre-order traversal to find the node that comes before the current one in document order
static Node* lastWithin(const ContainerNode&);
@@ -73,10 +71,6 @@ private:
static Node* traverseNextTemplate(NodeType&);
template <class NodeType>
static Node* traverseNextTemplate(NodeType&, const Node* stayWithin);
- template <class NodeType>
- static Node* traverseNextSkippingChildrenTemplate(NodeType&);
- template <class NodeType>
- static Node* traverseNextSkippingChildrenTemplate(NodeType&, const Node* stayWithin);
};
template <class NodeType>
@@ -101,16 +95,14 @@ inline Node* NodeTraversal::traverseNextTemplate(NodeType& current, const Node*
return nextAncestorSibling(current, stayWithin);
}
-template <class NodeType>
-inline Node* NodeTraversal::traverseNextSkippingChildrenTemplate(NodeType& current)
+inline Node* NodeTraversal::nextSkippingChildren(const Node& current)
{
if (current.nextSibling())
return current.nextSibling();
return nextAncestorSibling(current);
}
-template <class NodeType>
-inline Node* NodeTraversal::traverseNextSkippingChildrenTemplate(NodeType& current, const Node* stayWithin)
+inline Node* NodeTraversal::nextSkippingChildren(const Node& current, const Node* stayWithin)
{
if (current == stayWithin)
return 0;
« no previous file with comments | « Source/core/dom/ElementTraversal.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698