Reviewed by harrison
authorjusting <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Aug 2006 21:33:18 +0000 (21:33 +0000)
committerjusting <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 23 Aug 2006 21:33:18 +0000 (21:33 +0000)
        Removed the poorly named next/previousVisiblePosition
        and use next/previousCandidate and next/previousVisuallyDistinctCandidate.
        Removed the unused VisiblePosition::maxOffset()

        * editing/VisiblePosition.cpp:
        (WebCore::VisiblePosition::next):
        (WebCore::VisiblePosition::previous):
        (WebCore::VisiblePosition::canonicalPosition):
        * editing/VisiblePosition.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15996 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/editing/VisiblePosition.cpp
WebCore/editing/VisiblePosition.h

index 60610e8648b5e5c16e555fc751982ade43f72fa2..5cec33724fd3a7b1d647f8ecf87fd8b83aba155a 100644 (file)
@@ -1,3 +1,17 @@
+2006-08-23  Justin Garcia  <justin.garcia@apple.com>
+
+        Reviewed by harrison
+        
+        Removed the poorly named next/previousVisiblePosition
+        and use next/previousCandidate and next/previousVisuallyDistinctCandidate.
+        Removed the unused VisiblePosition::maxOffset()
+
+        * editing/VisiblePosition.cpp:
+        (WebCore::VisiblePosition::next):
+        (WebCore::VisiblePosition::previous):
+        (WebCore::VisiblePosition::canonicalPosition):
+        * editing/VisiblePosition.h: 
+
 2006-08-23  Nikolas Zimmermann  <zimmermann@kde.org>
 
         Reviewed by Alexey. Landed by rwlbuis.
index b4ff4935442e9b8c0f37a8c351a39cbfd3b7cadc..09e2c894eb8a4504bc194ae493e674b5d2899388 100644 (file)
@@ -62,7 +62,7 @@ void VisiblePosition::init(const Position& position, EAffinity affinity)
 
 VisiblePosition VisiblePosition::next(bool stayInEditableContent) const
 {
-    VisiblePosition next(nextVisiblePosition(m_deepPosition), m_affinity);
+    VisiblePosition next(nextVisuallyDistinctCandidate(m_deepPosition), m_affinity);
     
     if (!stayInEditableContent || next.isNull())
         return next;
@@ -81,7 +81,7 @@ VisiblePosition VisiblePosition::next(bool stayInEditableContent) const
 VisiblePosition VisiblePosition::previous(bool stayInEditableContent) const
 {
     // find first previous DOM position that is visible
-    Position pos = previousVisiblePosition(m_deepPosition);
+    Position pos = previousVisuallyDistinctCandidate(m_deepPosition);
     
     // return null visible position if there is no previous visible position
     if (pos.atStart())
@@ -114,50 +114,6 @@ VisiblePosition VisiblePosition::previous(bool stayInEditableContent) const
     return lastEditablePositionBeforePositionInRoot(prev.deepEquivalent(), highestRoot);
 }
 
-Position VisiblePosition::previousVisiblePosition(const Position& pos)
-{
-    if (!pos.inRenderedContent()) {
-        Position current = pos;
-        while (!current.atStart()) {
-            current = current.previous(UsingComposedCharacters);
-            if (current.inRenderedContent())
-                return current;
-        }
-        return Position();
-    }
-
-    Position downstreamStart = pos.downstream();
-    Position current = pos;
-    while (!current.atStart()) {
-        current = current.previous(UsingComposedCharacters);
-        if (current.inRenderedContent() && downstreamStart != current.downstream())
-            return current;
-    }
-    return Position();
-}
-
-Position VisiblePosition::nextVisiblePosition(const Position& pos)
-{
-    if (!pos.inRenderedContent()) {
-        Position current = pos;
-        while (!current.atEnd()) {
-            current = current.next(UsingComposedCharacters);
-            if (current.inRenderedContent())
-                return current;
-        }
-        return Position();
-    }
-
-    Position downstreamStart = pos.downstream();
-    Position current = pos;
-    while (!current.atEnd()) {
-        current = current.next(UsingComposedCharacters);
-        if (current.inRenderedContent() && downstreamStart != current.downstream())
-            return current;
-    }
-    return Position();
-}
-
 Position VisiblePosition::canonicalPosition(const Position& position)
 {
     // FIXME (9535):  Canonicalizing to the leftmost candidate means that if we're at a line wrap, we will 
@@ -180,8 +136,8 @@ Position VisiblePosition::canonicalPosition(const Position& position)
 
     // When neither upstream or downstream gets us to a candidate (upstream/downstream won't leave 
     // blocks or enter new ones), we search forward and backward until we find one.
-    Position next = nextVisiblePosition(position);
-    Position prev = previousVisiblePosition(position);
+    Position next = nextCandidate(position);
+    Position prev = previousCandidate(position);
     Node* nextNode = next.node();
     Node* prevNode = prev.node();
 
@@ -218,11 +174,6 @@ Position VisiblePosition::canonicalPosition(const Position& position)
     return next;
 }
 
-int VisiblePosition::maxOffset(const Node *node)
-{
-    return node->offsetInCharacters() ? (int)static_cast<const CharacterData *>(node)->length() : (int)node->childNodeCount();
-}
-
 UChar VisiblePosition::characterAfter() const
 {
     // We canonicalize to the first of two equivalent candidates, but the second of the two candidates
index e1693f640409bf8ebe3e7566a9e4b4b66c8380dd..e58a161be3ac39aa33afd539c18ca7f888f9f28f 100644 (file)
@@ -81,11 +81,6 @@ public:
 private:
     void init(const Position&, EAffinity);
     Position canonicalPosition(const Position&);
-
-    static int maxOffset(const Node*);
-    
-    static Position previousVisiblePosition(const Position&);
-    static Position nextVisiblePosition(const Position&);
         
     Position m_deepPosition;
     EAffinity m_affinity;