[webkit-changes] [147520] trunk

Tue, 02 Apr 2013 23:15:33 -0700

Title: [147520] trunk
Revision
147520
Author
[email protected]
Date
2013-04-02 23:17:21 -0700 (Tue, 02 Apr 2013)

Log Message

AXRoleDescripton for DL is "list"; should be "description list"
https://bugs.webkit.org/show_bug.cgi?id=111665

Patch by James Craig <[email protected]> on 2013-04-02
Reviewed by Chris Fleizach.

Source/WebCore:

Defining the role and description strings, and accessors for DL.
This previously relied on getting the role and description from the platform API.

Updated existing test coverage.

* English.lproj/Localizable.strings:
* accessibility/AccessibilityObject.h:
* accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::determineAccessibilityRole):
* accessibility/mac/WebAccessibilityObjectWrapperMac.mm:
(createAccessibilityRoleMap):
(-[WebAccessibilityObjectWrapper subrole]):
(-[WebAccessibilityObjectWrapper roleDescription]):
* platform/LocalizedStrings.cpp:
(WebCore::AXDescriptionListText):
(WebCore):
* platform/LocalizedStrings.h:
(WebCore):
* platform/blackberry/LocalizedStringsBlackBerry.cpp:
(WebCore::AXDescriptionListText):
(WebCore):
* platform/chromium/LocalizedStringsChromium.cpp:
(WebCore::AXDescriptionListText):
(WebCore):
* platform/efl/LocalizedStringsEfl.cpp:
(WebCore::AXDescriptionListText):
(WebCore):
* platform/gtk/LocalizedStringsGtk.cpp:
(WebCore::AXDescriptionListText):
(WebCore):
* platform/qt/LocalizedStringsQt.cpp:
(WebCore::AXDescriptionListText):
(WebCore):

Source/WebKit/chromium:

Defining the role and description strings.

* public/WebAccessibilityRole.h:
* src/AssertMatchingEnums.cpp:

Tools:

Defining the role and description strings.

* DumpRenderTree/chromium/TestRunner/src/AccessibilityUIElementChromium.cpp:

LayoutTests:

Updating expectation strings for DL role description.

* platform/mac/accessibility/lists-expected.txt:
* platform/mac/accessibility/role-subrole-roledescription-expected.txt:
* platform/mac/accessibility/role-subrole-roledescription.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (147519 => 147520)


--- trunk/LayoutTests/ChangeLog	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/LayoutTests/ChangeLog	2013-04-03 06:17:21 UTC (rev 147520)
@@ -1,3 +1,16 @@
+2013-04-02  James Craig  <[email protected]>
+
+        AXRoleDescripton for DL is "list"; should be "description list"
+        https://bugs.webkit.org/show_bug.cgi?id=111665
+
+        Reviewed by Chris Fleizach.
+
+        Updating expectation strings for DL role description.
+
+        * platform/mac/accessibility/lists-expected.txt:
+        * platform/mac/accessibility/role-subrole-roledescription-expected.txt:
+        * platform/mac/accessibility/role-subrole-roledescription.html:
+
 2013-04-02  Jer Noble  <[email protected]>
 
         Unreviewed gardening; rebaseline mac expected results after r147470.

Modified: trunk/LayoutTests/platform/mac/accessibility/lists-expected.txt (147519 => 147520)


--- trunk/LayoutTests/platform/mac/accessibility/lists-expected.txt	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/LayoutTests/platform/mac/accessibility/lists-expected.txt	2013-04-03 06:17:21 UTC (rev 147520)
@@ -69,7 +69,7 @@
 ------------
 AXRole: AXList
 AXSubrole: AXDescriptionList
-AXRoleDescription: list
+AXRoleDescription: description list
 AXChildren: <array of size 5>
 AXHelp: 
 AXParent: <AXList>

Modified: trunk/LayoutTests/platform/mac/accessibility/role-subrole-roledescription-expected.txt (147519 => 147520)


