Title: [251515] trunk
Revision
251515
Author
repst...@apple.com
Date
2019-10-23 17:09:48 -0700 (Wed, 23 Oct 2019)

Log Message

Unreviewed, rolling out r251469.

Broke accessibility/ios-simulator/link-with-images-text.html
on iOS

Reverted changeset:

"AX: Implement support for new ARIA roles: code, strong,
emphasis, generic"
https://bugs.webkit.org/show_bug.cgi?id=203257
https://trac.webkit.org/changeset/251469# Please enter the commit message for your changes. Lines starting

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (251514 => 251515)


--- trunk/LayoutTests/ChangeLog	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/ChangeLog	2019-10-24 00:09:48 UTC (rev 251515)
@@ -1,3 +1,17 @@
+2019-10-23  Russell Epstein  <repst...@apple.com>
+
+        Unreviewed, rolling out r251469.
+
+        Broke accessibility/ios-simulator/link-with-images-text.html
+        on iOS
+
+        Reverted changeset:
+
+        "AX: Implement support for new ARIA roles: code, strong,
+        emphasis, generic"
+        https://bugs.webkit.org/show_bug.cgi?id=203257
+        https://trac.webkit.org/changeset/251469
+
 2019-10-23  Truitt Savell  <tsav...@apple.com>
 
         Unreviewed, rolling out r251261.

Modified: trunk/LayoutTests/accessibility/gtk/xml-roles-exposed-expected.txt (251514 => 251515)


--- trunk/LayoutTests/accessibility/gtk/xml-roles-exposed-expected.txt	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/accessibility/gtk/xml-roles-exposed-expected.txt	2019-10-24 00:09:48 UTC (rev 251515)
@@ -174,25 +174,6 @@
 AXChecked: 0
 AXPlatformAttributes: computed-role:checkbox, xml-roles:checkbox, readonly:false, tag:div, toolkit:WebKitGtk
 ------------
-AXRole: AXStatic
-AXParent: AXWebArea
-AXChildren: 0
-AXPosition:  { 0.00000, 0.00000 }
-AXSize: { 0.00000, 0.00000 }
-AXTitle: 
-AXDescription: 
-AXValue: 
-AXFocusable: 0
-AXFocused: 0
-AXSelectable: 0
-AXSelected: 0
-AXMultiSelectable: 0
-AXEnabled: 1
-AXExpanded: 0
-AXRequired: 0
-AXChecked: 0
-AXPlatformAttributes: computed-role:code, xml-roles:code, tag:div, toolkit:WebKitGtk
-------------
 AXRole: AXComboBox
 AXParent: AXWebArea
 AXChildren: 0
@@ -345,25 +326,6 @@
 AXChecked: 0
 AXPlatformAttributes: computed-role:document, xml-roles:document, tag:div, toolkit:WebKitGtk
 ------------
-AXRole: AXStatic
-AXParent: AXWebArea
-AXChildren: 0
-AXPosition:  { 0.00000, 0.00000 }
-AXSize: { 0.00000, 0.00000 }
-AXTitle: 
-AXDescription: 
-AXValue: 
-AXFocusable: 0
-AXFocused: 0
-AXSelectable: 0
-AXSelected: 0
-AXMultiSelectable: 0
-AXEnabled: 1
-AXExpanded: 0
-AXRequired: 0
-AXChecked: 0
-AXPlatformAttributes: computed-role:emphasis, xml-roles:emphasis, tag:div, toolkit:WebKitGtk
-------------
 AXRole: AXGroup
 AXParent: AXWebArea
 AXChildren: 0
@@ -421,25 +383,6 @@
 AXChecked: 0
 AXPlatformAttributes: computed-role:form, xml-roles:form, tag:div, toolkit:WebKitGtk
 ------------
-AXRole: AXSection
-AXParent: AXWebArea
-AXChildren: 0
-AXPosition:  { 0.00000, 0.00000 }
-AXSize: { 0.00000, 0.00000 }
-AXTitle: 
-AXDescription: 
-AXValue: 
-AXFocusable: 0
-AXFocused: 0
-AXSelectable: 0
-AXSelected: 0
-AXMultiSelectable: 0
-AXEnabled: 1
-AXExpanded: 0
-AXRequired: 0
-AXChecked: 0
-AXPlatformAttributes: computed-role:generic, xml-roles:generic, tag:div, toolkit:WebKitGtk
-------------
 AXRole: AXGroup
 AXParent: AXWebArea
 AXChildren: 0
