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

Unified Diff: Source/core/dom/shadow/ComposedTreeTraversal.cpp

Issue 1030153002: Add ComposedTreeTraversal::lastWithin/lastWithinOrSelf (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 9 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
Index: Source/core/dom/shadow/ComposedTreeTraversal.cpp
diff --git a/Source/core/dom/shadow/ComposedTreeTraversal.cpp b/Source/core/dom/shadow/ComposedTreeTraversal.cpp
index 8966b8faf5431cd71aa70284474b6687d23da9be..ebb25a152ab51132c6f8ea2ed689da3bf0b91572 100644
--- a/Source/core/dom/shadow/ComposedTreeTraversal.cpp
+++ b/Source/core/dom/shadow/ComposedTreeTraversal.cpp
@@ -241,4 +241,18 @@ unsigned ComposedTreeTraversal::countChildren(const Node& node)
return count;
}
+Node* ComposedTreeTraversal::lastWithin(const Node& current)
+{
hayato 2015/03/26 03:28:27 Could you insert assertions like other public func
hajimehoshi 2015/03/26 03:43:26 Done.
+ Node* descendant = lastChild(current);
+ for (Node* child = descendant; child; child = lastChild(*child))
+ descendant = child;
+ return descendant;
+}
+
+Node& ComposedTreeTraversal::lastWithinOrSelf(Node& current)
+{
+ Node* lastDescendant = ComposedTreeTraversal::lastWithin(current);
+ return lastDescendant ? *lastDescendant : current;
+}
+
} // namespace

Powered by Google App Engine
This is Rietveld 408576698