Commit 19da2cde authored by Daniel Clark's avatar Daniel Clark Committed by Commit Bot

Change <figcaption>'s ControlType to Text and AriaRole to 'description'

Change the ControlType of <figcaption> from Group to Text per
https://w3c.github.io/html-aam/#el-figcaption <figcaption>

Also update the AriaRole exposed to UIA from 'group' to 'description';
the spec doesn't comment on this but it is consistent with the current
behavior for other inline wrappers like <mark>.

Bug: 997714
Change-Id: I26c9a55bbdc02bed0707e926be332f7ebb53b27f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769049
Commit-Queue: Kevin Babbitt <kbabbitt@microsoft.com>
Reviewed-by: default avatarKevin Babbitt <kbabbitt@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#690446}
parent 8f29c483
document
++group Name='Fig.1 - A green Box'
++++img Name='This is a green box.'
++++group
++++++description Name='Fig.1 - A green Box'
++++description ControlType='UIA_TextControlTypeId'
++++++description ControlType='UIA_TextControlTypeId' Name='Fig.1 - A green Box'
<!--
@UIA-WIN-ALLOW:ControlType='UIA_TextControlTypeId'
-->
<!DOCTYPE html>
<html>
<body>
......
......@@ -5408,7 +5408,7 @@ base::string16 AXPlatformNodeWin::UIAAriaRole() {
return L"group";
case ax::mojom::Role::kFigcaption:
return L"group";
return L"description";
case ax::mojom::Role::kFigure:
return L"group";
......@@ -6049,7 +6049,7 @@ LONG AXPlatformNodeWin::ComputeUIAControlType() { // NOLINT(runtime/int)
return UIA_GroupControlTypeId;
case ax::mojom::Role::kFigcaption:
return UIA_GroupControlTypeId;
return UIA_TextControlTypeId;
case ax::mojom::Role::kFigure:
return UIA_GroupControlTypeId;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment