Changeset 15966 in webkit

Timestamp:

Aug 22, 2006, 10:24:54 AM (18 years ago)

Author:

harrison

Message:

Reviewed by John.

<rdar://problem/4522205> Represent controls in AXAttributedStringForTextMarkerRange

Treat controls as replaced elements, so that AXAttributedStringForTextMarkerRange will, too.

• editing/TextIterator.cpp: (WebCore::TextIterator::advance): Include controls.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• editing/TextIterator.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2006-08-22  David Harrison  <harrison@apple.com>
+
+        Reviewed by John.
+
+        <rdar://problem/4522205> Represent controls in AXAttributedStringForTextMarkerRange
+
+        Treat controls as replaced elements, so that AXAttributedStringForTextMarkerRange will, too.
+        
+        * editing/TextIterator.cpp:
+        (WebCore::TextIterator::advance):
+        Include controls.
+
 2006-08-22  David Harrison  <harrison@apple.com>
 

trunk/WebCore/editing/TextIterator.cpp

@@ -158 +158 @@
                 if (renderer->style()->visibility() == VISIBLE)
                     m_handledNode = handleTextNode();
-            } else if (renderer && (renderer->isImage() || renderer->isWidget())) {
+            } else if (renderer && (renderer->isImage() || renderer->isWidget() || (renderer->element() && renderer->element()->isControl()))) {
                 if (renderer->style()->visibility() == VISIBLE)
                     m_handledNode = handleReplacedElement();