--- trunk/LayoutTests/platform/mac/accessibility/role-subrole-roledescription-expected.txt	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/LayoutTests/platform/mac/accessibility/role-subrole-roledescription-expected.txt	2013-04-03 06:17:21 UTC (rev 147520)
@@ -87,7 +87,7 @@
 PASS: dl
       AXRole: AXList
       AXSubrole: AXDescriptionList
-      AXRoleDescription: list
+      AXRoleDescription: description list
       
 PASS: dt
       AXRole: AXGroup

Modified: trunk/LayoutTests/platform/mac/accessibility/role-subrole-roledescription.html (147519 => 147520)


--- trunk/LayoutTests/platform/mac/accessibility/role-subrole-roledescription.html	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/LayoutTests/platform/mac/accessibility/role-subrole-roledescription.html	2013-04-03 06:17:21 UTC (rev 147520)
@@ -27,7 +27,7 @@
 <del data-role="" data-subrole="" data-roledescription="" class="ex">X</del>
 <!-- skipped <details/summary> http://webkit.org/b/108979 -->
 <!-- <dfn> http://webkit.org/b/108980 --><dfn data-role="" data-subrole="" data-roledescription="" class="ex">X</dfn>
-<dl data-role="AXList" data-subrole="AXDescriptionList" data-roledescription="list" class="ex">
+<dl data-role="AXList" data-subrole="AXDescriptionList" data-roledescription="description list" class="ex">
     <dt data-role="AXGroup" data-subrole="AXTerm" data-roledescription="term" class="ex">X</dt>
     <dd data-role="AXGroup" data-subrole="AXDescription" data-roledescription="description" class="ex">X</dd>
 </dl>

Modified: trunk/Source/WebCore/ChangeLog (147519 => 147520)


--- trunk/Source/WebCore/ChangeLog	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/ChangeLog	2013-04-03 06:17:21 UTC (rev 147520)
@@ -1,3 +1,44 @@
+2013-04-02  James Craig  <[email protected]>
+
+        AXRoleDescripton for DL is "list"; should be "description list"
+        https://bugs.webkit.org/show_bug.cgi?id=111665
+
+        Reviewed by Chris Fleizach.
+
+        Defining the role and description strings, and accessors for DL.
+        This previously relied on getting the role and description from the platform API.
+
+        Updated existing test coverage.
+
+        * English.lproj/Localizable.strings:
+        * accessibility/AccessibilityObject.h:
+        * accessibility/AccessibilityRenderObject.cpp:
+        (WebCore::AccessibilityRenderObject::determineAccessibilityRole):
+        * accessibility/mac/WebAccessibilityObjectWrapperMac.mm:
+        (createAccessibilityRoleMap):
+        (-[WebAccessibilityObjectWrapper subrole]):
+        (-[WebAccessibilityObjectWrapper roleDescription]):
+        * platform/LocalizedStrings.cpp:
+        (WebCore::AXDescriptionListText):
+        (WebCore):
+        * platform/LocalizedStrings.h:
+        (WebCore):
+        * platform/blackberry/LocalizedStringsBlackBerry.cpp:
+        (WebCore::AXDescriptionListText):
+        (WebCore):
+        * platform/chromium/LocalizedStringsChromium.cpp:
+        (WebCore::AXDescriptionListText):
+        (WebCore):
+        * platform/efl/LocalizedStringsEfl.cpp:
+        (WebCore::AXDescriptionListText):
+        (WebCore):
+        * platform/gtk/LocalizedStringsGtk.cpp:
+        (WebCore::AXDescriptionListText):
+        (WebCore):
+        * platform/qt/LocalizedStringsQt.cpp:
+        (WebCore::AXDescriptionListText):
+        (WebCore):
+
 2013-04-02  Eugene Klyuchnikov  <[email protected]>
 
         Web Inspector: [Network] Split business logic and presentation for initiator column.