@@ -897,25 +840,6 @@
 AXChecked: 0
 AXPlatformAttributes: atomic:true, relevant:additions text, live:polite, container-live-role:status, container-atomic:true, container-relevant:additions text, container-live:polite, computed-role:status, xml-roles:status, tag:div, toolkit:WebKitGtk
 ------------
-AXRole: AXStatic
-AXParent: AXWebArea
-AXChildren: 0
-AXPosition:  { 0.00000, 0.00000 }
-AXSize: { 0.00000, 0.00000 }
-AXTitle: 
-AXDescription: 
-AXValue: 
-AXFocusable: 0
-AXFocused: 0
-AXSelectable: 0
-AXSelected: 0
-AXMultiSelectable: 0
-AXEnabled: 1
-AXExpanded: 0
-AXRequired: 0
-AXChecked: 0
-AXPlatformAttributes: computed-role:strong, xml-roles:strong, tag:div, toolkit:WebKitGtk
-------------
 AXRole: AXSubscript
 AXParent: AXWebArea
 AXChildren: 0

Modified: trunk/LayoutTests/accessibility/gtk/xml-roles-exposed.html (251514 => 251515)


--- trunk/LayoutTests/accessibility/gtk/xml-roles-exposed.html	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/accessibility/gtk/xml-roles-exposed.html	2019-10-24 00:09:48 UTC (rev 251515)
@@ -14,7 +14,6 @@
  <div role="button"></div>
  <div role="caption"></div>
  <div role="checkbox"></div>
- <div role="code"></div>
  <div role="combobox"></div>
  <div role="complementary"></div>
  <div role="contentinfo"></div>
@@ -23,11 +22,9 @@
  <div role="dialog"></div>
  <div role="directory"></div>
  <div role="document"></div>
- <div role="emphasis"></div>
  <div role="feed"></div>
  <div role="figure"></div>
  <div role="form"></div>
- <div role="generic"></div>
  <div role="group"></div>
  <div role="heading"></div>
  <div role="img"></div>
@@ -53,7 +50,6 @@
  <div role="slider"></div>
  <div role="spinbutton"></div>
  <div role="status"></div>
- <div role="strong"></div>
  <div role="subscript"></div>
  <div role="superscript"></div>
  <div role="switch"></div>

Modified: trunk/LayoutTests/accessibility/roles-computedRoleString.html (251514 => 251515)


--- trunk/LayoutTests/accessibility/roles-computedRoleString.html	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/accessibility/roles-computedRoleString.html	2019-10-24 00:09:48 UTC (rev 251515)
@@ -12,14 +12,12 @@
 <article data-role="article" data-platform="atk,mac" class="ex">X</article>
 <aside data-role="complementary" data-platform="atk,mac" class="ex">X</aside>
 <button data-role="button" data-platform="atk,mac" class="ex">X</button>
-<code data-role="code" data-platform="atk,mac" class="ex">X</code>
-<del data-role="deletion" data-platform="atk,mac" class="ex">X</del>
+<del data-role="deletion" data-platform="atk,mac" class="ex">X</ins>
 <dfn data-role="definition" data-platform="atk,mac" class="ex">X</dfn>
 <dl data-role="" data-platform="atk,mac" class="ex">
     <dt>X</dt>
     <dd>X</dd>
 </dl>
-<em data-role="emphasis" data-platform="atk,mac" class="ex">X</em>
 <footer data-role="contentinfo" data-platform="atk,mac" class="ex">X</footer>
 <form data-role="form" data-platform="atk,mac" class="ex">X</form>
 <header data-role="banner" data-platform="atk,mac" class="ex">X</header>
@@ -100,7 +98,7 @@
 </select>
 <small data-role="" data-platform="atk,mac" class="ex">X</small>
 <span data-role="" data-platform="atk,mac" class="ex">X</span>
-<strong data-role="strong" data-platform="atk,mac" class="ex">X</strong>
+<strong data-role="" data-platform="atk,mac" class="ex">X</strong>
 <sub data-role="subscript" data-platform="atk,mac" class="ex">X</sub>
 <sup data-role="superscript" data-platform="atk,mac" class="ex">X</sup>
 <svg data-role="" data-platform="atk,mac" class="ex">X</svg>
