Introduce {next,previous}PositionOf() as replacement of {next,previous}() member functions in Positi

Source/core/editing/Position.h

 /*
  * Copyright (C) 2004, 2006, 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 27 matching lines @@
 
 namespace blink {
 
 class Element;
 class Node;
 class LayoutObject;
 class Text;
 enum class TextAffinity;
 class TreeScope;
 
-enum PositionMoveType {
+enum class PositionMoveType {
     CodePoint, // Move by a single code point.
     Character, // Move to the next Unicode character break.
     BackwardDeletion // Subject to platform conventions.
 };
 
 enum class PositionAnchorType : unsigned {
     OffsetInAnchor,
     BeforeAnchor,
     AfterAnchor,
     BeforeChildren,
@@ skipping 83 matching lines @@
 
     Node* anchorNode() const { return m_anchorNode.get(); }
 
     Document* document() const { return m_anchorNode ? &m_anchorNode->document() : 0; }
     bool inDocument() const { return m_anchorNode && m_anchorNode->inDocument(); }
 
     bool isNull() const { return !m_anchorNode; }
     bool isNotNull() const { return m_anchorNode; }
     bool isOrphan() const { return m_anchorNode && !m_anchorNode->inDocument(); }
 
-    // Move up or down the DOM by one position.
-    // Offsets are computed using layout text for nodes that have layoutObjects - but note that even when
-    // using composed characters, the result may be inside a single user-visible character if a ligature is formed.
-    PositionAlgorithm<Strategy> previous(PositionMoveType = CodePoint) const;
-    PositionAlgorithm<Strategy> next(PositionMoveType = CodePoint) const;
-
     int compareTo(const PositionAlgorithm<Strategy>&) const;
 
     // These can be either inside or just before/after the node, depending on
     // if the node is ignored by editing or not.
     // FIXME: These should go away. They only make sense for legacy positions.
     bool atFirstEditingPositionForNode() const;
     bool atLastEditingPositionForNode() const;
 
     bool atStartOfTree() const;
     bool atEndOfTree() const;
@@ skipping 232 matching lines @@
 {
     return position;
 }
 
 template <>
 inline PositionInComposedTree fromPositionInDOMTree<EditingInComposedTreeStrategy>(const Position& position)
 {
     return toPositionInComposedTree(position);
 }
 
+// TODO(yosin) We should move |previousPositionOf()| and |nextPositionOf()|
+// to "EditingUtilities.h" with |uncheckedPreviousOffset()| and
+// |uncheckedNextOffset()|.
+// Move up or down the DOM by one position.
+// Offsets are computed using layout text for nodes that have layoutObjects -
+// but note that even when using composed characters, the result may be inside
+// a single user-visible character if a ligature is formed.
+CORE_EXPORT Position previousPositionOf(const Position&, PositionMoveType);
+CORE_EXPORT Position nextPositionOf(const Position&, PositionMoveType);
+CORE_EXPORT PositionInComposedTree previousPositionOf(const PositionInComposedTree&, PositionMoveType);
+CORE_EXPORT PositionInComposedTree nextPositionOf(const PositionInComposedTree&, PositionMoveType);
+
 CORE_EXPORT int uncheckedPreviousOffset(const Node*, int current);
 CORE_EXPORT int uncheckedNextOffset(const Node*, int current);
 
 } // namespace blink
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 void showTree(const blink::Position&);
 void showTree(const blink::Position*);
 #endif
 
 #endif // Position_h