Make VisiblePosition constructor private

Source/core/editing/VisiblePosition.h

Index: Source/core/editing/VisiblePosition.h
diff --git a/Source/core/editing/VisiblePosition.h b/Source/core/editing/VisiblePosition.h
index 29493c0515b42e5d861a9fe24dfdf56f14b9e9d9..1422ef15ba57df6eefaca4e9ccc33dab9eb69f09 100644
--- a/Source/core/editing/VisiblePosition.h
+++ b/Source/core/editing/VisiblePosition.h
@@ -72,11 +72,6 @@ class CORE_EXPORT VisiblePosition final {
 public:
     VisiblePosition() : m_affinity(VP_DEFAULT_AFFINITY) { }
 
-    // TODO(yosin) We should use |createVisiblePosition()| instead of
-    // constructor, and make constructors to have private accessibility, since
-    // constructors aren't simple, e.g. they update layout tree.
-    explicit VisiblePosition(const Position&, TextAffinity = VP_DEFAULT_AFFINITY);
-
     // Node: Other than |createVisiblePosition()|, we should not use
     // |createWithoutCanonicalization()|.
     static VisiblePosition createWithoutCanonicalization(const PositionWithAffinity& canonicalized);
@@ -106,6 +101,8 @@ public:
 #endif
 
 private:
+    explicit VisiblePosition(const Position&, TextAffinity);
+
     // TODO(yosin) We should use |PositionWithAffinity| to make
     // |toPositionWithAffinity()| simpler.
     Position m_deepPosition;