@@ -190,7 +188,6 @@
 <div role="document"                 data-role="document" data-platform="atk,mac" class="ex">X</div>
 <div role="figure"                   data-role="figure" data-platform="atk,mac" class="ex">X</div>
 <div role="form"                     data-role="form" data-platform="atk,mac" class="ex">X</div>
-<div role="generic"                  data-role="generic" data-platform="atk,mac" class="ex">X</div>
 <div role="graphics-document"        data-role="document" data-platform="atk,mac" class="ex">X</div>
 <div role="graphics-object"          data-role="group" data-platform="atk,mac" class="ex">X</div>
 <div role="graphics-symbol"          data-role="img" data-platform="atk,mac" class="ex">X</div>

Modified: trunk/LayoutTests/accessibility/roles-exposed.html (251514 => 251515)


--- trunk/LayoutTests/accessibility/roles-exposed.html	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/accessibility/roles-exposed.html	2019-10-24 00:09:48 UTC (rev 251515)
@@ -252,7 +252,6 @@
 <div role="button"                   data-platform="atk,mac" class="ex">X</div>
 <div role="caption"                  data-platform="atk,mac" class="ex">X</div>
 <div role="checkbox"                 data-platform="atk,mac" class="ex">X</div>
-<div role="code"                     data-platform="atk,mac" class="ex">X</div>
 <div role="combobox"                 data-platform="atk,mac" class="ex">X</div>
 <div role="complementary"            data-platform="atk,mac" class="ex">X</div>
 <div role="contentinfo"              data-platform="atk,mac" class="ex">X</div>
@@ -302,10 +301,8 @@
 <div role="doc-tip"                  data-platform="atk,mac" class="ex">X</div>
 <div role="doc-toc"                  data-platform="atk,mac" class="ex">X</div>
 <div role="document"                 data-platform="atk,mac" class="ex">X</div>
-<div role="emphasis"                 data-platform="atk,mac" class="ex">X</div>
 <div role="figure"                   data-platform="atk,mac" class="ex">X</div>
 <div role="form"                     data-platform="atk,mac" class="ex">X</div>
-<div role="generic"                  data-platform="atk,mac" class="ex">X</div>
 <div role="graphics-document"        data-platform="atk,mac" class="ex">X</div>
 <div role="graphics-object"          data-platform="atk,mac" class="ex">X</div>
 <div role="graphics-symbol"          data-platform="atk,mac" class="ex">X</div>
@@ -368,7 +365,6 @@
 <div role="slider"                   data-platform="atk,mac" class="ex">X</div>
 <div role="spinbutton"               data-platform="atk,mac" class="ex">X</div>
 <div role="status"                   data-platform="atk,mac" class="ex">X</div>
-<div role="strong"                   data-platform="atk,mac" class="ex">X</div>
 <div role="subscript"                data-platform="atk,mac" class="ex">X</div>
 <div role="superscript"              data-platform="atk,mac" class="ex">X</div>
 <div role="switch"                   data-platform="atk,mac" class="ex">X</div>

Modified: trunk/LayoutTests/platform/gtk/accessibility/gtk/xml-roles-exposed-expected.txt (251514 => 251515)


--- trunk/LayoutTests/platform/gtk/accessibility/gtk/xml-roles-exposed-expected.txt	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/platform/gtk/accessibility/gtk/xml-roles-exposed-expected.txt	2019-10-24 00:09:48 UTC (rev 251515)
@@ -174,25 +174,6 @@
 AXChecked: 0
 AXPlatformAttributes: computed-role:checkbox, xml-roles:checkbox, readonly:false, tag:div, toolkit:WebKitGtk
 ------------
-AXRole: AXStatic
-AXParent: AXWebArea
-AXChildren: 0
-AXPosition:  { 0.00000, 0.00000 }
-AXSize: { 0.00000, 0.00000 }
-AXTitle: 
-AXDescription: 
-AXValue: 
-AXFocusable: 0
-AXFocused: 0
-AXSelectable: 0
-AXSelected: 0
-AXMultiSelectable: 0
-AXEnabled: 1
-AXExpanded: 0
-AXRequired: 0
-AXChecked: 0
-AXPlatformAttributes: computed-role:code, xml-roles:code, tag:div, toolkit:WebKitGtk
-------------
 AXRole: AXComboBox
 AXParent: AXWebArea
 AXChildren: 0
