2008-06-17 Beth Dakin <bdakin@apple.com>
authorbdakin@apple.com <bdakin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Jun 2008 04:44:07 +0000 (04:44 +0000)
committerbdakin@apple.com <bdakin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Jun 2008 04:44:07 +0000 (04:44 +0000)
        Reviewed by Tim.

        This patch removes all instances of the phrase "TextMarker" from
        function names in the C++ accessibility code. TextMarkers only
        exists in the Objective-C code.

        * page/AccessibilityObject.cpp:
        (WebCore::AccessibilityObject::visiblePositionRangeForLine):
        (WebCore::AccessibilityObject::visiblePositionRangeForUnorderedPositions):
        (WebCore::AccessibilityObject::positionOfLeftWord):
        (WebCore::AccessibilityObject::positionOfRightWord):
        (WebCore::AccessibilityObject::leftLineVisiblePositionRange):
        (WebCore::AccessibilityObject::rightLineVisiblePositionRange):
        (WebCore::AccessibilityObject::sentenceForPosition):
        (WebCore::AccessibilityObject::paragraphForPosition):
        (WebCore::AccessibilityObject::styleRangeForPosition):
        (WebCore::AccessibilityObject::visiblePositionRangeForRange):
        (WebCore::AccessibilityObject::stringForVisiblePositionRange):
        (WebCore::AccessibilityObject::boundsForVisiblePositionRange):
        (WebCore::AccessibilityObject::lengthForVisiblePositionRange):
        (WebCore::AccessibilityObject::setSelectedVisiblePositionRange):
        (WebCore::AccessibilityObject::visiblePositionForPoint):
        (WebCore::AccessibilityObject::nextVisiblePosition):
        (WebCore::AccessibilityObject::previousVisiblePosition):
        (WebCore::AccessibilityObject::nextWordEnd):
        (WebCore::AccessibilityObject::previousWordStart):
        (WebCore::AccessibilityObject::nextLineEndPosition):
        (WebCore::AccessibilityObject::previousLineStartPosition):
        (WebCore::AccessibilityObject::nextSentenceEndPosition):
        (WebCore::AccessibilityObject::previousSentenceStartPosition):
        (WebCore::AccessibilityObject::nextParagraphEndPosition):
        (WebCore::AccessibilityObject::previousParagraphStartPosition):
        (WebCore::AccessibilityObject::visiblePositionForIndex):
        (WebCore::AccessibilityObject::accessibilityObjectForPosition):
        (WebCore::AccessibilityObject::lineForPosition):
        (WebCore::AccessibilityObject::plainTextRangeForVisiblePositionRange):
        (WebCore::AccessibilityObject::index):
        (WebCore::AccessibilityObject::doAXRangeForPosition):
        (WebCore::AccessibilityObject::doAXStyleRangeForIndex):
        (WebCore::AccessibilityObject::doAXLineForIndex):
        * page/AccessibilityObject.h:
        * page/AccessibilityRenderObject.cpp:
        (WebCore::AccessibilityRenderObject::visiblePositionRangeForLine):
        (WebCore::AccessibilityRenderObject::boundsForVisiblePositionRange):
        (WebCore::AccessibilityRenderObject::setSelectedVisiblePositionRange):
        (WebCore::AccessibilityRenderObject::visiblePositionForPoint):
        (WebCore::AccessibilityRenderObject::visiblePositionForIndex):
        (WebCore::AccessibilityRenderObject::index):
        (WebCore::AccessibilityRenderObject::doAXBoundsForRange):
        * page/AccessibilityRenderObject.h:
        * page/mac/AccessibilityObjectWrapper.mm:
        (-[AccessibilityObjectWrapper accessibilityAttributeValue:]):
        (-[AccessibilityObjectWrapper accessibilitySetValue:forAttribute:]):
        (-[AccessibilityObjectWrapper doAXAttributedStringForRange:]):
        (-[AccessibilityObjectWrapper accessibilityAttributeValue:forParameter:]):

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

WebCore/ChangeLog
WebCore/page/AccessibilityObject.cpp
WebCore/page/AccessibilityObject.h
WebCore/page/AccessibilityRenderObject.cpp
WebCore/page/AccessibilityRenderObject.h
WebCore/page/mac/AccessibilityObjectWrapper.mm

