| Index: third_party/WebKit/Source/core/editing/Position.cpp
|
| diff --git a/third_party/WebKit/Source/core/editing/Position.cpp b/third_party/WebKit/Source/core/editing/Position.cpp
|
| index d75197ffa76232974ae0b3605ecdd10b4db513ff..77313e6fd4725f31767777abc8b3faeb762043a7 100644
|
| --- a/third_party/WebKit/Source/core/editing/Position.cpp
|
| +++ b/third_party/WebKit/Source/core/editing/Position.cpp
|
| @@ -175,7 +175,8 @@ PositionTemplate<Strategy>::parentAnchoredEquivalent() const {
|
| if (!m_anchorNode)
|
| return PositionTemplate<Strategy>();
|
|
|
| - // FIXME: This should only be necessary for legacy positions, but is also needed for positions before and after Tables
|
| + // FIXME: This should only be necessary for legacy positions, but is also
|
| + // needed for positions before and after Tables
|
| if (m_offset == 0 && !isAfterAnchorOrAfterChildren()) {
|
| if (Strategy::parent(*m_anchorNode) &&
|
| (Strategy::editingIgnoresContent(m_anchorNode.get()) ||
|
| @@ -348,8 +349,8 @@ template <typename Strategy>
|
| bool PositionTemplate<Strategy>::atFirstEditingPositionForNode() const {
|
| if (isNull())
|
| return true;
|
| - // FIXME: Position before anchor shouldn't be considered as at the first editing position for node
|
| - // since that position resides outside of the node.
|
| + // FIXME: Position before anchor shouldn't be considered as at the first
|
| + // editing position for node since that position resides outside of the node.
|
| switch (m_anchorType) {
|
| case PositionAnchorType::OffsetInAnchor:
|
| return m_offset == 0;
|
| @@ -400,10 +401,11 @@ bool PositionTemplate<Strategy>::atEndOfTree() const {
|
| template <typename Strategy>
|
| PositionTemplate<Strategy> PositionTemplate<Strategy>::inParentBeforeNode(
|
| const Node& node) {
|
| - // FIXME: This should DCHECK(node.parentNode())
|
| - // At least one caller currently hits this ASSERT though, which indicates
|
| - // that the caller is trying to make a position relative to a disconnected node (which is likely an error)
|
| - // Specifically, editing/deleting/delete-ligature-001.html crashes with DCHECK(node->parentNode())
|
| + // FIXME: This should DCHECK(node.parentNode()). At least one caller currently
|
| + // hits this DCHECK though, which indicates that the caller is trying to make
|
| + // a position relative to a disconnected node (which is likely an error)
|
| + // Specifically, editing/deleting/delete-ligature-001.html crashes with
|
| + // DCHECK(node->parentNode())
|
| return PositionTemplate<Strategy>(Strategy::parent(node),
|
| Strategy::index(node));
|
| }
|
|
|