@@ -345,25 +326,6 @@
 AXChecked: 0
 AXPlatformAttributes: computed-role:document, xml-roles:document, tag:div, toolkit:WebKitGtk
 ------------
-AXRole: AXStatic
-AXParent: AXWebArea
-AXChildren: 0
-AXPosition:  { 0.00000, 0.00000 }
-AXSize: { 0.00000, 0.00000 }
-AXTitle: 
-AXDescription: 
-AXValue: 
-AXFocusable: 0
-AXFocused: 0
-AXSelectable: 0
-AXSelected: 0
-AXMultiSelectable: 0
-AXEnabled: 1
-AXExpanded: 0
-AXRequired: 0
-AXChecked: 0
-AXPlatformAttributes: computed-role:emphasis, xml-roles:emphasis, tag:div, toolkit:WebKitGtk
-------------
 AXRole: AXGroup
 AXParent: AXWebArea
 AXChildren: 0
@@ -421,25 +383,6 @@
 AXChecked: 0
 AXPlatformAttributes: computed-role:form, xml-roles:form, tag:div, toolkit:WebKitGtk
 ------------
-AXRole: AXSection
-AXParent: AXWebArea
-AXChildren: 0
-AXPosition:  { 0.00000, 0.00000 }
-AXSize: { 0.00000, 0.00000 }
-AXTitle: 
-AXDescription: 
-AXValue: 
-AXFocusable: 0
-AXFocused: 0
-AXSelectable: 0
-AXSelected: 0
-AXMultiSelectable: 0
-AXEnabled: 1
-AXExpanded: 0
-AXRequired: 0
-AXChecked: 0
-AXPlatformAttributes: computed-role:generic, xml-roles:generic, tag:div, toolkit:WebKitGtk
-------------
 AXRole: AXGroup
 AXParent: AXWebArea
 AXChildren: 0
@@ -897,25 +840,6 @@
 AXChecked: 0
 AXPlatformAttributes: atomic:true, relevant:additions text, live:polite, container-live-role:status, container-atomic:true, container-relevant:additions text, container-live:polite, computed-role:status, xml-roles:status, tag:div, toolkit:WebKitGtk
 ------------
-AXRole: AXStatic
-AXParent: AXWebArea
-AXChildren: 0
-AXPosition:  { 0.00000, 0.00000 }
-AXSize: { 0.00000, 0.00000 }
-AXTitle: 
-AXDescription: 
-AXValue: 
-AXFocusable: 0
-AXFocused: 0
-AXSelectable: 0
-AXSelected: 0
-AXMultiSelectable: 0
-AXEnabled: 1
-AXExpanded: 0
-AXRequired: 0
-AXChecked: 0
-AXPlatformAttributes: computed-role:strong, xml-roles:strong, tag:div, toolkit:WebKitGtk
-------------
 AXRole: AXSubscript
 AXParent: AXWebArea
 AXChildren: 0

Modified: trunk/LayoutTests/platform/gtk/accessibility/roles-computedRoleString-expected.txt (251514 => 251515)


--- trunk/LayoutTests/platform/gtk/accessibility/roles-computedRoleString-expected.txt	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/platform/gtk/accessibility/roles-computedRoleString-expected.txt	2019-10-24 00:09:48 UTC (rev 251515)
@@ -7,11 +7,9 @@
 PASS: article -> article. 
 PASS: aside -> complementary. 
 PASS: button -> button. 
-PASS: code -> code. 
 PASS: del -> deletion. 
 PASS: dfn -> definition. 
 PASS: dl -> . 
-PASS: em -> emphasis. 
 PASS: footer -> contentinfo. 
 PASS: form -> form. 
 PASS: header -> banner. 
@@ -64,7 +62,6 @@
 PASS: optgroup -> option. 
 PASS: option -> option. 
 PASS: option -> option. 
-PASS: strong -> strong. 
 PASS: sub -> subscript. 
 PASS: sup -> superscript. 
 PASS: table -> table. 
@@ -118,7 +115,6 @@
 PASS: div[role="document"] -> document. 
 PASS: div[role="figure"] -> figure. 
 PASS: div[role="form"] -> form. 
-PASS: div[role="generic"] -> generic. 
 PASS: div[role="graphics-document"] -> document. 
 PASS: div[role="graphics-object"] -> group. 
 PASS: div[role="graphics-symbol"] -> img. 

Modified: trunk/LayoutTests/platform/gtk/accessibility/roles-exposed-expected.txt (251514 => 251515)


