Switch Accessibility code to new layout types
authorleviw@chromium.org <leviw@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Aug 2011 01:18:55 +0000 (01:18 +0000)
committerleviw@chromium.org <leviw@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Aug 2011 01:18:55 +0000 (01:18 +0000)
https://bugs.webkit.org/show_bug.cgi?id=66348

Reviewed by Eric Seidel.

Converting remaining Accessibility code to the LayoutUnit abstraction.

No new tests, no change in functionality

* accessibility/AccessibilityImageMapLink.cpp:
(WebCore::AccessibilityImageMapLink::elementRect):
* accessibility/AccessibilityImageMapLink.h:
* accessibility/AccessibilityListBox.cpp:
(WebCore::AccessibilityListBox::elementAccessibilityHitTest):
* accessibility/AccessibilityListBox.h:
* accessibility/AccessibilityListBoxOption.cpp:
(WebCore::AccessibilityListBoxOption::elementRect):
* accessibility/AccessibilityListBoxOption.h:
* accessibility/AccessibilityMenuListOption.cpp:
(WebCore::AccessibilityMenuListOption::elementRect):
* accessibility/AccessibilityMenuListOption.h:
(WebCore::AccessibilityMenuListOption::size):
* accessibility/AccessibilityMenuListPopup.h:
(WebCore::AccessibilityMenuListPopup::elementRect):
(WebCore::AccessibilityMenuListPopup::size):
* accessibility/AccessibilityObject.cpp:
(WebCore::AccessibilityObject::clickPoint):
(WebCore::AccessibilityObject::orientation):
(WebCore::AccessibilityObject::elementAccessibilityHitTest):
* accessibility/AccessibilityObject.h:
(WebCore::AccessibilityObject::accessibilityHitTest):
(WebCore::AccessibilityObject::boundingBoxRect):
(WebCore::AccessibilityObject::size):
(WebCore::AccessibilityObject::boundsForVisiblePositionRange):
(WebCore::AccessibilityObject::doAXBoundsForRange):
* accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::isOffScreen):
(WebCore::AccessibilityRenderObject::boundingBoxRect):
(WebCore::AccessibilityRenderObject::checkboxOrRadioRect):
(WebCore::AccessibilityRenderObject::elementRect):
(WebCore::AccessibilityRenderObject::size):
(WebCore::AccessibilityRenderObject::clickPoint):
(WebCore::AccessibilityRenderObject::accessibilityIsIgnored):
(WebCore::AccessibilityRenderObject::boundsForVisiblePositionRange):
(WebCore::AccessibilityRenderObject::visiblePositionForPoint):
(WebCore::AccessibilityRenderObject::doAXBoundsForRange):
* accessibility/AccessibilityRenderObject.h:
* accessibility/AccessibilityScrollView.cpp:
(WebCore::AccessibilityScrollView::accessibilityHitTest):
(WebCore::AccessibilityScrollView::elementRect):
* accessibility/AccessibilityScrollView.h:
* accessibility/AccessibilityScrollbar.cpp:
(WebCore::AccessibilityScrollbar::elementRect):
* accessibility/AccessibilityScrollbar.h:
* accessibility/AccessibilitySlider.cpp:
(WebCore::AccessibilitySliderThumb::elementRect):
(WebCore::AccessibilitySliderThumb::size):
* accessibility/AccessibilitySlider.h:
* accessibility/AccessibilityTableColumn.cpp:
(WebCore::AccessibilityTableColumn::elementRect):
(WebCore::AccessibilityTableColumn::size):
* accessibility/AccessibilityTableColumn.h:
* accessibility/AccessibilityTableHeaderContainer.cpp:
(WebCore::AccessibilityTableHeaderContainer::elementRect):
(WebCore::AccessibilityTableHeaderContainer::size):
* accessibility/AccessibilityTableHeaderContainer.h:
* accessibility/mac/AccessibilityObjectWrapper.mm:
(-[AccessibilityObjectWrapper position]):
(-[AccessibilityObjectWrapper accessibilityAttributeValue:]):
(-[AccessibilityObjectWrapper accessibilityShowContextMenu]):

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

25 files changed:
Source/WebCore/ChangeLog
Source/WebCore/accessibility/AccessibilityImageMapLink.cpp
Source/WebCore/accessibility/AccessibilityImageMapLink.h
Source/WebCore/accessibility/AccessibilityListBox.cpp
Source/WebCore/accessibility/AccessibilityListBox.h
Source/WebCore/accessibility/AccessibilityListBoxOption.cpp
Source/WebCore/accessibility/AccessibilityListBoxOption.h
Source/WebCore/accessibility/AccessibilityMenuListOption.cpp
Source/WebCore/accessibility/AccessibilityMenuListOption.h
Source/WebCore/accessibility/AccessibilityMenuListPopup.h
Source/WebCore/accessibility/AccessibilityObject.cpp
Source/WebCore/accessibility/AccessibilityObject.h
Source/WebCore/accessibility/AccessibilityRenderObject.cpp
Source/WebCore/accessibility/AccessibilityRenderObject.h
Source/WebCore/accessibility/AccessibilityScrollView.cpp
Source/WebCore/accessibility/AccessibilityScrollView.h
Source/WebCore/accessibility/AccessibilityScrollbar.cpp
Source/WebCore/accessibility/AccessibilityScrollbar.h
Source/WebCore/accessibility/AccessibilitySlider.cpp
Source/WebCore/accessibility/AccessibilitySlider.h
Source/WebCore/accessibility/AccessibilityTableColumn.cpp
Source/WebCore/accessibility/AccessibilityTableColumn.h
Source/WebCore/accessibility/AccessibilityTableHeaderContainer.cpp
Source/WebCore/accessibility/AccessibilityTableHeaderContainer.h
Source/WebCore/accessibility/mac/AccessibilityObjectWrapper.mm

