Let RangeBoundaryPoint::toPoisition return Position not considering Editing

third_party/WebKit/Source/core/dom/RangeBoundaryPoint.h

 /*
  * Copyright (C) 2008 Apple Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 111 matching lines @@
   if (m_offsetInContainer == invalidOffset) {
     DCHECK(!m_containerNode->isTextNode());
     return false;
   }
   return domTreeVersion() == m_domTreeVersion ||
          m_containerNode->isCharacterDataNode();
 }
 
 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 |Position::beforeAnchor|.

[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 {
   ensureOffsetIsValid();
   return m_offsetInContainer;
 }
 
 inline void RangeBoundaryPoint::clear() {
   m_containerNode.clear();
   m_offsetInContainer = 0;
@@ skipping 80 matching lines @@
   } else {
     if (a.offset() != b.offset())
       return false;
   }
   return true;
 }
 
 }  // namespace blink
 
 #endif