index 36b839b4d0751d09ab53413686c5382489084527..204195f184821db68bed5e9ed984bbb9b4ec23d6 100644 (file)
@@ -1,3 +1,61 @@
+2008-06-17  Beth Dakin  <bdakin@apple.com>
+
+        Reviewed by Tim.
+
+        This patch removes all instances of the phrase "TextMarker" from 
+        function names in the C++ accessibility code. TextMarkers only 
+        exists in the Objective-C code.
+
+        * page/AccessibilityObject.cpp:
+        (WebCore::AccessibilityObject::visiblePositionRangeForLine):
+        (WebCore::AccessibilityObject::visiblePositionRangeForUnorderedPositions):
+        (WebCore::AccessibilityObject::positionOfLeftWord):
+        (WebCore::AccessibilityObject::positionOfRightWord):
+        (WebCore::AccessibilityObject::leftLineVisiblePositionRange):
+        (WebCore::AccessibilityObject::rightLineVisiblePositionRange):
+        (WebCore::AccessibilityObject::sentenceForPosition):
+        (WebCore::AccessibilityObject::paragraphForPosition):
+        (WebCore::AccessibilityObject::styleRangeForPosition):
+        (WebCore::AccessibilityObject::visiblePositionRangeForRange):
+        (WebCore::AccessibilityObject::stringForVisiblePositionRange):
+        (WebCore::AccessibilityObject::boundsForVisiblePositionRange):
+        (WebCore::AccessibilityObject::lengthForVisiblePositionRange):
+        (WebCore::AccessibilityObject::setSelectedVisiblePositionRange):
+        (WebCore::AccessibilityObject::visiblePositionForPoint):
+        (WebCore::AccessibilityObject::nextVisiblePosition):
+        (WebCore::AccessibilityObject::previousVisiblePosition):
+        (WebCore::AccessibilityObject::nextWordEnd):
+        (WebCore::AccessibilityObject::previousWordStart):
+        (WebCore::AccessibilityObject::nextLineEndPosition):
+        (WebCore::AccessibilityObject::previousLineStartPosition):
+        (WebCore::AccessibilityObject::nextSentenceEndPosition):
+        (WebCore::AccessibilityObject::previousSentenceStartPosition):
+        (WebCore::AccessibilityObject::nextParagraphEndPosition):
+        (WebCore::AccessibilityObject::previousParagraphStartPosition):
+        (WebCore::AccessibilityObject::visiblePositionForIndex):
+        (WebCore::AccessibilityObject::accessibilityObjectForPosition):
+        (WebCore::AccessibilityObject::lineForPosition):
+        (WebCore::AccessibilityObject::plainTextRangeForVisiblePositionRange):
+        (WebCore::AccessibilityObject::index):
+        (WebCore::AccessibilityObject::doAXRangeForPosition):
+        (WebCore::AccessibilityObject::doAXStyleRangeForIndex):
+        (WebCore::AccessibilityObject::doAXLineForIndex):
+        * page/AccessibilityObject.h:
+        * page/AccessibilityRenderObject.cpp:
+        (WebCore::AccessibilityRenderObject::visiblePositionRangeForLine):
+        (WebCore::AccessibilityRenderObject::boundsForVisiblePositionRange):
+        (WebCore::AccessibilityRenderObject::setSelectedVisiblePositionRange):
+        (WebCore::AccessibilityRenderObject::visiblePositionForPoint):
+        (WebCore::AccessibilityRenderObject::visiblePositionForIndex):
+        (WebCore::AccessibilityRenderObject::index):
+        (WebCore::AccessibilityRenderObject::doAXBoundsForRange):
+        * page/AccessibilityRenderObject.h:
+        * page/mac/AccessibilityObjectWrapper.mm:
+        (-[AccessibilityObjectWrapper accessibilityAttributeValue:]):
+        (-[AccessibilityObjectWrapper accessibilitySetValue:forAttribute:]):
+        (-[AccessibilityObjectWrapper doAXAttributedStringForRange:]):
+        (-[AccessibilityObjectWrapper accessibilityAttributeValue:forParameter:]):
+
 2008-06-17  Mark Rowe  <mrowe@apple.com>
 
         Reviewed by Darin Adler.
index 978921f6e5ccf2544343f7be88c4d01f6b1e9f11..5cc7a067e5f247b69d4eb0bc06dd5235b3990dd8 100644 (file)
@@ -327,13 +327,13 @@ Element* AccessibilityObject::actionElement() const
 }
 
 // This function is like a cross-platform version of - (WebCoreTextMarkerRange*)textMarkerRange. It returns
-// a Range that we can convert to a WebCoreTextMarkerRange in the Obj-C file
+// a Range that we can convert to a WebCoreRange in the Obj-C file
 VisiblePositionRange AccessibilityObject::visiblePositionRange() const
 {
     return VisiblePositionRange();
 }
 
-VisiblePositionRange AccessibilityObject::doAXTextMarkerRangeForLine(unsigned lineCount) const
+VisiblePositionRange AccessibilityObject::visiblePositionRangeForLine(unsigned lineCount) const
 {
     return VisiblePositionRange();
 }
@@ -348,7 +348,7 @@ int AccessibilityObject::indexForVisiblePosition(const VisiblePosition& pos) con
     return 0;
 }
     
-VisiblePositionRange AccessibilityObject::doAXTextMarkerRangeForUnorderedTextMarkers(const VisiblePosition& visiblePos1, const VisiblePosition& visiblePos2) const
+VisiblePositionRange AccessibilityObject::visiblePositionRangeForUnorderedPositions(const VisiblePosition& visiblePos1, const VisiblePosition& visiblePos2) const
 {
     if (visiblePos1.isNull() || visiblePos2.isNull())
         return VisiblePositionRange();
@@ -376,14 +376,14 @@ VisiblePositionRange AccessibilityObject::doAXTextMarkerRangeForUnorderedTextMar
     return VisiblePositionRange(startPos, endPos);
 }
 
-VisiblePositionRange AccessibilityObject::doAXLeftWordTextMarkerRangeForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePositionRange AccessibilityObject::positionOfLeftWord(const VisiblePosition& visiblePos) const
 {
     VisiblePosition startPosition = startOfWord(visiblePos, LeftWordIfOnBoundary);
     VisiblePosition endPosition = endOfWord(startPosition);
     return VisiblePositionRange(startPosition, endPosition);
 }
 