index ffb2925..7238ca4 100644 (file)
@@ -1,3 +1,76 @@
+2011-08-17  Levi Weintraub  <leviw@chromium.org>
+
+        Switch Accessibility code to new layout types
+        https://bugs.webkit.org/show_bug.cgi?id=66348
+
+        Reviewed by Eric Seidel.
+
+        Converting remaining Accessibility code to the LayoutUnit abstraction.
+
+        No new tests, no change in functionality
+
+        * accessibility/AccessibilityImageMapLink.cpp:
+        (WebCore::AccessibilityImageMapLink::elementRect):
+        * accessibility/AccessibilityImageMapLink.h:
+        * accessibility/AccessibilityListBox.cpp:
+        (WebCore::AccessibilityListBox::elementAccessibilityHitTest):
+        * accessibility/AccessibilityListBox.h:
+        * accessibility/AccessibilityListBoxOption.cpp:
+        (WebCore::AccessibilityListBoxOption::elementRect):
+        * accessibility/AccessibilityListBoxOption.h:
+        * accessibility/AccessibilityMenuListOption.cpp:
+        (WebCore::AccessibilityMenuListOption::elementRect):
+        * accessibility/AccessibilityMenuListOption.h:
+        (WebCore::AccessibilityMenuListOption::size):
+        * accessibility/AccessibilityMenuListPopup.h:
+        (WebCore::AccessibilityMenuListPopup::elementRect):
+        (WebCore::AccessibilityMenuListPopup::size):
+        * accessibility/AccessibilityObject.cpp:
+        (WebCore::AccessibilityObject::clickPoint):
+        (WebCore::AccessibilityObject::orientation):
+        (WebCore::AccessibilityObject::elementAccessibilityHitTest):
+        * accessibility/AccessibilityObject.h:
+        (WebCore::AccessibilityObject::accessibilityHitTest):
+        (WebCore::AccessibilityObject::boundingBoxRect):
+        (WebCore::AccessibilityObject::size):
+        (WebCore::AccessibilityObject::boundsForVisiblePositionRange):
+        (WebCore::AccessibilityObject::doAXBoundsForRange):
+        * accessibility/AccessibilityRenderObject.cpp:
+        (WebCore::AccessibilityRenderObject::isOffScreen):
+        (WebCore::AccessibilityRenderObject::boundingBoxRect):
+        (WebCore::AccessibilityRenderObject::checkboxOrRadioRect):
+        (WebCore::AccessibilityRenderObject::elementRect):
+        (WebCore::AccessibilityRenderObject::size):
+        (WebCore::AccessibilityRenderObject::clickPoint):
+        (WebCore::AccessibilityRenderObject::accessibilityIsIgnored):
+        (WebCore::AccessibilityRenderObject::boundsForVisiblePositionRange):
+        (WebCore::AccessibilityRenderObject::visiblePositionForPoint):
+        (WebCore::AccessibilityRenderObject::doAXBoundsForRange):
+        * accessibility/AccessibilityRenderObject.h:
+        * accessibility/AccessibilityScrollView.cpp:
+        (WebCore::AccessibilityScrollView::accessibilityHitTest):
+        (WebCore::AccessibilityScrollView::elementRect):
+        * accessibility/AccessibilityScrollView.h:
+        * accessibility/AccessibilityScrollbar.cpp:
+        (WebCore::AccessibilityScrollbar::elementRect):
+        * accessibility/AccessibilityScrollbar.h:
+        * accessibility/AccessibilitySlider.cpp:
+        (WebCore::AccessibilitySliderThumb::elementRect):
+        (WebCore::AccessibilitySliderThumb::size):
+        * accessibility/AccessibilitySlider.h:
+        * accessibility/AccessibilityTableColumn.cpp:
+        (WebCore::AccessibilityTableColumn::elementRect):
+        (WebCore::AccessibilityTableColumn::size):
+        * accessibility/AccessibilityTableColumn.h:
+        * accessibility/AccessibilityTableHeaderContainer.cpp:
+        (WebCore::AccessibilityTableHeaderContainer::elementRect):
+        (WebCore::AccessibilityTableHeaderContainer::size):
+        * accessibility/AccessibilityTableHeaderContainer.h:
+        * accessibility/mac/AccessibilityObjectWrapper.mm:
+        (-[AccessibilityObjectWrapper position]):
+        (-[AccessibilityObjectWrapper accessibilityAttributeValue:]):
+        (-[AccessibilityObjectWrapper accessibilityShowContextMenu]):
+
 2011-08-17  Emil A Eklund  <eae@chromium.org>
 
         Switch focus handling to to new layout types
