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

Unified Diff: third_party/WebKit/Source/core/editing/Position.h

Issue 2399663003: Reflow comments in //third_party/WebKit/Source/core/editing (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/editing/Position.h
diff --git a/third_party/WebKit/Source/core/editing/Position.h b/third_party/WebKit/Source/core/editing/Position.h
index 1f63b31ace86e0c54b232a66eaad52f86e978594..18176c1f8bd20c3d45bfd96da1df459d75708d78 100644
--- a/third_party/WebKit/Source/core/editing/Position.h
+++ b/third_party/WebKit/Source/core/editing/Position.h
@@ -94,15 +94,20 @@ class CORE_TEMPLATE_CLASS_EXPORT PositionTemplate {
return m_anchorType == PositionAnchorType::OffsetInAnchor;
}
- // These are always DOM compliant values. Editing positions like [img, 0] (aka [img, before])
- // will return img->parentNode() and img->nodeIndex() from these functions.
- Node* computeContainerNode()
- const; // null for a before/after position anchored to a node with no parent
+ // These are always DOM compliant values. Editing positions like [img, 0]
+ // (aka [img, before]) will return img->parentNode() and img->nodeIndex() from
+ // these functions.
- int computeOffsetInContainerNode()
- const; // O(n) for before/after-anchored positions, O(1) for parent-anchored positions
- PositionTemplate<Strategy> parentAnchoredEquivalent()
- const; // Convenience method for DOM positions that also fixes up some positions for editing
+ // null for a before/after position anchored to a node with no parent
+ Node* computeContainerNode() const;
+
+ // O(n) for before/after-anchored positions, O(1) for parent-anchored
+ // positions
+ int computeOffsetInContainerNode() const;
+
+ // Convenience method for DOM positions that also fixes up some positions for
+ // editing
+ PositionTemplate<Strategy> parentAnchoredEquivalent() const;
// Returns |PositionIsAnchor| type |Position| which is compatible with
// |RangeBoundaryPoint| as safe to pass |Range| constructor. Return value
@@ -127,7 +132,8 @@ class CORE_TEMPLATE_CLASS_EXPORT PositionTemplate {
// anchor node.
int computeEditingOffset() const;
- // These are convenience methods which are smart about whether the position is neighbor anchored or parent anchored
+ // These are convenience methods which are smart about whether the position is
+ // neighbor anchored or parent anchored
Node* computeNodeBeforePosition() const;
Node* computeNodeAfterPosition() const;
@@ -204,9 +210,10 @@ class CORE_TEMPLATE_CLASS_EXPORT PositionTemplate {
}
Member<Node> m_anchorNode;
- // m_offset can be the offset inside m_anchorNode, or if editingIgnoresContent(m_anchorNode)
- // returns true, then other places in editing will treat m_offset == 0 as "before the anchor"
- // and m_offset > 0 as "after the anchor node". See parentAnchoredEquivalent for more info.
+ // m_offset can be the offset inside m_anchorNode, or if
+ // editingIgnoresContent(m_anchorNode) returns true, then other places in
+ // editing will treat m_offset == 0 as "before the anchor" and m_offset > 0 as
+ // "after the anchor node". See parentAnchoredEquivalent for more info.
int m_offset;
PositionAnchorType m_anchorType;
};
@@ -229,7 +236,8 @@ bool operator==(const PositionTemplate<Strategy>& a,
return false;
if (!a.isOffsetInAnchor()) {
- // Note: |m_offset| only has meaning when |PositionAnchorType::OffsetInAnchor|.
+ // Note: |m_offset| only has meaning when
+ // |PositionAnchorType::OffsetInAnchor|.
return true;
}
« no previous file with comments | « third_party/WebKit/Source/core/editing/PlainTextRange.cpp ('k') | third_party/WebKit/Source/core/editing/Position.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698