--- trunk/LayoutTests/platform/gtk/accessibility/roles-exposed-expected.txt	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/platform/gtk/accessibility/roles-exposed-expected.txt	2019-10-24 00:09:48 UTC (rev 251515)
@@ -71,7 +71,7 @@
       AXRole: AXSection
       
 em
-      AXRole: AXStatic
+      AXRole: 
       
 fieldset
       AXRole: AXGroup
@@ -386,7 +386,7 @@
       AXRole: AXStatic
       
 strong
-      AXRole: AXStatic
+      AXRole: 
       
 sub
       AXRole: AXSubscript
@@ -514,9 +514,6 @@
 div[role=checkbox]
       AXRole: AXCheckBox
       
-div[role=code]
-      AXRole: AXStatic
-      
 div[role=combobox]
       AXRole: AXComboBox
       
@@ -658,9 +655,6 @@
 div[role=document]
       AXRole: AXDocument
       
-div[role=emphasis]
-      AXRole: AXStatic
-      
 div[role=figure]
       AXRole: AXGroup
       
@@ -667,9 +661,6 @@
 div[role=form]
       AXRole: AXLandmarkForm
       
-div[role=generic]
-      AXRole: AXSection
-      
 div[role=graphics-document]
       AXRole: AXDocument
       
@@ -817,9 +808,6 @@
 div[role=status]
       AXRole: AXStatusBar
       
-div[role=strong]
-      AXRole: AXStatic
-      
 div[role=subscript]
       AXRole: AXSubscript
       

Modified: trunk/LayoutTests/platform/mac/accessibility/roles-computedRoleString-expected.txt (251514 => 251515)


--- trunk/LayoutTests/platform/mac/accessibility/roles-computedRoleString-expected.txt	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/platform/mac/accessibility/roles-computedRoleString-expected.txt	2019-10-24 00:09:48 UTC (rev 251515)
@@ -7,11 +7,9 @@
 PASS: article -> article. 
 PASS: aside -> complementary. 
 PASS: button -> button. 
-PASS: code -> code. 
 PASS: del -> deletion. 
 PASS: dfn -> definition. 
 PASS: dl -> . 
-PASS: em -> emphasis. 
 PASS: footer -> contentinfo. 
 PASS: form -> form. 
 PASS: header -> banner. 
@@ -64,7 +62,6 @@
 PASS: optgroup -> option. 
 PASS: option -> option. 
 PASS: option -> option. 
-PASS: strong -> strong. 
 PASS: sub -> subscript. 
 PASS: sup -> superscript. 
 PASS: table -> table. 
@@ -117,7 +114,6 @@
 PASS: div[role="document"] -> document. 
 PASS: div[role="figure"] -> figure. 
 PASS: div[role="form"] -> form. 
-PASS: div[role="generic"] -> generic. 
 PASS: div[role="graphics-document"] -> document. 
 PASS: div[role="graphics-object"] -> group. 
 PASS: div[role="graphics-symbol"] -> img. 

Modified: trunk/LayoutTests/platform/mac/accessibility/roles-exposed-expected.txt (251514 => 251515)


--- trunk/LayoutTests/platform/mac/accessibility/roles-exposed-expected.txt	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/platform/mac/accessibility/roles-exposed-expected.txt	2019-10-24 00:09:48 UTC (rev 251515)
@@ -115,9 +115,9 @@
       AXRoleDescription: group
       
 em
-      AXRole: AXGroup
-      AXSubrole: AXEmphasisStyleGroup
-      AXRoleDescription: group
+      AXRole: 
+      AXSubrole: 
+      AXRoleDescription: 
       
 fieldset
       AXRole: AXGroup
@@ -645,9 +645,9 @@
       AXRoleDescription: 
       
 strong
-      AXRole: AXGroup
-      AXSubrole: AXStrongStyleGroup
-      AXRoleDescription: group
+      AXRole: 
+      AXSubrole: 
+      AXRoleDescription: 
       
 sub
       AXRole: AXGroup
@@ -859,11 +859,6 @@
       AXSubrole: 
       AXRoleDescription: checkbox
       
-div[role=code]
-      AXRole: AXGroup
-      AXSubrole: AXCodeStyleGroup
-      AXRoleDescription: group
-      
 div[role=combobox]
       AXRole: AXComboBox
       AXSubrole: 
@@ -1099,11 +1094,6 @@
       AXSubrole: AXDocument
       AXRoleDescription: document
       
