Changeset 166175 in webkit
- Timestamp:
- Mar 24, 2014 10:08:00 AM (10 years ago)
- Location:
- trunk
- Files:
-
- 2 added
- 16 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r166174 r166175 1 2014-03-24 Chris Fleizach <cfleizach@apple.com> 2 3 <hr> should expose AXRole/AXSubrole, etc 4 https://bugs.webkit.org/show_bug.cgi?id=109015 5 6 Reviewed by Mario Sanchez Prada. 7 8 * accessibility/lists.html: 9 - An errant <hr> element was in this test which changed results 10 * accessibility/roles-computedRoleString-expected.txt: 11 * accessibility/roles-exposed.html: 12 * platform/gtk/accessibility/roles-exposed-expected.txt: Added. 13 * platform/mac/accessibility/hr-element-expected.txt: Added. 14 * platform/mac/accessibility/hr-element.html: Added. 15 * platform/mac/accessibility/lists-expected.txt: 16 * platform/mac/accessibility/roles-exposed-expected.txt: 17 1 18 2014-03-24 Antti Koivisto <antti@apple.com> 2 19 -
trunk/LayoutTests/accessibility/lists.html
r140974 r166175 26 26 </dl> 27 27 28 <BR><BR><BR>< hr><BR>28 <BR><BR><BR><BR> 29 29 <div id="result"></div> 30 30 -
trunk/LayoutTests/accessibility/roles-computedRoleString-expected.txt
r163891 r166175 20 20 PASS: h5 -> heading. 21 21 PASS: h5 -> heading. 22 PASS: hr -> separator. 22 23 PASS: img[alt='X'] -> img. 23 24 PASS: input[type='button'] -> button. -
trunk/LayoutTests/accessibility/roles-exposed.html
r162067 r166175 51 51 <h5 data-platform="atk,mac" data-role="AXHeading" data-subrole="" data-roledescription="heading" class="ex">X</h5> 52 52 <h6 data-platform="atk,mac" data-role="AXHeading" data-subrole="" data-roledescription="heading" class="ex">X</h6> 53 < !-- http://webkit.org/b/109015 --><hr data-platform="atk,mac" data-role="" data-role-atk="AXSeparator" data-subrole="" data-roledescription="" class="ex">53 <hr data-platform="atk,mac" data-role-mac="AXSplitter" data-role-atk="AXSeparator" data-subrole="AXContentSeparator" data-roledescription="separator" class="ex"> 54 54 <!-- skipped <html> --> 55 55 <!-- [ATK] Skipped: not focusable --><i data-platform="mac" data-role="" data-subrole="" data-roledescription="" class="ex">X</i> -
trunk/LayoutTests/platform/mac/accessibility/lists-expected.txt
r158025 r166175 111 111 AXHelp: 112 112 AXParent: <AXGroup> 113 AXSize: NSSize: {784, 54} 114 AXTitle: 115 AXDescription: 116 AXValue: 117 AXFocused: 0 118 AXEnabled: 1 119 AXWindow: <AXGroup> 120 AXSelectedTextMarkerRange: (null) 121 AXStartTextMarker: <AXGroup> 122 AXEndTextMarker: <AXGroup> 123 AXVisited: 0 124 AXLinkedUIElements: (null) 125 AXSelected: 0 126 AXBlockQuoteLevel: 0 127 AXTopLevelUIElement: <AXGroup> 128 AXLanguage: 129 AXDOMIdentifier: 130 AXDOMClassList: <array of size 0> 131 AXTitleUIElement: (null) 132 AXElementBusy: 0 133 134 ------------ 135 AXRole: AXGroup 136 AXSubrole: (null) 137 AXRoleDescription: group 138 AXChildren: <array of size 0> 139 AXHelp: 140 AXParent: <AXGroup> 141 AXSize: NSSize: {784, 18} 113 AXSize: NSSize: {784, 72} 142 114 AXTitle: 143 115 AXDescription: -
trunk/LayoutTests/platform/mac/accessibility/roles-exposed-expected.txt
r162082 r166175 171 171 172 172 PASS: hr 173 AXRole: 174 AXSubrole: 175 AXRoleDescription: 173 AXRole: AXSplitter 174 AXSubrole: AXContentSeparator 175 AXRoleDescription: separator 176 176 177 177 PASS: i -
trunk/Source/WebCore/ChangeLog
r166173 r166175 1 2014-03-24 Chris Fleizach <cfleizach@apple.com> 2 3 <hr> should expose AXRole/AXSubrole, etc 4 https://bugs.webkit.org/show_bug.cgi?id=109015 5 6 Reviewed by Mario Sanchez Prada. 7 8 Expose the <hr> element to Mac and iOS through the accessibility interface. 9 10 Test: platform/mac/accessibility/hr-element.html 11 12 * English.lproj/Localizable.strings: 13 * accessibility/AccessibilityNodeObject.cpp: 14 (WebCore::AccessibilityNodeObject::isEnabled): 15 * accessibility/AccessibilityObject.cpp: 16 (WebCore::AccessibilityObject::computedRoleString): 17 * accessibility/AccessibilityRenderObject.cpp: 18 (WebCore::AccessibilityRenderObject::computeAccessibilityIsIgnored): 19 * accessibility/atk/AccessibilityObjectAtk.cpp: 20 (WebCore::AccessibilityObject::accessibilityPlatformIncludesObject): 21 * accessibility/ios/WebAccessibilityObjectWrapperIOS.mm: 22 (-[WebAccessibilityObjectWrapper determineIsAccessibilityElement]): 23 (-[WebAccessibilityObjectWrapper accessibilityLabel]): 24 * accessibility/mac/WebAccessibilityObjectWrapperMac.mm: 25 (createAccessibilityRoleMap): 26 (-[WebAccessibilityObjectWrapper subrole]): 27 (-[WebAccessibilityObjectWrapper roleDescription]): 28 * platform/LocalizedStrings.cpp: 29 (WebCore::AXHorizontalRuleDescriptionText): 30 * platform/LocalizedStrings.h: 31 1 32 2014-03-24 Antti Koivisto <antti@apple.com> 2 33 … … 3063 3094 * platform/ios/WebSafeGCActivityCallbackIOS.h: 3064 3095 3065 =======3066 3096 2014-03-19 Antti Koivisto <antti@apple.com> 3067 3097 -
trunk/Source/WebCore/English.lproj/Localizable.strings
r164955 r166175 932 932 "select" = "select"; 933 933 934 /* accessibility role description for a horizontal rule [<hr>] */ 935 "separator" = "separator"; 936 934 937 /* HTTP result code string */ 935 938 "server error" = "server error"; -
trunk/Source/WebCore/accessibility/AccessibilityNodeObject.cpp
r165676 r166175 641 641 break; 642 642 } 643 643 644 if (roleValue() == HorizontalRuleRole) 645 return false; 646 644 647 Node* node = this->node(); 645 648 if (!node || !node->isElementNode()) -
trunk/Source/WebCore/accessibility/AccessibilityObject.cpp
r166069 r166175 1750 1750 { 1751 1751 // FIXME: Need a few special cases that aren't in the RoleMap: option, etc. http://webkit.org/b/128296 1752 return reverseAriaRoleMap().get(roleValue()); 1752 AccessibilityRole role = roleValue(); 1753 if (role == HorizontalRuleRole) 1754 role = SplitterRole; 1755 1756 return reverseAriaRoleMap().get(role); 1753 1757 } 1754 1758 -
trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp
r166069 r166175 1253 1253 if (ariaRoleAttribute() != UnknownRole) 1254 1254 return false; 1255 1255 1256 if (roleValue() == HorizontalRuleRole) 1257 return false; 1258 1256 1259 // don't ignore labels, because they serve as TitleUIElements 1257 1260 Node* node = m_renderer->node(); -
trunk/Source/WebCore/accessibility/atk/AccessibilityObjectAtk.cpp
r165699 r166175 43 43 44 44 AccessibilityRole role = roleValue(); 45 if (role == HorizontalRuleRole)46 return IncludeObject;47 48 45 // We expose the slider as a whole but not its value indicator. 49 46 if (role == SliderThumbRole) -
trunk/Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm
r165848 r166175 45 45 #import "IntRect.h" 46 46 #import "IntSize.h" 47 #import "LocalizedStrings.h" 47 48 #import "Page.h" 48 49 #import "Range.h" … … 621 622 case TabRole: 622 623 case DocumentMathRole: 624 case HorizontalRuleRole: 623 625 return true; 624 626 case StaticTextRole: … … 751 753 NSString *axDescription = [self accessibilityDescription]; 752 754 NSString *landmarkDescription = [self ariaLandmarkRoleDescription]; 753 755 754 756 NSMutableString *result = [NSMutableString string]; 755 757 if (m_object->roleValue() == HorizontalRuleRole) 758 appendStringToResult(result, AXHorizontalRuleDescriptionText()); 759 756 760 appendStringToResult(result, axTitle); 757 761 appendStringToResult(result, axDescription); -
trunk/Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperMac.mm
r165676 r166175 115 115 #ifndef NSAccessibilityDescriptionListSubrole 116 116 #define NSAccessibilityDescriptionListSubrole @"AXDescriptionList" 117 #endif 118 119 #ifndef NSAccessibilityContentSeparatorSubrole 120 #define NSAccessibilityContentSeparatorSubrole @"AXContentSeparator" 117 121 #endif 118 122 … … 1917 1921 { MathElementRole, NSAccessibilityGroupRole }, 1918 1922 { AudioRole, NSAccessibilityGroupRole }, 1919 { VideoRole, NSAccessibilityGroupRole } 1923 { VideoRole, NSAccessibilityGroupRole }, 1924 { HorizontalRuleRole, NSAccessibilitySplitterRole } 1920 1925 }; 1921 1926 AccessibilityRoleMap& roleMap = *new AccessibilityRoleMap; … … 1962 1967 } 1963 1968 } 1969 1970 if (m_object->roleValue() == HorizontalRuleRole) 1971 return NSAccessibilityContentSeparatorSubrole; 1964 1972 1965 1973 if (m_object->isSpinButtonPart()) { … … 2156 2164 } 2157 2165 2166 if (m_object->roleValue() == HorizontalRuleRole) 2167 return AXHorizontalRuleDescriptionText(); 2168 2158 2169 // AppKit also returns AXTab for the role description for a tab item. 2159 2170 if (m_object->isTabItem()) -
trunk/Source/WebCore/platform/LocalizedStrings.cpp
r165676 r166175 701 701 return String(); 702 702 } 703 704 String AXHorizontalRuleDescriptionText() 705 { 706 return WEB_UI_STRING("separator", "accessibility role description for a horizontal rule [<hr>]"); 707 } 708 703 709 #endif // PLATFORM(COCOA) 704 710 -
trunk/Source/WebCore/platform/LocalizedStrings.h
r165676 r166175 182 182 #if PLATFORM(COCOA) 183 183 String AXARIAContentGroupText(const String& ariaType); 184 String AXHorizontalRuleDescriptionText(); 184 185 #endif 185 186
Note: See TracChangeset
for help on using the changeset viewer.