-VisiblePositionRange AccessibilityObject::doAXRightWordTextMarkerRangeForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePositionRange AccessibilityObject::positionOfRightWord(const VisiblePosition& visiblePos) const
 {
     VisiblePosition startPosition = startOfWord(visiblePos, RightWordIfOnBoundary);
     VisiblePosition endPosition = endOfWord(startPosition);
@@ -419,7 +419,7 @@ static VisiblePosition updateAXLineStartForVisiblePosition(const VisiblePosition
     return startPosition;
 }
 
-VisiblePositionRange AccessibilityObject::doAXLeftLineTextMarkerRangeForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePositionRange AccessibilityObject::leftLineVisiblePositionRange(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return VisiblePositionRange();
@@ -432,7 +432,7 @@ VisiblePositionRange AccessibilityObject::doAXLeftLineTextMarkerRangeForTextMark
 
     VisiblePosition startPosition = startOfLine(prevVisiblePos);
 
-    // keep searching for a valid line start position.  Unless the textmarker is at the very beginning, there should
+    // keep searching for a valid line start position.  Unless the VisiblePosition is at the very beginning, there should
     // always be a valid line range.  However, startOfLine will return null for position next to a floating object,
     // since floating object doesn't really belong to any line.
     // This check will reposition the marker before the floating object, to ensure we get a line start.
@@ -448,7 +448,7 @@ VisiblePositionRange AccessibilityObject::doAXLeftLineTextMarkerRangeForTextMark
     return VisiblePositionRange(startPosition, endPosition);
 }
 
-VisiblePositionRange AccessibilityObject::doAXRightLineTextMarkerRangeForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePositionRange AccessibilityObject::rightLineVisiblePositionRange(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return VisiblePositionRange();
@@ -470,7 +470,7 @@ VisiblePositionRange AccessibilityObject::doAXRightLineTextMarkerRangeForTextMar
     VisiblePosition endPosition = endOfLine(nextVisiblePos);
 
     // as long as the position hasn't reached the end of the doc,  keep searching for a valid line end position
-    // Unless the textmarker is at the very end, there should always be a valid line range.  However, endOfLine will
+    // Unless the VisiblePosition is at the very end, there should always be a valid line range.  However, endOfLine will
     // return null for position by a floating object, since floating object doesn't really belong to any line.
     // This check will reposition the marker after the floating object, to ensure we get a line end.
     while (endPosition.isNull() && nextVisiblePos.isNotNull()) {
@@ -481,7 +481,7 @@ VisiblePositionRange AccessibilityObject::doAXRightLineTextMarkerRangeForTextMar
     return VisiblePositionRange(startPosition, endPosition);
 }
 
-VisiblePositionRange AccessibilityObject::doAXSentenceTextMarkerRangeForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePositionRange AccessibilityObject::sentenceForPosition(const VisiblePosition& visiblePos) const
 {
     // FIXME: FO 2 IMPLEMENT (currently returns incorrect answer)
     // Related? <rdar://problem/3927736> Text selection broken in 8A336
@@ -490,7 +490,7 @@ VisiblePositionRange AccessibilityObject::doAXSentenceTextMarkerRangeForTextMark
     return VisiblePositionRange(startPosition, endPosition);
 }
 
-VisiblePositionRange AccessibilityObject::doAXParagraphTextMarkerRangeForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePositionRange AccessibilityObject::paragraphForPosition(const VisiblePosition& visiblePos) const
 {
     VisiblePosition startPosition = startOfParagraph(visiblePos);
     VisiblePosition endPosition = endOfParagraph(startPosition);
@@ -543,7 +543,7 @@ static VisiblePosition endOfStyleRange(const VisiblePosition visiblePos)
     return VisiblePosition(endRenderer->node(), maxDeepOffset(endRenderer->node()), VP_DEFAULT_AFFINITY);
 }
 
-VisiblePositionRange AccessibilityObject::doAXStyleTextMarkerRangeForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePositionRange AccessibilityObject::styleRangeForPosition(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return VisiblePositionRange();
@@ -552,7 +552,7 @@ VisiblePositionRange AccessibilityObject::doAXStyleTextMarkerRangeForTextMarker(
 }
 
 // NOTE: Consider providing this utility method as AX API
-VisiblePositionRange AccessibilityObject::textMarkerRangeForRange(const PlainTextRange& range) const
+VisiblePositionRange AccessibilityObject::visiblePositionRangeForRange(const PlainTextRange& range) const
 {
     if (range.start + range.length > text().length())
         return VisiblePositionRange();
@@ -579,7 +579,7 @@ static String stringForReplacedNode(Node* replacedNode)
     return String(&objectReplacementCharacter, 1);
 }
 
-String AccessibilityObject::doAXStringForTextMarkerRange(const VisiblePositionRange& visiblePositionRange) const
+String AccessibilityObject::stringForVisiblePositionRange(const VisiblePositionRange& visiblePositionRange) const
 {
     if (visiblePositionRange.isNull())
         return String();
@@ -608,41 +608,41 @@ String AccessibilityObject::doAXStringForTextMarkerRange(const VisiblePositionRa
     return resultString.isEmpty() ? String() : resultString;
 }
 
-IntRect AccessibilityObject::doAXBoundsForTextMarkerRange(const VisiblePositionRange& visiblePositionRange) const
+IntRect AccessibilityObject::boundsForVisiblePositionRange(const VisiblePositionRange& visiblePositionRange) const
 {
     return IntRect();
 }
 
-int AccessibilityObject::doAXLengthForTextMarkerRange(const VisiblePositionRange& visiblePositionRange) const
+int AccessibilityObject::lengthForVisiblePositionRange(const VisiblePositionRange& visiblePositionRange) const
 {
     // FIXME: Multi-byte support
-    String string = doAXStringForTextMarkerRange(visiblePositionRange);
+    String string = stringForVisiblePositionRange(visiblePositionRange);
     if (string.isEmpty())
         return -1;
 
     return string.length();
 }
 
-void AccessibilityObject::doSetAXSelectedTextMarkerRange(const VisiblePositionRange& textMarkerRange) const
+void AccessibilityObject::setSelectedVisiblePositionRange(const VisiblePositionRange&) const
 {
 }
 
-VisiblePosition AccessibilityObject::doAXTextMarkerForPosition(const IntPoint& point) const
+VisiblePosition AccessibilityObject::visiblePositionForPoint(const IntPoint& point) const
 {
     return VisiblePosition();
 }
 
-VisiblePosition AccessibilityObject::doAXNextTextMarkerForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePosition AccessibilityObject::nextVisiblePosition(const VisiblePosition& visiblePos) const
 {
     return visiblePos.next();
 }
 
-VisiblePosition AccessibilityObject::doAXPreviousTextMarkerForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePosition AccessibilityObject::previousVisiblePosition(const VisiblePosition& visiblePos) const
 {
     return visiblePos.previous();
 }
 
-VisiblePosition AccessibilityObject::doAXNextWordEndTextMarkerForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePosition AccessibilityObject::nextWordEnd(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return VisiblePosition();
@@ -655,7 +655,7 @@ VisiblePosition AccessibilityObject::doAXNextWordEndTextMarkerForTextMarker(cons
     return endOfWord(nextVisiblePos, LeftWordIfOnBoundary);
 }
 
-VisiblePosition AccessibilityObject::doAXPreviousWordStartTextMarkerForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePosition AccessibilityObject::previousWordStart(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return VisiblePosition();
@@ -668,7 +668,7 @@ VisiblePosition AccessibilityObject::doAXPreviousWordStartTextMarkerForTextMarke
     return startOfWord(prevVisiblePos, RightWordIfOnBoundary);
 }
 
-VisiblePosition AccessibilityObject::doAXNextLineEndTextMarkerForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePosition AccessibilityObject::nextLineEndPosition(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return VisiblePosition();
@@ -690,7 +690,7 @@ VisiblePosition AccessibilityObject::doAXNextLineEndTextMarkerForTextMarker(cons
     return endPosition;
 }
 
-VisiblePosition AccessibilityObject::doAXPreviousLineStartTextMarkerForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePosition AccessibilityObject::previousLineStartPosition(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return VisiblePosition();
@@ -715,7 +715,7 @@ VisiblePosition AccessibilityObject::doAXPreviousLineStartTextMarkerForTextMarke
     return startPosition;
 }
 
-VisiblePosition AccessibilityObject::doAXNextSentenceEndTextMarkerForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePosition AccessibilityObject::nextSentenceEndPosition(const VisiblePosition& visiblePos) const
 {
     // FIXME: FO 2 IMPLEMENT (currently returns incorrect answer)
     // Related? <rdar://problem/3927736> Text selection broken in 8A336
@@ -739,7 +739,7 @@ VisiblePosition AccessibilityObject::doAXNextSentenceEndTextMarkerForTextMarker(
     return endPosition;
 }
 
-VisiblePosition AccessibilityObject::doAXPreviousSentenceStartTextMarkerForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePosition AccessibilityObject::previousSentenceStartPosition(const VisiblePosition& visiblePos) const
 {
     // FIXME: FO 2 IMPLEMENT (currently returns incorrect answer)
     // Related? <rdar://problem/3927736> Text selection broken in 8A336
@@ -762,7 +762,7 @@ VisiblePosition AccessibilityObject::doAXPreviousSentenceStartTextMarkerForTextM
     return startPosition;
 }
 
-VisiblePosition AccessibilityObject::doAXNextParagraphEndTextMarkerForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePosition AccessibilityObject::nextParagraphEndPosition(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return VisiblePosition();
@@ -775,7 +775,7 @@ VisiblePosition AccessibilityObject::doAXNextParagraphEndTextMarkerForTextMarker
     return endOfParagraph(nextPos);
 }
 
-VisiblePosition AccessibilityObject::doAXPreviousParagraphStartTextMarkerForTextMarker(const VisiblePosition& visiblePos) const
+VisiblePosition AccessibilityObject::previousParagraphStartPosition(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return VisiblePosition();
@@ -789,12 +789,12 @@ VisiblePosition AccessibilityObject::doAXPreviousParagraphStartTextMarkerForText
 }
 
 // NOTE: Consider providing this utility method as AX API
-VisiblePosition AccessibilityObject::textMarkerForIndex(unsigned indexValue, bool lastIndexOK) const
+VisiblePosition AccessibilityObject::visiblePositionForIndex(unsigned indexValue, bool lastIndexOK) const
 {
     return VisiblePosition();
 }
 
-AccessibilityObject* AccessibilityObject::doAXUIElementForTextMarker(const VisiblePosition& visiblePos) const
+AccessibilityObject* AccessibilityObject::accessibilityObjectForPosition(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return 0;
@@ -806,7 +806,7 @@ AccessibilityObject* AccessibilityObject::doAXUIElementForTextMarker(const Visib
     return obj->document()->axObjectCache()->get(obj);
 }
 
-unsigned AccessibilityObject::doAXLineForTextMarker(const VisiblePosition& visiblePos) const
+unsigned AccessibilityObject::lineForPosition(const VisiblePosition& visiblePos) const
 {
     if (visiblePos.isNull())
         return 0;
@@ -829,10 +829,10 @@ unsigned AccessibilityObject::doAXLineForTextMarker(const VisiblePosition& visib
 }
 
 // NOTE: Consider providing this utility method as AX API
-PlainTextRange AccessibilityObject::rangeForTextMarkerRange(const VisiblePositionRange& positionRange) const
+PlainTextRange AccessibilityObject::plainTextRangeForVisiblePositionRange(const VisiblePositionRange& positionRange) const
 {
-    int index1 = indexForTextMarker(positionRange.start);
-    int index2 = indexForTextMarker(positionRange.end);
+    int index1 = index(positionRange.start);
+    int index2 = index(positionRange.end);
     if (index1 < 0 || index2 < 0 || index1 > index2)
         return PlainTextRange();
 
@@ -840,7 +840,7 @@ PlainTextRange AccessibilityObject::rangeForTextMarkerRange(const VisiblePositio
 }
 
 // NOTE: Consider providing this utility method as AX API
-int AccessibilityObject::indexForTextMarker(const VisiblePosition& position) const
+int AccessibilityObject::index(const VisiblePosition& position) const
 {
     return -1;
 }
@@ -860,11 +860,11 @@ PlainTextRange AccessibilityObject::doAXRangeForLine(unsigned lineNumber) const
 // an error in that case. We return textControl->text().length(), 1. Does this matter?
 PlainTextRange AccessibilityObject::doAXRangeForPosition(const IntPoint& point) const
 {
-    int index = indexForTextMarker(doAXTextMarkerForPosition(point));
-    if (index < 0)
+    int i = index(visiblePositionForPoint(point));
+    if (i < 0)
         return PlainTextRange();
 
-    return PlainTextRange(index, 1);
+    return PlainTextRange(i, 1);
 }
 
 // The composed character range in the text associated with this accessibility object that
@@ -879,8 +879,8 @@ PlainTextRange AccessibilityObject::doAXRangeForIndex(unsigned index) const
 // over which the style in effect at that character index applies.
 PlainTextRange AccessibilityObject::doAXStyleRangeForIndex(unsigned index) const
 {
-    VisiblePositionRange textMarkerRange = doAXStyleTextMarkerRangeForTextMarker(textMarkerForIndex(index, false));
-    return rangeForTextMarkerRange(textMarkerRange);
+    VisiblePositionRange range = styleRangeForPosition(visiblePositionForIndex(index, false));
+    return plainTextRangeForVisiblePositionRange(range);
 }
 
 // A substring of the text associated with this accessibility object that is
@@ -902,7 +902,7 @@ IntRect AccessibilityObject::doAXBoundsForRange(const PlainTextRange& range) con
 // object that contains the character.
 unsigned AccessibilityObject::doAXLineForIndex(unsigned index)
 {
-    return doAXLineForTextMarker(textMarkerForIndex(index, false));
+    return lineForPosition(visiblePositionForIndex(index, false));
 }
 
 FrameView* AccessibilityObject::documentFrameView() const 
index c31e036880cc0c4cb12e1d255e2492fea93b9754..ffc0b47a790c6b9f052acd60a7f12a01e30e2084 100644 (file)
@@ -53,8 +53,6 @@ typedef struct _NSRange NSRange;
 @class NSString;
 @class NSValue;
 @class NSView;
-@class WebCoreTextMarker;
-@class WebCoreTextMarkerRange;
 #else
 class NSArray;
 class NSAttributedString;
@@ -63,8 +61,6 @@ class NSMutableAttributedString;
 class NSString;
 class NSValue;
 class NSView;
-class WebCoreTextMarker;
-class WebCoreTextMarkerRange;
 #if PLATFORM(GTK)
 typedef struct _AtkObject AtkObject;
 typedef struct _AtkObject AccessibilityObjectWrapper;
@@ -321,43 +317,43 @@ public:
     virtual void handleActiveDescendantChanged() { }
 
     virtual VisiblePositionRange visiblePositionRange() const;
-    virtual VisiblePositionRange doAXTextMarkerRangeForLine(unsigned) const;
+    virtual VisiblePositionRange visiblePositionRangeForLine(unsigned) const;
     
-    VisiblePositionRange doAXTextMarkerRangeForUnorderedTextMarkers(const VisiblePosition&, const VisiblePosition&) const;
-    VisiblePositionRange doAXLeftWordTextMarkerRangeForTextMarker(const VisiblePosition&) const;
-    VisiblePositionRange doAXRightWordTextMarkerRangeForTextMarker(const VisiblePosition&) const;
-    VisiblePositionRange doAXLeftLineTextMarkerRangeForTextMarker(const VisiblePosition&) const;
-    VisiblePositionRange doAXRightLineTextMarkerRangeForTextMarker(const VisiblePosition&) const;
-    VisiblePositionRange doAXSentenceTextMarkerRangeForTextMarker(const VisiblePosition&) const;
-    VisiblePositionRange doAXParagraphTextMarkerRangeForTextMarker(const VisiblePosition&) const;
-    VisiblePositionRange doAXStyleTextMarkerRangeForTextMarker(const VisiblePosition&) const;
-    VisiblePositionRange textMarkerRangeForRange(const PlainTextRange&) const;
-
-    String doAXStringForTextMarkerRange(const VisiblePositionRange&) const;
-    virtual IntRect doAXBoundsForTextMarkerRange(const VisiblePositionRange&) const;    
-    int doAXLengthForTextMarkerRange(const VisiblePositionRange&) const;
-    virtual void doSetAXSelectedTextMarkerRange(const VisiblePositionRange&) const;
-    PlainTextRange rangeForTextMarkerRange(const VisiblePositionRange&) const;
-
-    virtual VisiblePosition doAXTextMarkerForPosition(const IntPoint&) const;
-    VisiblePosition doAXNextTextMarkerForTextMarker(const VisiblePosition&) const;
-    VisiblePosition doAXPreviousTextMarkerForTextMarker(const VisiblePosition&) const;
-    VisiblePosition doAXNextWordEndTextMarkerForTextMarker(const VisiblePosition&) const;
-    VisiblePosition doAXPreviousWordStartTextMarkerForTextMarker(const VisiblePosition&) const;
-    VisiblePosition doAXNextLineEndTextMarkerForTextMarker(const VisiblePosition&) const;
-    VisiblePosition doAXPreviousLineStartTextMarkerForTextMarker(const VisiblePosition&) const;
-    VisiblePosition doAXNextSentenceEndTextMarkerForTextMarker(const VisiblePosition&) const;
-    VisiblePosition doAXPreviousSentenceStartTextMarkerForTextMarker(const VisiblePosition&) const;
-    VisiblePosition doAXNextParagraphEndTextMarkerForTextMarker(const VisiblePosition&) const;
-    VisiblePosition doAXPreviousParagraphStartTextMarkerForTextMarker(const VisiblePosition&) const;
-    virtual VisiblePosition textMarkerForIndex(unsigned indexValue, bool lastIndexOK) const;
+    VisiblePositionRange visiblePositionRangeForUnorderedPositions(const VisiblePosition&, const VisiblePosition&) const;
+    VisiblePositionRange positionOfLeftWord(const VisiblePosition&) const;
+    VisiblePositionRange positionOfRightWord(const VisiblePosition&) const;
+    VisiblePositionRange leftLineVisiblePositionRange(const VisiblePosition&) const;
+    VisiblePositionRange rightLineVisiblePositionRange(const VisiblePosition&) const;
+    VisiblePositionRange sentenceForPosition(const VisiblePosition&) const;
+    VisiblePositionRange paragraphForPosition(const VisiblePosition&) const;
+    VisiblePositionRange styleRangeForPosition(const VisiblePosition&) const;
+    VisiblePositionRange visiblePositionRangeForRange(const PlainTextRange&) const;
+
+    String stringForVisiblePositionRange(const VisiblePositionRange&) const;
+    virtual IntRect boundsForVisiblePositionRange(const VisiblePositionRange&) const;    
+    int lengthForVisiblePositionRange(const VisiblePositionRange&) const;
+    virtual void setSelectedVisiblePositionRange(const VisiblePositionRange&) const;
+
+    virtual VisiblePosition visiblePositionForPoint(const IntPoint&) const;
+    VisiblePosition nextVisiblePosition(const VisiblePosition&) const;
+    VisiblePosition previousVisiblePosition(const VisiblePosition&) const;
+    VisiblePosition nextWordEnd(const VisiblePosition&) const;
+    VisiblePosition previousWordStart(const VisiblePosition&) const;
+    VisiblePosition nextLineEndPosition(const VisiblePosition&) const;
+    VisiblePosition previousLineStartPosition(const VisiblePosition&) const;
+    VisiblePosition nextSentenceEndPosition(const VisiblePosition&) const;
+    VisiblePosition previousSentenceStartPosition(const VisiblePosition&) const;
+    VisiblePosition nextParagraphEndPosition(const VisiblePosition&) const;
+    VisiblePosition previousParagraphStartPosition(const VisiblePosition&) const;
+    virtual VisiblePosition visiblePositionForIndex(unsigned indexValue, bool lastIndexOK) const;
     
     virtual VisiblePosition visiblePositionForIndex(int) const;
     virtual int indexForVisiblePosition(const VisiblePosition&) const;
 
-    AccessibilityObject* doAXUIElementForTextMarker(const VisiblePosition&) const;
-    unsigned doAXLineForTextMarker(const VisiblePosition&) const;
-    virtual int indexForTextMarker(const VisiblePosition&) const;
+    AccessibilityObject* accessibilityObjectForPosition(const VisiblePosition&) const;
+    unsigned lineForPosition(const VisiblePosition&) const;
+    PlainTextRange plainTextRangeForVisiblePositionRange(const VisiblePositionRange&) const;
+    virtual int index(const VisiblePosition&) const;
 
     virtual PlainTextRange doAXRangeForLine(unsigned) const;
     PlainTextRange doAXRangeForPosition(const IntPoint&) const;
index 345a5bd611388bac211146f21684e5c3e92e9260..6bd3aa8db825f195c1f25f066b8d118876d0b6af 100644 (file)
@@ -1437,7 +1437,7 @@ VisiblePositionRange AccessibilityRenderObject::visiblePositionRange() const
     return VisiblePositionRange(startPos, endPos);
 }
 
-VisiblePositionRange AccessibilityRenderObject::doAXTextMarkerRangeForLine(unsigned lineCount) const
+VisiblePositionRange AccessibilityRenderObject::visiblePositionRangeForLine(unsigned lineCount) const
 {
     if (lineCount == 0 || !m_renderer)
         return VisiblePositionRange();
@@ -1511,7 +1511,7 @@ int AccessibilityRenderObject::indexForVisiblePosition(const VisiblePosition& po
     return TextIterator::rangeLength(range.get());
 }
 
-IntRect AccessibilityRenderObject::doAXBoundsForTextMarkerRange(const VisiblePositionRange& visiblePositionRange) const
+IntRect AccessibilityRenderObject::boundsForVisiblePositionRange(const VisiblePositionRange& visiblePositionRange) const
 {
     if (visiblePositionRange.isNull())
         return IntRect();
@@ -1553,22 +1553,22 @@ IntRect AccessibilityRenderObject::doAXBoundsForTextMarkerRange(const VisiblePos
 #endif
 }
     
-void AccessibilityRenderObject::doSetAXSelectedTextMarkerRange(const VisiblePositionRange& textMarkerRange) const
+void AccessibilityRenderObject::setSelectedVisiblePositionRange(const VisiblePositionRange& range) const
 {
-    if (textMarkerRange.start.isNull() || textMarkerRange.end.isNull())
+    if (range.start.isNull() || range.end.isNull())
         return;
     
     // make selection and tell the document to use it. if it's zero length, then move to that position
-    if (textMarkerRange.start == textMarkerRange.end) {
-        m_renderer->document()->frame()->selection()->moveTo(textMarkerRange.start, true);
+    if (range.start == range.end) {
+        m_renderer->document()->frame()->selection()->moveTo(range.start, true);
     }
     else {
-        Selection newSelection = Selection(textMarkerRange.start, textMarkerRange.end);
+        Selection newSelection = Selection(range.start, range.end);
         m_renderer->document()->frame()->selection()->setSelection(newSelection);
     }    
 }
 
-VisiblePosition AccessibilityRenderObject::doAXTextMarkerForPosition(const IntPoint& point) const
+VisiblePosition AccessibilityRenderObject::visiblePositionForPoint(const IntPoint& point) const
 {
     // convert absolute point to view coordinates
     FrameView* frameView = m_renderer->document()->topDocument()->renderer()->view()->frameView();
@@ -1616,7 +1616,7 @@ VisiblePosition AccessibilityRenderObject::doAXTextMarkerForPosition(const IntPo
 }
 
 // NOTE: Consider providing this utility method as AX API
-VisiblePosition AccessibilityRenderObject::textMarkerForIndex(unsigned indexValue, bool lastIndexOK) const
+VisiblePosition AccessibilityRenderObject::visiblePositionForIndex(unsigned indexValue, bool lastIndexOK) const
 {
     if (!isTextControl())
         return VisiblePosition();
@@ -1632,7 +1632,7 @@ VisiblePosition AccessibilityRenderObject::textMarkerForIndex(unsigned indexValu
 }
 
 // NOTE: Consider providing this utility method as AX API
-int AccessibilityRenderObject::indexForTextMarker(const VisiblePosition& position) const
+int AccessibilityRenderObject::index(const VisiblePosition& position) const
 {
     if (!isTextControl())
         return -1;
@@ -1733,7 +1733,7 @@ String AccessibilityRenderObject::doAXStringForRange(const PlainTextRange& range
 IntRect AccessibilityRenderObject::doAXBoundsForRange(const PlainTextRange& range) const
 {
     if (isTextControl())
-        return doAXBoundsForTextMarkerRange(textMarkerRangeForRange(range));
+        return boundsForVisiblePositionRange(visiblePositionRangeForRange(range));
     return IntRect();
 }
 
index 722027fc501ab18c5e4377bf1e885bb734a7b1ff..ed6558373d9b1560ff7346533690177c7bd9437f 100644 (file)
@@ -187,13 +187,13 @@ public:
     virtual void handleActiveDescendantChanged();
 
     virtual VisiblePositionRange visiblePositionRange() const;
-    virtual VisiblePositionRange doAXTextMarkerRangeForLine(unsigned) const;
-    virtual IntRect doAXBoundsForTextMarkerRange(const VisiblePositionRange&) const;
-    virtual void doSetAXSelectedTextMarkerRange(const VisiblePositionRange&) const;
+    virtual VisiblePositionRange visiblePositionRangeForLine(unsigned) const;
+    virtual IntRect boundsForVisiblePositionRange(const VisiblePositionRange&) const;
+    virtual void setSelectedVisiblePositionRange(const VisiblePositionRange&) const;
     
-    virtual VisiblePosition doAXTextMarkerForPosition(const IntPoint&) const;
-    virtual VisiblePosition textMarkerForIndex(unsigned indexValue, bool lastIndexOK) const;    
-    virtual int indexForTextMarker(const VisiblePosition&) const;
+    virtual VisiblePosition visiblePositionForPoint(const IntPoint&) const;
+    virtual VisiblePosition visiblePositionForIndex(unsigned indexValue, bool lastIndexOK) const;    
+    virtual int index(const VisiblePosition&) const;
 
     virtual VisiblePosition visiblePositionForIndex(int) const;
     virtual int indexForVisiblePosition(const VisiblePosition&) const;
index f81fa9d5117e3f17233e8b6cd3378a855ffc25d5..5a42ddda6f1332eb53cc6d83a8fd9977c483a26c 100644 (file)
@@ -1021,7 +1021,7 @@ static NSString* roleValueToNSString(AccessibilityRole value)
         if ([attributeName isEqualToString: NSAccessibilityInsertionPointLineNumberAttribute]) {
             if (m_object->isPasswordField() || m_object->selectionStart() != m_object->selectionEnd())
                 return nil;
-            return [NSNumber numberWithInt:m_object->doAXLineForTextMarker(m_object->textMarkerForIndex(m_object->selectionStart(), true))];
+            return [NSNumber numberWithInt:m_object->lineForPosition(m_object->visiblePositionForIndex(m_object->selectionStart(), true))];
         }
     }
     
@@ -1339,7 +1339,7 @@ static NSString* roleValueToNSString(AccessibilityRole value)
     // handle the command
     if ([attributeName isEqualToString: @"AXSelectedTextMarkerRange"]) {
         ASSERT(textMarkerRange);
-        m_object->doSetAXSelectedTextMarkerRange([self visiblePositionRangeForTextMarkerRange:textMarkerRange]);        
+        m_object->setSelectedVisiblePositionRange([self visiblePositionRangeForTextMarkerRange:textMarkerRange]);        
     } else if ([attributeName isEqualToString: NSAccessibilityFocusedAttribute]) {
         ASSERT(number);
         m_object->setFocused([number intValue] != 0);
@@ -1408,7 +1408,7 @@ static RenderObject* rendererForView(NSView* view)
 - (NSAttributedString*)doAXAttributedStringForRange:(NSRange)range
 {
     PlainTextRange textRange = PlainTextRange(range.location, range.length);
-    VisiblePositionRange visiblePosRange = m_object->textMarkerRangeForRange(textRange);
+    VisiblePositionRange visiblePosRange = m_object->visiblePositionRangeForRange(textRange);
     return [self doAXAttributedStringForTextMarkerRange:textMarkerRangeFromVisiblePositions(visiblePosRange.start, visiblePosRange.end)];
 }
 
@@ -1485,7 +1485,7 @@ static RenderObject* rendererForView(NSView* view)
 
     // dispatch
     if ([attribute isEqualToString: @"AXUIElementForTextMarker"])
-        return m_object->doAXUIElementForTextMarker(visiblePos)->wrapper();
+        return m_object->accessibilityObjectForPosition(visiblePos)->wrapper();
 
     if ([attribute isEqualToString: @"AXTextMarkerRangeForUIElement"]) {
         VisiblePositionRange vpRange = uiElement.get()->visiblePositionRange();
@@ -1493,21 +1493,21 @@ static RenderObject* rendererForView(NSView* view)
     }
 
     if ([attribute isEqualToString: @"AXLineForTextMarker"])
-        return [NSNumber numberWithUnsignedInt:m_object->doAXLineForTextMarker(visiblePos)];
+        return [NSNumber numberWithUnsignedInt:m_object->lineForPosition(visiblePos)];
 
     if ([attribute isEqualToString: @"AXTextMarkerRangeForLine"]) {
-        VisiblePositionRange vpRange = m_object->doAXTextMarkerRangeForLine(intNumber);
+        VisiblePositionRange vpRange = m_object->visiblePositionRangeForLine(intNumber);
         return (id)textMarkerRangeFromVisiblePositions(vpRange.start, vpRange.end);
     }
 
     if ([attribute isEqualToString: @"AXStringForTextMarkerRange"])
-        return m_object->doAXStringForTextMarkerRange(visiblePosRange);
+        return m_object->stringForVisiblePositionRange(visiblePosRange);
 
     if ([attribute isEqualToString: @"AXTextMarkerForPosition"])
-        return pointSet ? textMarkerForVisiblePosition(m_object->doAXTextMarkerForPosition(webCorePoint)) : nil;
+        return pointSet ? textMarkerForVisiblePosition(m_object->visiblePositionForPoint(webCorePoint)) : nil;
 
     if ([attribute isEqualToString: @"AXBoundsForTextMarkerRange"]) {
-        NSRect rect = m_object->doAXBoundsForTextMarkerRange(visiblePosRange);
+        NSRect rect = m_object->boundsForVisiblePositionRange(visiblePosRange);
         return [NSValue valueWithRect:rect];
     }
 
@@ -1526,77 +1526,77 @@ static RenderObject* rendererForView(NSView* view)
 
         VisiblePosition visiblePos1 = visiblePositionForTextMarker(textMarker1);
         VisiblePosition visiblePos2 = visiblePositionForTextMarker(textMarker2);
-        VisiblePositionRange vpRange = m_object->doAXTextMarkerRangeForUnorderedTextMarkers(visiblePos1, visiblePos2);
+        VisiblePositionRange vpRange = m_object->visiblePositionRangeForUnorderedPositions(visiblePos1, visiblePos2);
         return (id)textMarkerRangeFromVisiblePositions(vpRange.start, vpRange.end);
     }
 
     if ([attribute isEqualToString: @"AXNextTextMarkerForTextMarker"])
-        return textMarkerForVisiblePosition(m_object->doAXNextTextMarkerForTextMarker(visiblePos));
+        return textMarkerForVisiblePosition(m_object->nextVisiblePosition(visiblePos));
 
     if ([attribute isEqualToString: @"AXPreviousTextMarkerForTextMarker"])
-        return textMarkerForVisiblePosition(m_object->doAXPreviousTextMarkerForTextMarker(visiblePos));
+        return textMarkerForVisiblePosition(m_object->previousVisiblePosition(visiblePos));
 
     if ([attribute isEqualToString: @"AXLeftWordTextMarkerRangeForTextMarker"]) {
-        VisiblePositionRange vpRange = m_object->doAXLeftWordTextMarkerRangeForTextMarker(visiblePos);
+        VisiblePositionRange vpRange = m_object->positionOfLeftWord(visiblePos);
         return (id)textMarkerRangeFromVisiblePositions(vpRange.start, vpRange.end);
     }
 
     if ([attribute isEqualToString: @"AXRightWordTextMarkerRangeForTextMarker"]) {
-        VisiblePositionRange vpRange = m_object->doAXRightWordTextMarkerRangeForTextMarker(visiblePos);
+        VisiblePositionRange vpRange = m_object->positionOfRightWord(visiblePos);
         return (id)textMarkerRangeFromVisiblePositions(vpRange.start, vpRange.end);
     }
 
     if ([attribute isEqualToString: @"AXLeftLineTextMarkerRangeForTextMarker"]) {
-        VisiblePositionRange vpRange = m_object->doAXLeftLineTextMarkerRangeForTextMarker(visiblePos);
+        VisiblePositionRange vpRange = m_object->leftLineVisiblePositionRange(visiblePos);
         return (id)textMarkerRangeFromVisiblePositions(vpRange.start, vpRange.end);
     }
 
     if ([attribute isEqualToString: @"AXRightLineTextMarkerRangeForTextMarker"]) {
-        VisiblePositionRange vpRange = m_object->doAXRightLineTextMarkerRangeForTextMarker(visiblePos);
+        VisiblePositionRange vpRange = m_object->rightLineVisiblePositionRange(visiblePos);
         return (id)textMarkerRangeFromVisiblePositions(vpRange.start, vpRange.end);
     }
 
     if ([attribute isEqualToString: @"AXSentenceTextMarkerRangeForTextMarker"]) {
-        VisiblePositionRange vpRange = m_object->doAXSentenceTextMarkerRangeForTextMarker(visiblePos);
+        VisiblePositionRange vpRange = m_object->sentenceForPosition(visiblePos);
         return (id)textMarkerRangeFromVisiblePositions(vpRange.start, vpRange.end);
     }
 
     if ([attribute isEqualToString: @"AXParagraphTextMarkerRangeForTextMarker"]) {
-        VisiblePositionRange vpRange = m_object->doAXParagraphTextMarkerRangeForTextMarker(visiblePos);
+        VisiblePositionRange vpRange = m_object->paragraphForPosition(visiblePos);
         return (id)textMarkerRangeFromVisiblePositions(vpRange.start, vpRange.end);
     }
 
     if ([attribute isEqualToString: @"AXNextWordEndTextMarkerForTextMarker"])
-        return textMarkerForVisiblePosition(m_object->doAXNextWordEndTextMarkerForTextMarker(visiblePos));
+        return textMarkerForVisiblePosition(m_object->nextWordEnd(visiblePos));
 
     if ([attribute isEqualToString: @"AXPreviousWordStartTextMarkerForTextMarker"])
-        return textMarkerForVisiblePosition(m_object->doAXPreviousWordStartTextMarkerForTextMarker(visiblePos));
+        return textMarkerForVisiblePosition(m_object->previousWordStart(visiblePos));
 
     if ([attribute isEqualToString: @"AXNextLineEndTextMarkerForTextMarker"])
-        return textMarkerForVisiblePosition(m_object->doAXNextLineEndTextMarkerForTextMarker(visiblePos));
+        return textMarkerForVisiblePosition(m_object->nextLineEndPosition(visiblePos));
 
     if ([attribute isEqualToString: @"AXPreviousLineStartTextMarkerForTextMarker"])
-        return textMarkerForVisiblePosition(m_object->doAXPreviousLineStartTextMarkerForTextMarker(visiblePos));
+        return textMarkerForVisiblePosition(m_object->previousLineStartPosition(visiblePos));
 
     if ([attribute isEqualToString: @"AXNextSentenceEndTextMarkerForTextMarker"])
-        return textMarkerForVisiblePosition(m_object->doAXNextSentenceEndTextMarkerForTextMarker(visiblePos));
+        return textMarkerForVisiblePosition(m_object->nextSentenceEndPosition(visiblePos));
 
     if ([attribute isEqualToString: @"AXPreviousSentenceStartTextMarkerForTextMarker"])
-        return textMarkerForVisiblePosition(m_object->doAXPreviousSentenceStartTextMarkerForTextMarker(visiblePos));
+        return textMarkerForVisiblePosition(m_object->previousSentenceStartPosition(visiblePos));
 
     if ([attribute isEqualToString: @"AXNextParagraphEndTextMarkerForTextMarker"])
-        return textMarkerForVisiblePosition(m_object->doAXNextParagraphEndTextMarkerForTextMarker(visiblePos));
+        return textMarkerForVisiblePosition(m_object->nextParagraphEndPosition(visiblePos));
 
     if ([attribute isEqualToString: @"AXPreviousParagraphStartTextMarkerForTextMarker"])
-        return textMarkerForVisiblePosition(m_object->doAXPreviousParagraphStartTextMarkerForTextMarker(visiblePos));
+        return textMarkerForVisiblePosition(m_object->previousParagraphStartPosition(visiblePos));
 
     if ([attribute isEqualToString: @"AXStyleTextMarkerRangeForTextMarker"]) {
-        VisiblePositionRange vpRange = m_object->doAXStyleTextMarkerRangeForTextMarker(visiblePos);
+        VisiblePositionRange vpRange = m_object->styleRangeForPosition(visiblePos);
         return (id)textMarkerRangeFromVisiblePositions(vpRange.start, vpRange.end);
     }
 
     if ([attribute isEqualToString: @"AXLengthForTextMarkerRange"]) {
-        int length = m_object->doAXLengthForTextMarkerRange(visiblePosRange);
+        int length = m_object->lengthForVisiblePositionRange(visiblePosRange);
         if (length < 0)
             return nil;
         return [NSNumber numberWithInt:length];