Move computeInlineBoxPosition() to VisibleUnits.{cpp,h} from Position.{cpp,h} and VisiblePosition.h

Source/core/editing/Position.h

Index: Source/core/editing/Position.h
diff --git a/Source/core/editing/Position.h b/Source/core/editing/Position.h
index 6aa73a495a5194e87d2d23ca466b116a28cf3a96..4de46fa26712a851f1be4f18d5ac729bfec47fcb 100644
--- a/Source/core/editing/Position.h
+++ b/Source/core/editing/Position.h
@@ -39,28 +39,12 @@
 namespace blink {
 
 class Element;
-class InlineBox;
 class Node;
 class LayoutObject;
 class Text;
 enum class TextAffinity;
 class TreeScope;
 
-struct InlineBoxPosition {
-    InlineBox* inlineBox;
-    int offsetInBox;
-
-    InlineBoxPosition()
-        : inlineBox(nullptr), offsetInBox(0)
-    {
-    }
-
-    InlineBoxPosition(InlineBox* inlineBox, int offsetInBox)
-        : inlineBox(inlineBox), offsetInBox(offsetInBox)
-    {
-    }
-};
-
 enum PositionMoveType {
     CodePoint, // Move by a single code point.
     Character, // Move to the next Unicode character break.
@@ -423,17 +407,6 @@ inline PositionInComposedTree fromPositionInDOMTree<EditingInComposedTreeStrateg
     return toPositionInComposedTree(position);
 }
 
-// TODO(yosin) |isRenderedCharacter()| should be removed, and we should use
-// |VisiblePosition::characterAfter()| and |VisiblePosition::characterBefore()|.
-// TODO(yosin) We should move |isRenderedCharacter()| to "VisibleUnits.cpp",
-// since it is used only in "editing/commands/"
-CORE_EXPORT bool isRenderedCharacter(const Position&);
-// TODO(yosin) We should move |computeInlineBoxPosition()| to "VisibleUnits.cpp"
-CORE_EXPORT InlineBoxPosition computeInlineBoxPosition(const Position&, TextAffinity);
-CORE_EXPORT InlineBoxPosition computeInlineBoxPosition(const Position&, TextAffinity, TextDirection primaryDirection);
-CORE_EXPORT InlineBoxPosition computeInlineBoxPosition(const PositionInComposedTree&, TextAffinity);
-CORE_EXPORT InlineBoxPosition computeInlineBoxPosition(const PositionInComposedTree&, TextAffinity, TextDirection primaryDirection);
-
 // TODO(yosin) We should move |inRenderedText()| to "VisibleUnits.h" for
 // reduce dependency of |LayoutObject| in |Position| class.
 CORE_EXPORT bool inRenderedText(const Position&);