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

Unified Diff: third_party/WebKit/Source/core/dom/SelectorQuery.cpp

Issue 2390543002: Reflow comments in core/dom/. (Closed)
Patch Set: Reformat comments in core/dom/. Created 4 years, 2 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: third_party/WebKit/Source/core/dom/SelectorQuery.cpp
diff --git a/third_party/WebKit/Source/core/dom/SelectorQuery.cpp b/third_party/WebKit/Source/core/dom/SelectorQuery.cpp
index d4f2ef6e4c7ea4b073d0c24b8fb3ec9e458844ae..e614db0fdc9fc8724a3f95f9dedb47bce63f5b77 100644
--- a/third_party/WebKit/Source/core/dom/SelectorQuery.cpp
+++ b/third_party/WebKit/Source/core/dom/SelectorQuery.cpp
@@ -253,19 +253,22 @@ inline bool ancestorHasClassName(ContainerNode& rootNode,
return false;
}
-// If returns true, traversalRoots has the elements that may match the selector query.
+// If returns true, traversalRoots has the elements that may match the selector
+// query.
//
-// If returns false, traversalRoots has the rootNode parameter or descendants of rootNode representing
-// the subtree for which we can limit the querySelector traversal.
+// If returns false, traversalRoots has the rootNode parameter or descendants of
+// rootNode representing the subtree for which we can limit the querySelector
+// traversal.
//
-// The travseralRoots may be empty, regardless of the returned bool value, if this method finds that the selectors won't
-// match any element.
+// The travseralRoots may be empty, regardless of the returned bool value, if
+// this method finds that the selectors won't match any element.
template <typename SelectorQueryTrait>
void SelectorDataList::findTraverseRootsAndExecute(
ContainerNode& rootNode,
typename SelectorQueryTrait::OutputType& output) const {
- // We need to return the matches in document order. To use id lookup while there is possiblity of multiple matches
- // we would need to sort the results. For now, just traverse the document in that case.
+ // We need to return the matches in document order. To use id lookup while
+ // there is possiblity of multiple matches we would need to sort the
+ // results. For now, just traverse the document in that case.
DCHECK_EQ(m_selectors.size(), 1u);
bool isRightmostSelector = true;
@@ -300,8 +303,8 @@ void SelectorDataList::findTraverseRootsAndExecute(
return;
}
- // If we have both CSSSelector::Id and CSSSelector::Class at the same time, we should use Id
- // to find traverse root.
+ // If we have both CSSSelector::Id and CSSSelector::Class at the same time,
+ // we should use Id to find traverse root.
if (!SelectorQueryTrait::shouldOnlyMatchFirstElement && !startFromParent &&
selector->match() == CSSSelector::Class) {
if (isRightmostSelector) {
@@ -312,7 +315,8 @@ void SelectorDataList::findTraverseRootsAndExecute(
output);
return;
}
- // Since there exists some ancestor element which has the class name, we need to see all children of rootNode.
+ // Since there exists some ancestor element which has the class name, we
+ // need to see all children of rootNode.
if (ancestorHasClassName(rootNode, selector->value())) {
executeForTraverseRoot<SelectorQueryTrait>(*m_selectors[0], &rootNode,
DoesNotMatchTraverseRoots,
@@ -425,8 +429,9 @@ void SelectorDataList::executeSlow(
}
}
-// FIXME: Move the following helper functions, authorShadowRootOf, firstWithinTraversingShadowTree,
-// nextTraversingShadowTree to the best place, e.g. NodeTraversal.
+// FIXME: Move the following helper functions, authorShadowRootOf,
+// firstWithinTraversingShadowTree, nextTraversingShadowTree to the best place,
+// e.g. NodeTraversal.
static ShadowRoot* authorShadowRootOf(const ContainerNode& node) {
if (!node.isElementNode() || !isShadowHost(&node))
return nullptr;
@@ -534,7 +539,8 @@ void SelectorDataList::execute(
const CSSSelector& selector = *m_selectors[0];
const CSSSelector& firstSelector = selector;
- // Fast path for querySelector*('#id'), querySelector*('tag#id'), querySelector*('tag[id=example]').
+ // Fast path for querySelector*('#id'), querySelector*('tag#id'),
+ // querySelector*('tag[id=example]').
if (const CSSSelector* idSelector = selectorForIdLookup(firstSelector)) {
const AtomicString& idToMatch = idSelector->value();
if (rootNode.treeScope().containsMultipleElementsWithId(idToMatch)) {

Powered by Google App Engine
This is Rietveld 408576698