AX: WebArea claims to have a subrole, but always returns nil.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 Sep 2013 16:18:10 +0000 (16:18 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 20 Sep 2013 16:18:10 +0000 (16:18 +0000)
https://bugs.webkit.org/show_bug.cgi?id=121658

Patch by Sam White <samuel_white@apple.com> on 2013-09-20
Reviewed by Chris Fleizach.

Source/WebCore:

No new tests, updated existing AXWebArea related test to reflect unused attribute removal.

* accessibility/mac/WebAccessibilityObjectWrapperMac.mm:
(-[WebAccessibilityObjectWrapper accessibilityAttributeNames]):

LayoutTests:

Updated test output to reflect removal of unused NSAccessibilitySubroleAttribute in AXWebArea.

* platform/mac/accessibility/document-attributes-expected.txt:

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

LayoutTests/ChangeLog
LayoutTests/platform/mac/accessibility/document-attributes-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperMac.mm

index 91c3f16e8626c28679ee4288f1698982d05e64cb..43880f7707804f4a26512ea83ee864c2f3689ca0 100644 (file)
@@ -1,3 +1,14 @@
+2013-09-20  Sam White  <samuel_white@apple.com>
+
+        AX: WebArea claims to have a subrole, but always returns nil.
+        https://bugs.webkit.org/show_bug.cgi?id=121658
+
+        Reviewed by Chris Fleizach.
+
+        Updated test output to reflect removal of unused NSAccessibilitySubroleAttribute in AXWebArea.
+
+        * platform/mac/accessibility/document-attributes-expected.txt:
+
 2013-09-20  Denis Nomiyama  <d.nomiyama@samsung.com>
 
         [ATK][WTR] accessibility/aria-invalid.html times out
index ffacb6dd7d330fc9e6f411db048dd6bcf1fa1460..e32e0a93260a124719a3b17cef9ab4de7a1d68ad 100644 (file)
@@ -1,5 +1,4 @@
 AXRole: AXWebArea
-AXSubrole: (null)
 AXRoleDescription: HTML content
 AXChildren: <array of size 0>
 AXHelp: 
index 6210a1158c690c19a03b31c96eb8256d4ec0296c..25e5d9f63920dabbe6a8de7f3b6d497569f5ec65 100644 (file)
@@ -1,3 +1,15 @@
+2013-09-20  Sam White  <samuel_white@apple.com>
+
+        AX: WebArea claims to have a subrole, but always returns nil.
+        https://bugs.webkit.org/show_bug.cgi?id=121658
+
+        Reviewed by Chris Fleizach.
+
+        No new tests, updated existing AXWebArea related test to reflect unused attribute removal.
+
+        * accessibility/mac/WebAccessibilityObjectWrapperMac.mm:
+        (-[WebAccessibilityObjectWrapper accessibilityAttributeNames]):
+
 2013-09-20  Andreas Kling  <akling@apple.com>
 
         Add valueForLength/minimumValueForLength wrappers to RenderElement.
index 69ff0c76e1ccaf108f298737dde555626970dabf..2d8bb8308792c37a1fd1eebff7e276ecedfb6f2b 100644 (file)
@@ -1143,6 +1143,8 @@ static id textMarkerRangeFromVisiblePositions(AXObjectCache *cache, VisiblePosit
     }
     if (webAreaAttrs == nil) {
         tempArray = [[NSMutableArray alloc] initWithArray:attributes];
+        // WebAreas should not expose AXSubrole.
+        [tempArray removeObject:NSAccessibilitySubroleAttribute];
         [tempArray addObject:@"AXLinkUIElements"];
         [tempArray addObject:@"AXLoaded"];
         [tempArray addObject:@"AXLayoutCount"];