Changeset 93578 in webkit

Timestamp:

Aug 22, 2011 8:55:42 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

Unreviewed, rolling out r93565.
​http://trac.webkit.org/changeset/93565
​https://bugs.webkit.org/show_bug.cgi?id=66745

Breaks layout tests (Requested by koz2 on #webkit).

Patch by Sheriff Bot <​webkit.review.bot@gmail.com> on 2011-08-22

Source/WebCore:

• accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::ariaLabeledByElements):
(WebCore::AccessibilityRenderObject::ariaLabeledByAttribute):
(WebCore::AccessibilityRenderObject::title):
(WebCore::AccessibilityRenderObject::hasTextAlternative):

• accessibility/AccessibilityRenderObject.h:

LayoutTests:

• accessibility/aria-labelledby-overrides-aria-labeledby-expected.txt: Removed.
• accessibility/aria-labelledby-overrides-aria-labeledby.html: Removed.

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/accessibility/aria-labelledby-overrides-aria-labeledby-expected.txt (deleted)
• LayoutTests/accessibility/aria-labelledby-overrides-aria-labeledby.html (deleted)
• Source/WebCore/ChangeLog (modified) (1 diff)
• Source/WebCore/accessibility/AccessibilityRenderObject.cpp (modified) (3 diffs)
• Source/WebCore/accessibility/AccessibilityRenderObject.h (modified) (2 diffs)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2011-08-22  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r93565.
+        http://trac.webkit.org/changeset/93565
+        https://bugs.webkit.org/show_bug.cgi?id=66745
+
+        Breaks layout tests (Requested by koz2 on #webkit).
+
+        * accessibility/aria-labelledby-overrides-aria-labeledby-expected.txt: Removed.
+        * accessibility/aria-labelledby-overrides-aria-labeledby.html: Removed.
+
 2011-08-22  Peter Kasting  <pkasting@google.com>
 

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-08-22  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r93565.
+        http://trac.webkit.org/changeset/93565
+        https://bugs.webkit.org/show_bug.cgi?id=66745
+
+        Breaks layout tests (Requested by koz2 on #webkit).
+
+        * accessibility/AccessibilityRenderObject.cpp:
+        (WebCore::AccessibilityRenderObject::ariaLabeledByElements):
+        (WebCore::AccessibilityRenderObject::ariaLabeledByAttribute):
+        (WebCore::AccessibilityRenderObject::title):
+        (WebCore::AccessibilityRenderObject::hasTextAlternative):
+        * accessibility/AccessibilityRenderObject.h:
+
 2011-08-22  Alice Boxhall  <aboxhall@chromium.org>
 

trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp

@@ -1242 +1242 @@
 }
     
-void AccessibilityRenderObject::ariaLabelledByElements(Vector<Element*>& elements) const
-{
-    elementsFromAttribute(elements, aria_labelledbyAttr);
+void AccessibilityRenderObject::ariaLabeledByElements(Vector<Element*>& elements) const
+{
+    elementsFromAttribute(elements, aria_labeledbyAttr);
     if (!elements.size())
-        elementsFromAttribute(elements, aria_labeledbyAttr);
+        elementsFromAttribute(elements, aria_labelledbyAttr);
 }
    
-String AccessibilityRenderObject::ariaLabelledByAttribute() const
+String AccessibilityRenderObject::ariaLabeledByAttribute() const
 {
     Vector<Element*> elements;
-    ariaLabelledByElements(elements);
+    ariaLabeledByElements(elements);
     
     return accessibilityDescriptionForElements(elements);
@@ -1301 +1301 @@
         return String();
     
-
     const AtomicString& title = getAttribute(titleAttr);
     if (!title.isEmpty())
@@ -1592 +1591 @@
 bool AccessibilityRenderObject::hasTextAlternative() const
 {
-    // ARIA: section 2A, bullet #3 says if aria-labelledby or aria-label appears, it should
+    // ARIA: section 2A, bullet #3 says if aria-labeledby or aria-label appears, it should
     // override the "label" element association.
-    if (!ariaLabelledByAttribute().isEmpty() || !getAttribute(aria_labelAttr).isEmpty())
+    if (!ariaLabeledByAttribute().isEmpty() || !getAttribute(aria_labelAttr).isEmpty())
         return true;
         

trunk/Source/WebCore/accessibility/AccessibilityRenderObject.h

@@ -186 +186 @@
     virtual VisibleSelection selection() const;
     virtual String stringValue() const;
-    virtual String ariaLabelledByAttribute() const;
+    virtual String ariaLabeledByAttribute() const;
     virtual String title() const;
     virtual String ariaDescribedByAttribute() const;
@@ -267 +267 @@
     
     void setRenderObject(RenderObject* renderer) { m_renderer = renderer; }
-    void ariaLabelledByElements(Vector<Element*>& elements) const;
+    void ariaLabeledByElements(Vector<Element*>& elements) const;
     bool needsToUpdateChildren() const { return m_childrenDirty; }