Modified: trunk/Source/WebCore/English.lproj/Localizable.strings (147519 => 147520)


--- trunk/Source/WebCore/English.lproj/Localizable.strings	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/English.lproj/Localizable.strings	2013-04-03 06:17:21 UTC (rev 147520)
@@ -643,6 +643,9 @@
 /* description detail */
 "description" = "description";
 
+/* description list */
+"description list" = "description list";
+
 /* An ARIA accessibility group that acts as an dialog. */
 "dialog" = "dialog";

Modified: trunk/Source/WebCore/accessibility/AccessibilityObject.h (147519 => 147520)


--- trunk/Source/WebCore/accessibility/AccessibilityObject.h	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/accessibility/AccessibilityObject.h	2013-04-03 06:17:21 UTC (rev 147520)
@@ -112,6 +112,7 @@
     ColumnHeaderRole,
     ComboBoxRole,
     DefinitionRole,
+    DescriptionListRole,
     DescriptionListTermRole,
     DescriptionListDetailRole,
     DirectoryRole,

Modified: trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp (147519 => 147520)


--- trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp	2013-04-03 06:17:21 UTC (rev 147520)
@@ -2477,6 +2477,9 @@
     if (node && node->hasTagName(dtTag))
         return DescriptionListTermRole;
 
+    if (node && node->hasTagName(dlTag))
+        return DescriptionListRole;
+
     if (node && (node->hasTagName(rpTag) || node->hasTagName(rtTag)))
         return AnnotationRole;

Modified: trunk/Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperMac.mm (147519 => 147520)


--- trunk/Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperMac.mm	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperMac.mm	2013-04-03 06:17:21 UTC (rev 147520)
@@ -1590,6 +1590,7 @@
         { DefinitionRole, NSAccessibilityGroupRole },
         { DescriptionListDetailRole, NSAccessibilityGroupRole },
         { DescriptionListTermRole, NSAccessibilityGroupRole },
+        { DescriptionListRole, NSAccessibilityListRole },
         { SliderThumbRole, NSAccessibilityValueIndicatorRole },
         { LandmarkApplicationRole, NSAccessibilityGroupRole },
         { LandmarkBannerRole, NSAccessibilityGroupRole },
@@ -1740,6 +1741,8 @@
             return @"AXTabPanel";
         case DefinitionRole:
             return @"AXDefinition";
+        case DescriptionListRole:
+            return @"AXDescriptionList";
         case DescriptionListTermRole:
             return @"AXTerm";
         case DescriptionListDetailRole:
@@ -1876,6 +1879,14 @@
     if (m_object->isFileUploadButton())
         return AXFileUploadButtonText();
     
+    // Only returning for DL (not UL or OL) because description changed with HTML5 from 'definition list' to
+    // superset 'description list' and does not return the same values in AX API on some OS versions. 
+    if (m_object->isList()) {
+        AccessibilityList* listObject = static_cast<AccessibilityList*>(m_object);
+        if (listObject->isDescriptionList())
+            return AXDescriptionListText();
+    }
+    
     // AppKit also returns AXTab for the role description for a tab item.
     if (m_object->isTabItem())
         return NSAccessibilityRoleDescription(@"AXTab", nil);

Modified: trunk/Source/WebCore/platform/LocalizedStrings.cpp (147519 => 147520)


--- trunk/Source/WebCore/platform/LocalizedStrings.cpp	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/platform/LocalizedStrings.cpp	2013-04-03 06:17:21 UTC (rev 147520)
@@ -562,6 +562,11 @@
     return WEB_UI_STRING("definition", "role description of ARIA definition role");
 }
 
+String AXDescriptionListText()
+{
+    return WEB_UI_STRING("description list", "accessibility role description of a description list");
+}
+
 String AXDescriptionListTermText()
 {
     return WEB_UI_STRING("term", "term word of a description list");

Modified: trunk/Source/WebCore/platform/LocalizedStrings.h (147519 => 147520)


--- trunk/Source/WebCore/platform/LocalizedStrings.h	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/platform/LocalizedStrings.h	2013-04-03 06:17:21 UTC (rev 147520)
@@ -153,6 +153,7 @@
     String AXImageMapText();
     String AXHeadingText();
     String AXDefinitionText();
+    String AXDescriptionListText();
     String AXDescriptionListTermText();
     String AXDescriptionListDetailText();
     String AXFooterRoleDescriptionText();

Modified: trunk/Source/WebCore/platform/blackberry/LocalizedStringsBlackBerry.cpp (147519 => 147520)


--- trunk/Source/WebCore/platform/blackberry/LocalizedStringsBlackBerry.cpp	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/platform/blackberry/LocalizedStringsBlackBerry.cpp	2013-04-03 06:17:21 UTC (rev 147520)
@@ -394,6 +394,12 @@
     return String();
 }
 
+String AXDescriptionListText()
+{
+    notImplemented();
+    return String();
+}
+
 String AXDescriptionListDetailText()
 {
     notImplemented();

Modified: trunk/Source/WebCore/platform/chromium/LocalizedStringsChromium.cpp (147519 => 147520)


--- trunk/Source/WebCore/platform/chromium/LocalizedStringsChromium.cpp	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/platform/chromium/LocalizedStringsChromium.cpp	2013-04-03 06:17:21 UTC (rev 147520)
@@ -151,6 +151,12 @@
     return String("definition");
 }
 
+String AXDescriptionListText()
+{
+    notImplemented();
+    return String("description list");
+}
+
 String AXDescriptionListTermText()
 {
     notImplemented();

Modified: trunk/Source/WebCore/platform/efl/LocalizedStringsEfl.cpp (147519 => 147520)


--- trunk/Source/WebCore/platform/efl/LocalizedStringsEfl.cpp	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/platform/efl/LocalizedStringsEfl.cpp	2013-04-03 06:17:21 UTC (rev 147520)
@@ -363,6 +363,11 @@
     return String::fromUTF8("definition");
 }
 
+String AXDescriptionListText()
+{
+    return String::fromUTF8("description list");
+}
+
 String AXDescriptionListTermText()
 {
     return String::fromUTF8("term");

Modified: trunk/Source/WebCore/platform/gtk/LocalizedStringsGtk.cpp (147519 => 147520)


--- trunk/Source/WebCore/platform/gtk/LocalizedStringsGtk.cpp	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/platform/gtk/LocalizedStringsGtk.cpp	2013-04-03 06:17:21 UTC (rev 147520)
@@ -412,6 +412,11 @@
     return String::fromUTF8(_("definition"));
 }
 
+String AXDescriptionListText()
+{
+    return String::fromUTF8(_("description list"));
+}
+
 String AXDescriptionListTermText()
 {
     return String::fromUTF8(_("term"));

Modified: trunk/Source/WebCore/platform/qt/LocalizedStringsQt.cpp (147519 => 147520)


--- trunk/Source/WebCore/platform/qt/LocalizedStringsQt.cpp	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebCore/platform/qt/LocalizedStringsQt.cpp	2013-04-03 06:17:21 UTC (rev 147520)
@@ -385,6 +385,12 @@
     return String();
 }
 
