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

Unified Diff: third_party/WebKit/Source/core/xml/XPathFunctions.cpp

Issue 2389883005: reflow comments in core/xml* (Closed)
Patch Set: 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/xml/XPathFunctions.cpp
diff --git a/third_party/WebKit/Source/core/xml/XPathFunctions.cpp b/third_party/WebKit/Source/core/xml/XPathFunctions.cpp
index 2de19a1eaa9fe9620dc319d18c89286529d50f0f..8fe6c2a563024496d1667ab559ea5a7ff1642203 100644
--- a/third_party/WebKit/Source/core/xml/XPathFunctions.cpp
+++ b/third_party/WebKit/Source/core/xml/XPathFunctions.cpp
@@ -309,7 +309,8 @@ inline bool Interval::contains(int value) const {
void Function::setArguments(HeapVector<Member<Expression>>& args) {
DCHECK(!subExprCount());
- // Some functions use context node as implicit argument, so when explicit arguments are added, they may no longer be context node sensitive.
+ // Some functions use context node as implicit argument, so when explicit
+ // arguments are added, they may no longer be context node sensitive.
if (m_name != "lang" && !args.isEmpty())
setIsContextNodeSensitive(false);
@@ -358,8 +359,9 @@ Value FunId::evaluate(EvaluationContext& context) const {
while (endPos < length && !isWhitespace(idList[endPos]))
++endPos;
- // If there are several nodes with the same id, id() should return the first one.
- // In WebKit, getElementById behaves so, too, although its behavior in this case is formally undefined.
+ // If there are several nodes with the same id, id() should return the first
+ // one. In WebKit, getElementById behaves so, too, although its behavior in
jbroman 2016/10/04 19:56:51 ditto
+ // this case is formally undefined.
Node* node = contextScope.getElementById(
AtomicString(idList.substring(startPos, endPos - startPos)));
if (node && resultSet.add(node).isNewEntry)
@@ -374,7 +376,8 @@ Value FunId::evaluate(EvaluationContext& context) const {
}
static inline String expandedNameLocalPart(Node* node) {
- // The local part of an XPath expanded-name matches DOM local name for most node types, except for namespace nodes and processing instruction nodes.
+ // The local part of an XPath expanded-name matches DOM local name for most
+ // node types, except for namespace nodes and processing instruction nodes.
// But note that Blink does not support namespace nodes.
switch (node->getNodeType()) {
case Node::kElementNode:
@@ -654,8 +657,9 @@ Value FunSum::evaluate(EvaluationContext& context) const {
double sum = 0.0;
const NodeSet& nodes = a.toNodeSet(&context);
- // To be really compliant, we should sort the node-set, as floating point addition is not associative.
- // However, this is unlikely to ever become a practical issue, and sorting is slow.
+ // To be really compliant, we should sort the node-set, as floating point
+ // addition is not associative. However, this is unlikely to ever become a
+ // practical issue, and sorting is slow.
for (unsigned i = 0; i < nodes.size(); i++)
sum += Value(stringValue(nodes[i])).toNumber();

Powered by Google App Engine
This is Rietveld 408576698