Chromium Code Reviews| Index: third_party/WebKit/Source/core/dom/RangeBoundaryPoint.h |
| diff --git a/third_party/WebKit/Source/core/dom/RangeBoundaryPoint.h b/third_party/WebKit/Source/core/dom/RangeBoundaryPoint.h |
| index 32d2681fea6b9042c17366bdf35f5b4a1d17d74b..c4aead765ab63dc820710103d405f60df266fac4 100644 |
| --- a/third_party/WebKit/Source/core/dom/RangeBoundaryPoint.h |
| +++ b/third_party/WebKit/Source/core/dom/RangeBoundaryPoint.h |
| @@ -129,10 +129,7 @@ inline bool RangeBoundaryPoint::isOffsetValid() const { |
| inline const Position RangeBoundaryPoint::toPosition() const { |
| ensureOffsetIsValid(); |
| - // TODO(yosin): We should return |Position::beforeAnchor| when |
|
yosin_UTC9
2017/03/30 05:44:58
Could you keep this TODO? We still want to return
yoichio
2017/03/30 07:04:09
Could you explain why ?
|
| - // |m_containerNode| isn't |Text| node. |
| - return Position::editingPositionOf(m_containerNode.get(), |
| - m_offsetInContainer); |
| + return Position(m_containerNode.get(), m_offsetInContainer); |
| } |
| inline unsigned RangeBoundaryPoint::offset() const { |