-div[role=emphasis]
-      AXRole: AXGroup
-      AXSubrole: AXEmphasisStyleGroup
-      AXRoleDescription: group
-      
 div[role=figure]
       AXRole: AXGroup
       AXSubrole: 
@@ -1114,11 +1104,6 @@
       AXSubrole: 
       AXRoleDescription: group
       
-div[role=generic]
-      AXRole: AXGroup
-      AXSubrole: 
-      AXRoleDescription: group
-      
 div[role=graphics-document]
       AXRole: AXGroup
       AXSubrole: AXDocument
@@ -1364,11 +1349,6 @@
       AXSubrole: AXApplicationStatus
       AXRoleDescription: application status
       
-div[role=strong]
-      AXRole: AXGroup
-      AXSubrole: AXStrongStyleGroup
-      AXRoleDescription: group
-      
 div[role=subscript]
       AXRole: AXGroup
       AXSubrole: AXSubscriptStyleGroup

Modified: trunk/LayoutTests/platform/mac-wk2/accessibility/roles-exposed-expected.txt (251514 => 251515)


--- trunk/LayoutTests/platform/mac-wk2/accessibility/roles-exposed-expected.txt	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/LayoutTests/platform/mac-wk2/accessibility/roles-exposed-expected.txt	2019-10-24 00:09:48 UTC (rev 251515)
@@ -115,9 +115,9 @@
       AXRoleDescription: group
       
 em
-      AXRole: AXGroup
-      AXSubrole: AXEmphasisStyleGroup
-      AXRoleDescription: group
+      AXRole: 
+      AXSubrole: 
+      AXRoleDescription: 
       
 fieldset
       AXRole: AXGroup
@@ -645,9 +645,9 @@
       AXRoleDescription: 
       
 strong
-      AXRole: AXGroup
-      AXSubrole: AXStrongStyleGroup
-      AXRoleDescription: group
+      AXRole: 
+      AXSubrole: 
+      AXRoleDescription: 
       
 sub
       AXRole: AXGroup
@@ -859,11 +859,6 @@
       AXSubrole: 
       AXRoleDescription: checkbox
       
-div[role=code]
-      AXRole: AXGroup
-      AXSubrole: AXCodeStyleGroup
-      AXRoleDescription: group
-      
 div[role=combobox]
       AXRole: AXComboBox
       AXSubrole: 
@@ -1099,11 +1094,6 @@
       AXSubrole: AXDocument
       AXRoleDescription: document
       
-div[role=emphasis]
-      AXRole: AXGroup
-      AXSubrole: AXEmphasisStyleGroup
-      AXRoleDescription: group
-      
 div[role=figure]
       AXRole: AXGroup
       AXSubrole: 
@@ -1114,11 +1104,6 @@
       AXSubrole: 
       AXRoleDescription: group
       
-div[role=generic]
-      AXRole: AXGroup
-      AXSubrole: 
-      AXRoleDescription: group
-      
 div[role=graphics-document]
       AXRole: AXGroup
       AXSubrole: AXDocument
@@ -1364,11 +1349,6 @@
       AXSubrole: AXApplicationStatus
       AXRoleDescription: application status
       
-div[role=strong]
-      AXRole: AXGroup
-      AXSubrole: AXStrongStyleGroup
-      AXRoleDescription: group
-      
 div[role=subscript]
       AXRole: AXGroup
       AXSubrole: AXSubscriptStyleGroup

Modified: trunk/Source/WebCore/ChangeLog (251514 => 251515)


--- trunk/Source/WebCore/ChangeLog	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/Source/WebCore/ChangeLog	2019-10-24 00:09:48 UTC (rev 251515)
@@ -1,3 +1,17 @@
+2019-10-23  Russell Epstein  <repst...@apple.com>
+
+        Unreviewed, rolling out r251469.
+
+        Broke accessibility/ios-simulator/link-with-images-text.html
+        on iOS
+
+        Reverted changeset:
+
+        "AX: Implement support for new ARIA roles: code, strong,
+        emphasis, generic"
+        https://bugs.webkit.org/show_bug.cgi?id=203257
+        https://trac.webkit.org/changeset/251469
+
 2019-10-23  Chris Dumez  <cdu...@apple.com>
 
         FileSystemDirectoryReader / FileSystemEntry should not prevent entering the back/forward cache