+String AXDescriptionListText()
+{
+    notImplemented();
+    return String();
+}
+
 String AXDescriptionListTermText()
 {
     notImplemented();

Modified: trunk/Source/WebKit/chromium/ChangeLog (147519 => 147520)


--- trunk/Source/WebKit/chromium/ChangeLog	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebKit/chromium/ChangeLog	2013-04-03 06:17:21 UTC (rev 147520)
@@ -1,3 +1,15 @@
+2013-04-02  James Craig  <[email protected]>
+
+        AXRoleDescripton for DL is "list"; should be "description list"
+        https://bugs.webkit.org/show_bug.cgi?id=111665
+
+        Reviewed by Chris Fleizach.
+
+        Defining the role and description strings.
+
+        * public/WebAccessibilityRole.h:
+        * src/AssertMatchingEnums.cpp:
+
 2013-04-02  Raymond Toy  <[email protected]>
 
         Add support for using ARM FFT in WebAudio

Modified: trunk/Source/WebKit/chromium/public/WebAccessibilityRole.h (147519 => 147520)


--- trunk/Source/WebKit/chromium/public/WebAccessibilityRole.h	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebKit/chromium/public/WebAccessibilityRole.h	2013-04-03 06:17:21 UTC (rev 147520)
@@ -57,6 +57,7 @@
     WebAccessibilityRoleColumnHeader,
     WebAccessibilityRoleComboBox,
     WebAccessibilityRoleDefinition,
+    WebAccessibilityRoleDescriptionList,
     WebAccessibilityRoleDescriptionListTerm,
     WebAccessibilityRoleDescriptionListDetail,
     WebAccessibilityRoleDirectory,

Modified: trunk/Source/WebKit/chromium/src/AssertMatchingEnums.cpp (147519 => 147520)


--- trunk/Source/WebKit/chromium/src/AssertMatchingEnums.cpp	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Source/WebKit/chromium/src/AssertMatchingEnums.cpp	2013-04-03 06:17:21 UTC (rev 147520)
@@ -226,6 +226,7 @@
 COMPILE_ASSERT_MATCHING_ENUM(WebAccessibilityRoleMenuListPopup, MenuListPopupRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAccessibilityRoleTableHeaderContainer, TableHeaderContainerRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAccessibilityRoleDefinition, DefinitionRole);
+COMPILE_ASSERT_MATCHING_ENUM(WebAccessibilityRoleDescriptionList, DescriptionListRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAccessibilityRoleDescriptionListTerm, DescriptionListTermRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAccessibilityRoleDescriptionListDetail, DescriptionListDetailRole);
 COMPILE_ASSERT_MATCHING_ENUM(WebAccessibilityRoleAnnotation, AnnotationRole);

Modified: trunk/Tools/ChangeLog (147519 => 147520)


--- trunk/Tools/ChangeLog	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Tools/ChangeLog	2013-04-03 06:17:21 UTC (rev 147520)
@@ -1,3 +1,14 @@
+2013-04-02  James Craig  <[email protected]>
+
+        AXRoleDescripton for DL is "list"; should be "description list"
+        https://bugs.webkit.org/show_bug.cgi?id=111665
+
+        Reviewed by Chris Fleizach.
+
+        Defining the role and description strings.
+
+        * DumpRenderTree/chromium/TestRunner/src/AccessibilityUIElementChromium.cpp:
+
 2013-04-02  Roger Fong  <[email protected]>
 
         Unreviewed. Disable running tests on Win EWS queue.

Modified: trunk/Tools/DumpRenderTree/chromium/TestRunner/src/AccessibilityUIElementChromium.cpp (147519 => 147520)


--- trunk/Tools/DumpRenderTree/chromium/TestRunner/src/AccessibilityUIElementChromium.cpp	2013-04-03 06:00:21 UTC (rev 147519)
+++ trunk/Tools/DumpRenderTree/chromium/TestRunner/src/AccessibilityUIElementChromium.cpp	2013-04-03 06:17:21 UTC (rev 147520)
@@ -170,6 +170,8 @@
         return result.append("TableHeaderContainer");
     case WebAccessibilityRoleDefinition:
         return result.append("Definition");
+    case WebAccessibilityRoleDescriptionList:
+        return result.append("DescriptionList");
     case WebAccessibilityRoleDescriptionListTerm:
         return result.append("DescriptionListTerm");
     case WebAccessibilityRoleDescriptionListDetail:

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to