index a41f228..5b6d3a7 100644 (file)
@@ -119,11 +119,11 @@ String AccessibilityImageMapLink::title() const
 
     return String();
 }
-    
-IntRect AccessibilityImageMapLink::elementRect() const
+
+LayoutRect AccessibilityImageMapLink::elementRect() const
 {
     if (!m_mapElement.get() || !m_areaElement.get())
-        return IntRect();
+        return LayoutRect();
 
     RenderObject* renderer;
     if (m_parent && m_parent->isAccessibilityRenderObject())
@@ -132,7 +132,7 @@ IntRect AccessibilityImageMapLink::elementRect() const
         renderer = m_mapElement->renderer();
     
     if (!renderer)
-        return IntRect();
+        return LayoutRect();
     
     return m_areaElement->computeRect(renderer);
 }
index 4b02b2b..6d14f0d 100644 (file)
@@ -68,7 +68,7 @@ public:
     virtual String stringValueForMSAA() const;
     virtual String nameForMSAA() const;
 
-    virtual IntRect elementRect() const;
+    virtual LayoutRect elementRect() const;
 
 private:    
     RefPtr<HTMLAreaElement> m_areaElement;
index f300dfd..91a2e94 100644 (file)
@@ -163,7 +163,7 @@ bool AccessibilityListBox::accessibilityIsIgnored() const
     return false;
 }
 