Modified: trunk/Source/WebCore/accessibility/AccessibilityObject.cpp (251514 => 251515)


--- trunk/Source/WebCore/accessibility/AccessibilityObject.cpp	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/Source/WebCore/accessibility/AccessibilityObject.cpp	2019-10-24 00:09:48 UTC (rev 251515)
@@ -2414,7 +2414,6 @@
         { "button", AccessibilityRole::Button },
         { "caption", AccessibilityRole::Caption },
         { "checkbox", AccessibilityRole::CheckBox },
-        { "code", AccessibilityRole::Code },
         { "complementary", AccessibilityRole::LandmarkComplementary },
         { "contentinfo", AccessibilityRole::LandmarkContentInfo },
         { "deletion", AccessibilityRole::Deletion },
@@ -2474,11 +2473,9 @@
         { "combobox", AccessibilityRole::ComboBox },
         { "definition", AccessibilityRole::Definition },
         { "document", AccessibilityRole::Document },
-        { "emphasis", AccessibilityRole::Emphasis },
         { "feed", AccessibilityRole::Feed },
         { "form", AccessibilityRole::Form },
         { "rowheader", AccessibilityRole::RowHeader },
-        { "generic", AccessibilityRole::Generic },
         { "group", AccessibilityRole::ApplicationGroup },
         { "heading", AccessibilityRole::Heading },
         { "img", AccessibilityRole::Image },
@@ -2516,7 +2513,6 @@
         { "slider", AccessibilityRole::Slider },
         { "spinbutton", AccessibilityRole::SpinButton },
         { "status", AccessibilityRole::ApplicationStatus },
-        { "strong", AccessibilityRole::Strong },
         { "subscript", AccessibilityRole::Subscript },
         { "superscript", AccessibilityRole::Superscript },
         { "switch", AccessibilityRole::Switch },
@@ -3589,8 +3585,7 @@
     || node->hasTagName(preTag) || node->hasTagName(sampTag)
     || node->hasTagName(varTag) || node->hasTagName(citeTag)
     || node->hasTagName(insTag) || node->hasTagName(delTag)
-    || node->hasTagName(supTag) || node->hasTagName(subTag)
-    || node->hasTagName(emTag) || node->hasTagName(strongTag);
+    || node->hasTagName(supTag) || node->hasTagName(subTag);
 }
 
 bool AccessibilityObject::isFigureElement() const

Modified: trunk/Source/WebCore/accessibility/AccessibilityObjectInterface.h (251514 => 251515)


--- trunk/Source/WebCore/accessibility/AccessibilityObjectInterface.h	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/Source/WebCore/accessibility/AccessibilityObjectInterface.h	2019-10-24 00:09:48 UTC (rev 251515)
@@ -70,7 +70,6 @@
     Caption,
     Cell,
     CheckBox,
-    Code,
     ColorWell,
     Column,
     ColumnHeader,
@@ -90,13 +89,11 @@
     DocumentNote,
     Drawer,
     EditableText,
-    Emphasis,
     Feed,
     Figure,
     Footer,
     Footnote,
     Form,
-    Generic,
     GraphicsDocument,
     GraphicsObject,
     GraphicsSymbol,
@@ -171,7 +168,6 @@
     SplitGroup,
     Splitter,
     StaticText,
-    Strong,
     Subscript,
     Summary,
     Superscript,

Modified: trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp (251514 => 251515)


--- trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/Source/WebCore/accessibility/AccessibilityRenderObject.cpp	2019-10-24 00:09:48 UTC (rev 251515)
@@ -2816,16 +2816,10 @@
         return AccessibilityRole::SVGRoot;
     
     if (isStyleFormatGroup()) {
-        if (node->hasTagName(codeTag))
-            return AccessibilityRole::Code;
         if (node->hasTagName(delTag))
             return AccessibilityRole::Deletion;
-        if (node->hasTagName(emTag))
-            return AccessibilityRole::Emphasis;
         if (node->hasTagName(insTag))
             return AccessibilityRole::Insertion;
-        if (node->hasTagName(strongTag))
-            return AccessibilityRole::Strong;
         if (node->hasTagName(subTag))
             return AccessibilityRole::Subscript;
         if (node->hasTagName(supTag))

Modified: trunk/Source/WebCore/accessibility/atk/WebKitAccessible.cpp (251514 => 251515)


--- trunk/Source/WebCore/accessibility/atk/WebKitAccessible.cpp	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/Source/WebCore/accessibility/atk/WebKitAccessible.cpp	2019-10-24 00:09:48 UTC (rev 251515)
@@ -728,7 +728,6 @@
 #endif
     case AccessibilityRole::ApplicationTextGroup:
     case AccessibilityRole::Div:
-    case AccessibilityRole::Generic:
     case AccessibilityRole::Pre:
     case AccessibilityRole::SVGText:
     case AccessibilityRole::TextGroup:
@@ -819,10 +818,7 @@
         return ATK_ROLE_SUBSCRIPT;
     case AccessibilityRole::Superscript:
         return ATK_ROLE_SUPERSCRIPT;
-    case AccessibilityRole::Code:
-    case AccessibilityRole::Emphasis:
     case AccessibilityRole::Inline:
-    case AccessibilityRole::Strong:
     case AccessibilityRole::SVGTextPath:
     case AccessibilityRole::SVGTSpan:
     case AccessibilityRole::Time:

Modified: trunk/Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm (251514 => 251515)


--- trunk/Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/Source/WebCore/accessibility/ios/WebAccessibilityObjectWrapperIOS.mm	2019-10-24 00:09:48 UTC (rev 251515)
@@ -945,7 +945,6 @@
     case AccessibilityRole::Canvas:
     case AccessibilityRole::Caption:
     case AccessibilityRole::Cell:
-    case AccessibilityRole::Code:
     case AccessibilityRole::Column:
     case AccessibilityRole::ColumnHeader:
     case AccessibilityRole::Definition:
@@ -961,13 +960,11 @@
     case AccessibilityRole::DocumentNote:
     case AccessibilityRole::Drawer:
     case AccessibilityRole::EditableText:
-    case AccessibilityRole::Emphasis:
     case AccessibilityRole::Feed:
     case AccessibilityRole::Figure:
     case AccessibilityRole::Footer:
     case AccessibilityRole::Footnote:
     case AccessibilityRole::Form:
-    case AccessibilityRole::Generic:
     case AccessibilityRole::GraphicsDocument:
     case AccessibilityRole::GraphicsObject:
     case AccessibilityRole::GraphicsSymbol:
@@ -1018,7 +1015,6 @@
     case AccessibilityRole::SpinButtonPart:
     case AccessibilityRole::SplitGroup:
     case AccessibilityRole::Splitter:
-    case AccessibilityRole::Strong:
     case AccessibilityRole::Subscript:
     case AccessibilityRole::Superscript:
     case AccessibilityRole::Summary:

Modified: trunk/Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperMac.mm (251514 => 251515)


--- trunk/Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperMac.mm	2019-10-24 00:05:58 UTC (rev 251514)
+++ trunk/Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperMac.mm	2019-10-24 00:09:48 UTC (rev 251515)
@@ -2199,10 +2199,6 @@
         { AccessibilityRole::Insertion, NSAccessibilityGroupRole },
         { AccessibilityRole::Subscript, NSAccessibilityGroupRole },
         { AccessibilityRole::Superscript, NSAccessibilityGroupRole },
-        { AccessibilityRole::Code, NSAccessibilityGroupRole },
-        { AccessibilityRole::Emphasis, NSAccessibilityGroupRole },
-        { AccessibilityRole::Strong, NSAccessibilityGroupRole },
-        { AccessibilityRole::Generic, NSAccessibilityGroupRole },
     };
     AccessibilityRoleMap roleMap;
     for (auto& role : roles)
@@ -2416,17 +2412,13 @@
         return @"AXSuperscriptStyleGroup";
     if (role == AccessibilityRole::Subscript)
         return @"AXSubscriptStyleGroup";
-    if (role == AccessibilityRole::Code)
-        return @"AXCodeStyleGroup";
-    if (role == AccessibilityRole::Emphasis)
-        return @"AXEmphasisStyleGroup";
-    if (role == AccessibilityRole::Strong)
-        return @"AXStrongStyleGroup";
 
     if (m_object->isStyleFormatGroup()) {
         if (Node* node = m_object->node()) {
             if (node->hasTagName(kbdTag))
                 return @"AXKeyboardInputStyleGroup";
+            if (node->hasTagName(codeTag))
+                return @"AXCodeStyleGroup";
             if (node->hasTagName(preTag))
                 return @"AXPreformattedStyleGroup";
             if (node->hasTagName(sampTag))
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to