Get rid of flat tree version of createVisibleSelection() taking two PositionInFlatTree

third_party/WebKit/Source/core/editing/VisibleSelection.cpp

Index: third_party/WebKit/Source/core/editing/VisibleSelection.cpp
diff --git a/third_party/WebKit/Source/core/editing/VisibleSelection.cpp b/third_party/WebKit/Source/core/editing/VisibleSelection.cpp
index 7bff35ae99f6747efcc4e0c4feb8d0ee8f18f826..0d7f82dd8c0198ef3f9f0cc5ab7c1cbc4d38c0b9 100644
--- a/third_party/WebKit/Source/core/editing/VisibleSelection.cpp
+++ b/third_party/WebKit/Source/core/editing/VisibleSelection.cpp
@@ -128,22 +128,6 @@ VisibleSelectionInFlatTree createVisibleSelection(
   return VisibleSelectionInFlatTree::create(selection);
 }
 
-VisibleSelectionInFlatTree createVisibleSelection(
-    const PositionInFlatTree& base,
-    const PositionInFlatTree& extent,
-    TextAffinity affinity,
-    bool isDirectional) {
-  DCHECK(!needsLayoutTreeUpdate(base));
-  DCHECK(!needsLayoutTreeUpdate(extent));
-  // TODO(yosin): We should use |Builder::setBaseAndExtent()| once we get rid
-  // of callers passing |base.istNull()| but |extent.isNotNull()|.
-  SelectionInFlatTree::Builder builder;
-  builder.setBaseAndExtentDeprecated(base, extent)
-      .setAffinity(affinity)
-      .setIsDirectional(isDirectional);
-  return createVisibleSelection(builder.build());
-}
-
 template <typename Strategy>
 static SelectionType computeSelectionType(
     const PositionTemplate<Strategy>& start,