-AccessibilityObject* AccessibilityListBox::elementAccessibilityHitTest(const IntPoint& point) const
+AccessibilityObject* AccessibilityListBox::elementAccessibilityHitTest(const LayoutPoint& point) const
 {
     // the internal HTMLSelectElement methods for returning a listbox option at a point
     // ignore optgroup elements.
@@ -174,12 +174,12 @@ AccessibilityObject* AccessibilityListBox::elementAccessibilityHitTest(const Int
     if (!node)
         return 0;
     
-    IntRect parentRect = boundingBoxRect();
+    LayoutRect parentRect = boundingBoxRect();
     
     AccessibilityObject* listBoxOption = 0;
     unsigned length = m_children.size();
     for (unsigned i = 0; i < length; i++) {
-        IntRect rect = toRenderListBox(m_renderer)->itemBoundingBoxRect(parentRect.location(), i);
+        LayoutRect rect = toRenderListBox(m_renderer)->itemBoundingBoxRect(parentRect.location(), i);
         // The cast to HTMLElement below is safe because the only other possible listItem type
         // would be a WMLElement, but WML builds don't use accessibility features at all.
         if (rect.contains(point)) {
index cfccb52..62cda6c 100644 (file)
@@ -56,7 +56,7 @@ public:
 private:    
     AccessibilityObject* listBoxOptionAccessibilityObject(HTMLElement*) const;
     virtual bool accessibilityIsIgnored() const;
-    virtual AccessibilityObject* elementAccessibilityHitTest(const IntPoint&) const;
+    virtual AccessibilityObject* elementAccessibilityHitTest(const LayoutPoint&) const;
 };
     
 } // namespace WebCore
index a24d06f..a20782a 100644 (file)
@@ -83,9 +83,9 @@ bool AccessibilityListBoxOption::isSelected() const
     return static_cast<HTMLOptionElement*>(m_optionElement)->selected();
 }
 
-IntRect AccessibilityListBoxOption::elementRect() const
+LayoutRect AccessibilityListBoxOption::elementRect() const
 {
-    IntRect rect;
+    LayoutRect rect;
     if (!m_optionElement)
         return rect;
     
@@ -97,7 +97,7 @@ IntRect AccessibilityListBoxOption::elementRect() const
     if (!listBoxRenderer)
         return rect;
     
-    IntRect parentRect = listBoxRenderer->document()->axObjectCache()->getOrCreate(listBoxRenderer)->boundingBoxRect();
+    LayoutRect parentRect = listBoxRenderer->document()->axObjectCache()->getOrCreate(listBoxRenderer)->boundingBoxRect();
     int index = listBoxOptionIndex();
     if (index != -1)
         rect = toRenderListBox(listBoxRenderer)->itemBoundingBoxRect(parentRect.location(), index);
index 597c731..c5184d7 100644 (file)
@@ -60,7 +60,7 @@ public:
     virtual void setSelected(bool);
     virtual bool canSetSelectedAttribute() const;
 
-    virtual IntRect elementRect() const;
+    virtual LayoutRect elementRect() const;
     virtual AccessibilityObject* parentObject() const;
     bool isListBoxOption() const { return true; }
     
index 5bca580..c706512 100644 (file)
@@ -99,7 +99,7 @@ bool AccessibilityMenuListOption::canSetSelectedAttribute() const
     return isEnabled();
 }
 
-IntRect AccessibilityMenuListOption::elementRect() const
+LayoutRect AccessibilityMenuListOption::elementRect() const
 {
     AccessibilityObject* parent = parentObject();
     ASSERT(parent->isMenuListPopup());
index 9393d56..f225927 100644 (file)
@@ -47,7 +47,7 @@ private:
 
     virtual AccessibilityRole roleValue() const { return MenuListOptionRole; }
     virtual bool canHaveChildren() const { return false; }
-    virtual IntSize size() const { return elementRect().size(); }
+    virtual LayoutSize size() const { return elementRect().size(); }
 
     virtual Element* actionElement() const;
     virtual AccessibilityObject* parentObject() const;
@@ -58,7 +58,7 @@ private:
     virtual String nameForMSAA() const;
     virtual void setSelected(bool);
     virtual bool canSetSelectedAttribute() const;
-    virtual IntRect elementRect() const;
+    virtual LayoutRect elementRect() const;
     virtual String stringValue() const;
 
     RefPtr<HTMLElement> m_element;
index 380ccd0..6cd1bea 100644 (file)
@@ -51,8 +51,8 @@ private:
 
     virtual bool isMenuListPopup() const { return true; }
 
-    virtual IntRect elementRect() const { return IntRect(); }
-    virtual IntSize size() const { return IntSize(); }
+    virtual LayoutRect elementRect() const { return LayoutRect(); }
+    virtual LayoutSize size() const { return LayoutSize(); }
     virtual AccessibilityRole roleValue() const { return MenuListPopupRole; }
 
     virtual bool isVisible() const;
index 6b71a3f..346be05 100644 (file)
@@ -393,10 +393,10 @@ bool AccessibilityObject::isARIAControl(AccessibilityRole ariaRole)
     || ariaRole == ComboBoxRole || ariaRole == SliderRole; 
 }
 
-IntPoint AccessibilityObject::clickPoint() const
+LayoutPoint AccessibilityObject::clickPoint() const
 {
-    IntRect rect = elementRect();
-    return IntPoint(rect.x() + rect.width() / 2, rect.y() + rect.height() / 2);
+    LayoutRect rect = elementRect();
+    return LayoutPoint(rect.x() + rect.width() / 2, rect.y() + rect.height() / 2);
 }
 
 bool AccessibilityObject::press() const
@@ -1144,7 +1144,7 @@ const AtomicString& AccessibilityObject::getAttribute(const QualifiedName& attri
 // Lacking concrete evidence of orientation, horizontal means width > height. vertical is height > width;
 AccessibilityOrientation AccessibilityObject::orientation() const
 {
-    IntRect bounds = elementRect();
+    LayoutRect bounds = elementRect();
     if (bounds.size().width() > bounds.size().height())
         return AccessibilityOrientationHorizontal;
     if (bounds.size().height() > bounds.size().width())
@@ -1287,14 +1287,14 @@ bool AccessibilityObject::supportsARIALiveRegion() const
     return equalIgnoringCase(liveRegion, "polite") || equalIgnoringCase(liveRegion, "assertive");
 }
 
-AccessibilityObject* AccessibilityObject::elementAccessibilityHitTest(const IntPoint& point) const
+AccessibilityObject* AccessibilityObject::elementAccessibilityHitTest(const LayoutPoint& point) const
 { 
     // Send the hit test back into the sub-frame if necessary.
     if (isAttachment()) {
         Widget* widget = widgetForAttachmentView();
         // Normalize the point for the widget's bounds.
         if (widget && widget->isFrameView())
-            return axObjectCache()->getOrCreate(widget)->accessibilityHitTest(IntPoint(point - widget->frameRect().location()));
+            return axObjectCache()->getOrCreate(widget)->accessibilityHitTest(toPoint(point - widget->frameRect().location()));
     }
 
     return const_cast<AccessibilityObject*>(this); 
index 3392b75..f125bce 100644 (file)
@@ -30,7 +30,7 @@
 #ifndef AccessibilityObject_h
 #define AccessibilityObject_h
 
-#include "IntRect.h"
+#include "LayoutTypes.h"
 #include "VisiblePosition.h"
 #include "VisibleSelection.h"
 #include <wtf/Forward.h>
@@ -455,9 +455,9 @@ public:
     virtual void determineARIADropEffects(Vector<String>&) { }
     
     // Called on the root AX object to return the deepest available element.
-    virtual AccessibilityObject* accessibilityHitTest(const IntPoint&) const { return 0; }
+    virtual AccessibilityObject* accessibilityHitTest(const LayoutPoint&) const { return 0; }
     // Called on the AX object after the render tree determines which is the right AccessibilityRenderObject.
-    virtual AccessibilityObject* elementAccessibilityHitTest(const IntPoint&) const;
+    virtual AccessibilityObject* elementAccessibilityHitTest(const LayoutPoint&) const;
 
     virtual AccessibilityObject* focusedUIElement() const;
 
@@ -496,10 +496,10 @@ public:
     static AccessibilityObject* anchorElementForNode(Node*);
     virtual Element* anchorElement() const { return 0; }
     virtual Element* actionElement() const { return 0; }
-    virtual IntRect boundingBoxRect() const { return IntRect(); }
-    virtual IntRect elementRect() const = 0;
-    virtual IntSize size() const { return elementRect().size(); }
-    virtual IntPoint clickPoint() const;
+    virtual LayoutRect boundingBoxRect() const { return LayoutRect(); }
+    virtual LayoutRect elementRect() const = 0;
+    virtual LayoutSize size() const { return elementRect().size(); }
+    virtual LayoutPoint clickPoint() const;
 
     virtual PlainTextRange selectedTextRange() const { return PlainTextRange(); }
     unsigned selectionStart() const { return selectedTextRange().start; }
@@ -573,7 +573,7 @@ public:
     VisiblePositionRange visiblePositionRangeForRange(const PlainTextRange&) const;
 
     String stringForVisiblePositionRange(const VisiblePositionRange&) const;
-    virtual IntRect boundsForVisiblePositionRange(const VisiblePositionRange&) const { return IntRect(); }
+    virtual LayoutRect boundsForVisiblePositionRange(const VisiblePositionRange&) const { return LayoutRect(); }
     int lengthForVisiblePositionRange(const VisiblePositionRange&) const;
     virtual void setSelectedVisiblePositionRange(const VisiblePositionRange&) const { }
 
@@ -604,7 +604,7 @@ public:
     PlainTextRange doAXStyleRangeForIndex(unsigned) const;
 
     virtual String doAXStringForRange(const PlainTextRange&) const { return String(); }
-    virtual IntRect doAXBoundsForRange(const PlainTextRange&) const { return IntRect(); }
+    virtual LayoutRect doAXBoundsForRange(const PlainTextRange&) const { return LayoutRect(); }
     String listMarkerTextForNodeAndPosition(Node*, const VisiblePosition&) const;
 
     unsigned doAXLineForIndex(unsigned);
index 167231d..0d8a7b9 100644 (file)
@@ -685,7 +685,7 @@ bool AccessibilityRenderObject::isReadOnly() const
 bool AccessibilityRenderObject::isOffScreen() const
 {
     ASSERT(m_renderer);
-    IntRect contentRect = m_renderer->absoluteClippedOverflowRect();
+    LayoutRect contentRect = m_renderer->absoluteClippedOverflowRect();
     FrameView* view = m_renderer->frame()->view();
     FloatRect viewRect = view->visibleContentRect();
     viewRect.intersect(contentRect);
@@ -1415,12 +1415,12 @@ String AccessibilityRenderObject::accessibilityDescription() const
     return String();
 }
 
-IntRect AccessibilityRenderObject::boundingBoxRect() const
+LayoutRect AccessibilityRenderObject::boundingBoxRect() const
 {
     RenderObject* obj = m_renderer;
     
     if (!obj)
-        return IntRect();
+        return LayoutRect();
     
     if (obj->node()) // If we are a continuation, we want to make sure to use the primary renderer.
         obj = obj->node()->renderer();
@@ -1436,11 +1436,11 @@ IntRect AccessibilityRenderObject::boundingBoxRect() const
         obj->absoluteFocusRingQuads(quads);
     const size_t n = quads.size();
     if (!n)
-        return IntRect();
+        return LayoutRect();
 
-    IntRect result;
+    LayoutRect result;
     for (size_t i = 0; i < n; ++i) {
-        IntRect r = quads[i].enclosingBoundingBox();
+        LayoutRect r = quads[i].enclosingBoundingBox();
         if (!r.isEmpty()) {
             if (obj->style()->hasAppearance())
                 obj->theme()->adjustRepaintRect(obj, r);
@@ -1455,21 +1455,21 @@ IntRect AccessibilityRenderObject::boundingBoxRect() const
     return result;
 }
     
-IntRect AccessibilityRenderObject::checkboxOrRadioRect() const
+LayoutRect AccessibilityRenderObject::checkboxOrRadioRect() const
 {
     if (!m_renderer)
-        return IntRect();
+        return LayoutRect();
     
     HTMLLabelElement* label = labelForElement(static_cast<Element*>(m_renderer->node()));
     if (!label || !label->renderer())
         return boundingBoxRect();
     
-    IntRect labelRect = axObjectCache()->getOrCreate(label->renderer())->elementRect();
+    LayoutRect labelRect = axObjectCache()->getOrCreate(label->renderer())->elementRect();
     labelRect.unite(boundingBoxRect());
     return labelRect;
 }
 
-IntRect AccessibilityRenderObject::elementRect() const
+LayoutRect AccessibilityRenderObject::elementRect() const
 {
     // a checkbox or radio button should encompass its label
     if (isCheckboxOrRadio())
@@ -1478,13 +1478,13 @@ IntRect AccessibilityRenderObject::elementRect() const
     return boundingBoxRect();
 }
 
-IntSize AccessibilityRenderObject::size() const
+LayoutSize AccessibilityRenderObject::size() const
 {
-    IntRect rect = elementRect();
+    LayoutRect rect = elementRect();
     return rect.size();
 }
 
-IntPoint AccessibilityRenderObject::clickPoint() const
+LayoutPoint AccessibilityRenderObject::clickPoint() const
 {
     // use the default position unless this is an editable web area, in which case we use the selection bounds.
     if (!isWebArea() || isReadOnly())
@@ -1492,11 +1492,11 @@ IntPoint AccessibilityRenderObject::clickPoint() const
     
     VisibleSelection visSelection = selection();
     VisiblePositionRange range = VisiblePositionRange(visSelection.visibleStart(), visSelection.visibleEnd());
-    IntRect bounds = boundsForVisiblePositionRange(range);
+    LayoutRect bounds = boundsForVisiblePositionRange(range);
 #if PLATFORM(MAC)
     bounds.setLocation(m_renderer->document()->view()->screenToContents(bounds.location()));
 #endif        
-    return IntPoint(bounds.x() + (bounds.width() / 2), bounds.y() - (bounds.height() / 2));
+    return LayoutPoint(bounds.x() + (bounds.width() / 2), bounds.y() - (bounds.height() / 2));
 }
     
 AccessibilityObject* AccessibilityRenderObject::internalLinkElement() const
@@ -1896,7 +1896,7 @@ bool AccessibilityRenderObject::accessibilityIsIgnored() const
             
             // check whether rendered image was stretched from one-dimensional file image
             if (image->cachedImage()) {
-                IntSize imageSize = image->cachedImage()->imageSize(image->view()->zoomFactor());
+                LayoutSize imageSize = image->cachedImage()->imageSize(image->view()->zoomFactor());
                 return imageSize.height() <= 1 || imageSize.width() <= 1;
             }
         }
@@ -2535,15 +2535,15 @@ int AccessibilityRenderObject::indexForVisiblePosition(const VisiblePosition& po
 #endif
 }
 
-IntRect AccessibilityRenderObject::boundsForVisiblePositionRange(const VisiblePositionRange& visiblePositionRange) const
+LayoutRect AccessibilityRenderObject::boundsForVisiblePositionRange(const VisiblePositionRange& visiblePositionRange) const
 {
     if (visiblePositionRange.isNull())
-        return IntRect();
+        return LayoutRect();
     
     // Create a mutable VisiblePositionRange.
     VisiblePositionRange range(visiblePositionRange);
-    IntRect rect1 = range.start.absoluteCaretBounds();
-    IntRect rect2 = range.end.absoluteCaretBounds();
+    LayoutRect rect1 = range.start.absoluteCaretBounds();
+    LayoutRect rect2 = range.end.absoluteCaretBounds();
     
     // readjust for position at the edge of a line.  This is to exclude line rect that doesn't need to be accounted in the range bounds
     if (rect2.y() != rect1.y()) {
@@ -2558,13 +2558,13 @@ IntRect AccessibilityRenderObject::boundsForVisiblePositionRange(const VisiblePo
         }
     }
     
-    IntRect ourrect = rect1;
+    LayoutRect ourrect = rect1;
     ourrect.unite(rect2);
     
     // if the rectangle spans lines and contains multiple text chars, use the range's bounding box intead
     if (rect1.maxY() != rect2.maxY()) {
         RefPtr<Range> dataRange = makeRange(range.start, range.end);
-        IntRect boundingBox = dataRange->boundingBox();
+        LayoutRect boundingBox = dataRange->boundingBox();
         String rangeString = plainText(dataRange.get());
         if (rangeString.length() > 1 && !boundingBox.isEmpty())
             ourrect = boundingBox;
@@ -2591,7 +2591,7 @@ void AccessibilityRenderObject::setSelectedVisiblePositionRange(const VisiblePos
     }    
 }
 
-VisiblePosition AccessibilityRenderObject::visiblePositionForPoint(const IntPoint& point) const
+VisiblePosition AccessibilityRenderObject::visiblePositionForPoint(const LayoutPoint& point) const
 {
     if (!m_renderer)
         return VisiblePosition();
@@ -2612,9 +2612,9 @@ VisiblePosition AccessibilityRenderObject::visiblePositionForPoint(const IntPoin
     Node* innerNode = 0;
     
     // locate the node containing the point
-    IntPoint pointResult;
+    LayoutPoint pointResult;
     while (1) {
-        IntPoint ourpoint;
+        LayoutPoint ourpoint;
 #if PLATFORM(MAC)
         ourpoint = frameView->screenToContents(point);
 #else
@@ -2752,11 +2752,11 @@ String AccessibilityRenderObject::doAXStringForRange(const PlainTextRange& range
 // The bounding rectangle of the text associated with this accessibility object that is
 // specified by the given range. This is the bounding rectangle a sighted user would see
 // on the display screen, in pixels.
-IntRect AccessibilityRenderObject::doAXBoundsForRange(const PlainTextRange& range) const
+LayoutRect AccessibilityRenderObject::doAXBoundsForRange(const PlainTextRange& range) const
 {
     if (allowsTextRanges())
         return boundsForVisiblePositionRange(visiblePositionRangeForRange(range));
-    return IntRect();
+    return LayoutRect();
 }
 
 AccessibilityObject* AccessibilityRenderObject::accessibilityImageMapHitTest(HTMLAreaElement* area, const IntPoint& point) const
index 689aadc..3970c5e 100644 (file)
@@ -31,6 +31,7 @@
 #define AccessibilityRenderObject_h
 
 #include "AccessibilityObject.h"
+#include "LayoutTypes.h"
 #include <wtf/Forward.h>
 
 namespace WebCore {
@@ -163,10 +164,10 @@ public:
     AccessibilityObject* menuForMenuButton() const;
     AccessibilityObject* menuButtonForMenu() const;
     
-    virtual IntRect boundingBoxRect() const;
-    virtual IntRect elementRect() const;
-    virtual IntSize size() const;
-    virtual IntPoint clickPoint() const;
+    virtual LayoutRect boundingBoxRect() const;
+    virtual LayoutRect elementRect() const;
+    virtual LayoutSize size() const;
+    virtual LayoutPoint clickPoint() const;
     
     void setRenderer(RenderObject* renderer) { m_renderer = renderer; }
     virtual RenderObject* renderer() const { return m_renderer; }
@@ -229,7 +230,7 @@ public:
     
     virtual VisiblePositionRange visiblePositionRange() const;
     virtual VisiblePositionRange visiblePositionRangeForLine(unsigned) const;
-    virtual IntRect boundsForVisiblePositionRange(const VisiblePositionRange&) const;
+    virtual LayoutRect boundsForVisiblePositionRange(const VisiblePositionRange&) const;
     virtual void setSelectedVisiblePositionRange(const VisiblePositionRange&) const;
     virtual bool supportsARIAFlowTo() const;
     virtual void ariaFlowToElements(AccessibilityChildrenVector&) const;
@@ -240,7 +241,7 @@ public:
     virtual bool isARIAGrabbed();
     virtual void determineARIADropEffects(Vector<String>&);
     
-    virtual VisiblePosition visiblePositionForPoint(const IntPoint&) const;
+    virtual VisiblePosition visiblePositionForPoint(const LayoutPoint&) const;
     virtual VisiblePosition visiblePositionForIndex(unsigned indexValue, bool lastIndexOK) const;    
     virtual int index(const VisiblePosition&) const;
 
@@ -251,7 +252,7 @@ public:
     virtual PlainTextRange doAXRangeForIndex(unsigned) const;
     
     virtual String doAXStringForRange(const PlainTextRange&) const;
-    virtual IntRect doAXBoundsForRange(const PlainTextRange&) const;
+    virtual LayoutRect doAXBoundsForRange(const PlainTextRange&) const;
     
     virtual String stringValueForMSAA() const;
     virtual String stringRoleForMSAA() const;
@@ -290,7 +291,7 @@ private:
     void alterSliderValue(bool increase);
     void changeValueByStep(bool increase);
     bool isNativeCheckboxOrRadio() const;
-    IntRect checkboxOrRadioRect() const;
+    LayoutRect checkboxOrRadioRect() const;
     void addRadioButtonGroupMembers(AccessibilityChildrenVector& linkedUIElements) const;
     AccessibilityObject* internalLinkElement() const;
     AccessibilityObject* accessibilityImageMapHitTest(HTMLAreaElement*, const IntPoint&) const;
index 3ee049b..cbc7759 100644 (file)
@@ -138,7 +138,7 @@ AccessibilityObject* AccessibilityScrollView::webAreaObject() const
     return axObjectCache()->getOrCreate(doc->renderer());
 }
 
-AccessibilityObject* AccessibilityScrollView::accessibilityHitTest(const IntPoint& point) const
+AccessibilityObject* AccessibilityScrollView::accessibilityHitTest(const LayoutPoint& point) const
 {
     AccessibilityObject* webArea = webAreaObject();
     if (!webArea)
@@ -152,7 +152,7 @@ AccessibilityObject* AccessibilityScrollView::accessibilityHitTest(const IntPoin
     return webArea->accessibilityHitTest(point);
 }
 
-IntRect AccessibilityScrollView::elementRect() const
+LayoutRect AccessibilityScrollView::elementRect() const
 {
     return m_scrollView->frameRect();
 }
index 3d72ab5..9b865b5 100644 (file)
@@ -51,12 +51,12 @@ private:
     
     virtual AccessibilityObject* scrollBar(AccessibilityOrientation) const;
     virtual void addChildren();
-    virtual AccessibilityObject* accessibilityHitTest(const IntPoint&) const;
+    virtual AccessibilityObject* accessibilityHitTest(const LayoutPoint&) const;
     virtual const AccessibilityChildrenVector& children();
     virtual void updateChildrenIfNecessary();
     
     virtual FrameView* documentFrameView() const;
-    virtual IntRect elementRect() const;
+    virtual LayoutRect elementRect() const;
     virtual AccessibilityObject* parentObject() const;
     
     AccessibilityObject* webAreaObject() const;
index b225af8..0ecdab1 100644 (file)
@@ -48,10 +48,10 @@ PassRefPtr<AccessibilityScrollbar> AccessibilityScrollbar::create(Scrollbar* scr
     return adoptRef(new AccessibilityScrollbar(scrollbar));
 }
     
-IntRect AccessibilityScrollbar::elementRect() const
+LayoutRect AccessibilityScrollbar::elementRect() const
 {
     if (!m_scrollbar)
-        return IntRect();
+        return LayoutRect();
     
     return m_scrollbar->frameRect();
 }
index 3268347..3d54604 100644 (file)
@@ -51,7 +51,7 @@ private:
 
     virtual bool isAccessibilityScrollbar() const { return true; }
     virtual AccessibilityObject* parentObject() const { return m_parent; }
-    virtual IntRect elementRect() const;
+    virtual LayoutRect elementRect() const;
     
     virtual AccessibilityRole roleValue() const { return ScrollBarRole; }
     virtual AccessibilityOrientation orientation() const;
index fa0c27f..f80adc7 100644 (file)
@@ -174,15 +174,15 @@ PassRefPtr<AccessibilitySliderThumb> AccessibilitySliderThumb::create()
     return adoptRef(new AccessibilitySliderThumb());
 }
     
-IntRect AccessibilitySliderThumb::elementRect() const
+LayoutRect AccessibilitySliderThumb::elementRect() const
 {
     RenderObject* sliderRenderer = m_parentSlider->renderer();
     if (!sliderRenderer || !sliderRenderer->isSlider())
-        return IntRect();
+        return LayoutRect();
     return sliderThumbElementOf(sliderRenderer->node())->getRect();
 }
 
-IntSize AccessibilitySliderThumb::size() const
+LayoutSize AccessibilitySliderThumb::size() const
 {
     return elementRect().size();
 }
index 80a988b..8d260bb 100644 (file)
@@ -77,8 +77,8 @@ public:
     void setParentObject(AccessibilitySlider* slider) { m_parentSlider = slider; }
     virtual AccessibilityObject* parentObject() const { return m_parentSlider; }
 
-    virtual IntSize size() const;
-    virtual IntRect elementRect() const;
+    virtual LayoutSize size() const;
+    virtual LayoutRect elementRect() const;
 
     virtual bool accessibilityIsIgnored() const;
 
index f776e40..da1694d 100644 (file)
@@ -63,13 +63,13 @@ void AccessibilityTableColumn::setParentTable(AccessibilityTable* table)
     clearChildren();
 }
     
-IntRect AccessibilityTableColumn::elementRect() const
+LayoutRect AccessibilityTableColumn::elementRect() const
 {
     // this will be filled in when addChildren is called
     return m_columnRect;
 }
 
-IntSize AccessibilityTableColumn::size() const
+LayoutSize AccessibilityTableColumn::size() const
 {
     return elementRect().size();
 }
index 15d300c..1706e5a 100644 (file)
@@ -59,13 +59,13 @@ public:
     virtual const AccessibilityChildrenVector& children();
     virtual void addChildren();
     
-    virtual IntSize size() const;
-    virtual IntRect elementRect() const;
+    virtual LayoutSize size() const;
+    virtual LayoutRect elementRect() const;
     
 private:    
     AccessibilityTable* m_parentTable;
     int m_columnIndex;
-    IntRect m_columnRect;
+    LayoutRect m_columnRect;
     
     AccessibilityObject* headerObjectForSection(RenderTableSection*, bool thTagRequired);
 };
index dad6963..c7a8c95 100644 (file)
@@ -56,18 +56,18 @@ const AccessibilityObject::AccessibilityChildrenVector& AccessibilityTableHeader
         addChildren();
     return m_children;
 }
-   
-IntRect AccessibilityTableHeaderContainer::elementRect() const
+
+LayoutRect AccessibilityTableHeaderContainer::elementRect() const
 {
     // this will be filled in when addChildren is called
     return m_headerRect;
 }
-    
-IntSize AccessibilityTableHeaderContainer::size() const
+
+LayoutSize AccessibilityTableHeaderContainer::size() const
 {
     return elementRect().size();
 }
-    
+
 bool AccessibilityTableHeaderContainer::accessibilityIsIgnored() const
 {
     if (!m_parentTable)
index 79521c0..2e73372 100644 (file)
@@ -51,12 +51,12 @@ public:
     virtual const AccessibilityChildrenVector& children();
     virtual void addChildren();
     
-    virtual IntSize size() const;
-    virtual IntRect elementRect() const;
+    virtual LayoutSize size() const;
+    virtual LayoutRect elementRect() const;
     
 private:
     AccessibilityTable* m_parentTable;
-    IntRect m_headerRect;
+    LayoutRect m_headerRect;
     
     virtual bool accessibilityIsIgnored() const;
 }; 
index ba65932..8a45afe 100644 (file)
@@ -1371,7 +1371,7 @@ static NSMutableArray* convertToNSArray(const AccessibilityObject::Accessibility
 
 - (NSValue *)position
 {
-    IntRect rect = m_object->elementRect();
+    LayoutRect rect = m_object->elementRect();
     NSPoint point;
     
     FrameView* frameView = m_object->documentFrameView();
@@ -1388,7 +1388,7 @@ static NSMutableArray* convertToNSArray(const AccessibilityObject::Accessibility
         
         NSPoint scrollPosition = NSMakePoint(0, 0);
         if (frameView && !m_object->isScrollbar() && !m_object->isScrollView()) {
-            IntPoint frameScrollPos = frameView->scrollPosition();
+            LayoutPoint frameScrollPos = frameView->scrollPosition();
             scrollPosition = NSMakePoint(frameScrollPos.x(), frameScrollPos.y());
         }
         
@@ -1957,7 +1957,7 @@ static NSString* roleValueToNSString(AccessibilityRole value)
         return [NSNumber numberWithBool: m_object->isEnabled()];
 
     if ([attributeName isEqualToString: NSAccessibilitySizeAttribute]) {
-        IntSize s = m_object->size();
+        LayoutSize s = m_object->size();
         return [NSValue valueWithSize: NSMakeSize(s.width(), s.height())];
     }
 
@@ -2569,7 +2569,7 @@ static NSString* roleValueToNSString(AccessibilityRole value)
         return;
 
     // Simulate a click in the middle of the object.
-    IntPoint clickPoint = m_object->clickPoint();
+    LayoutPoint clickPoint = m_object->clickPoint();
     
     PlatformMouseEvent mouseEvent(clickPoint, clickPoint, RightButton, MouseEventPressed, 1, false, false, false, false, currentTime());
     bool handled = frame->eventHandler()->sendContextMenuEvent(mouseEvent);