Attachment 359439 Details for Bug 193371 – Patch

[patch] Patch

bug-193371-20190117183849.patch (text/plain), 7.81 KB, created by Eric Liang on 2019-01-17 18:38:49 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Eric Liang

Created: 2019-01-17 18:38:49 PST

Size: 7.81 KB

patch

obsolete

>Subversion Revision: 240018
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index e00d3145c707897b692f580be3269ab6d189ed48..0628915906be090384cd47b09adbfba453b4eca9 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,15 @@
>+2019-01-17  Eric Liang  <ericliang@apple.com>
>+
>+        AXSelected attribute on RadioButton should not be settable.
>+        https://bugs.webkit.org/show_bug.cgi?id=193371
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Test: accessibility/set-selected-editable.html
>+
>+        * accessibility/AccessibilityNodeObject.cpp:
>+        (WebCore::AccessibilityNodeObject::canSetSelectedAttribute const):
>+
> 2019-01-15  Myles C. Maxfield  <mmaxfield@apple.com>
> 
>         [WHLSL] Implement the loop checker
>diff --git a/Source/WebCore/accessibility/AccessibilityNodeObject.cpp b/Source/WebCore/accessibility/AccessibilityNodeObject.cpp
>index c2a7ace9066751f9ebdd2e4190d5a157ca188ea4..8b7d6e6ffa15545cda1b45c6177c267916713b0d 100644
>--- a/Source/WebCore/accessibility/AccessibilityNodeObject.cpp
>+++ b/Source/WebCore/accessibility/AccessibilityNodeObject.cpp
>@@ -2232,7 +2232,6 @@ bool AccessibilityNodeObject::canSetSelectedAttribute() const
>     switch (roleValue()) {
>     case AccessibilityRole::Cell:
>     case AccessibilityRole::GridCell:
>-    case AccessibilityRole::RadioButton:
>     case AccessibilityRole::RowHeader:
>     case AccessibilityRole::Row:
>     case AccessibilityRole::TabList:
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index fb54e06ef6722b0c063e0bef3bbeca099124e05a..8b5d24f28cb36fc684a2c52f08c1ccff4fea6db0 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,15 @@
>+2019-01-17  Eric Liang  <ericliang@apple.com>
>+
>+        AXSelected attribute on RadioButton should not be settable.
>+        https://bugs.webkit.org/show_bug.cgi?id=193371
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        This change make RadioButton AXSelected attribute no-settable. If this attribute is not writable, then the trackpad should work correctly.
>+
>+        * accessibility/set-selected-editable-expected.txt: Added.
>+        * accessibility/set-selected-editable.html: Added.
>+
> 2019-01-15  Chris Dumez  <cdumez@apple.com>
> 
>         Unreviewed, rolling out r239993, r239995, r239997, and
>diff --git a/LayoutTests/accessibility/set-selected-editable-expected.txt b/LayoutTests/accessibility/set-selected-editable-expected.txt
>new file mode 100644
>index 0000000000000000000000000000000000000000..709a7786b3b626d64a48d318c55fbe3ea3dc995f
>--- /dev/null
>+++ b/LayoutTests/accessibility/set-selected-editable-expected.txt
>@@ -0,0 +1,51 @@
>+X
>+X
>+X
>+X
>+X
>+X
>+X
>+X
>+X
>+X
>+X
>+tree item
>+Crust
>+Item one
>+Item four
>+Item five
>+  
>+This tests that the accessibility Selected attribute can be settable only on certain elements.
>+
>+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
>+
>+
>+PASS axElement.isAttributeSettable('AXSelected') is false
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is false
>+PASS axElement.isAttributeSettable('AXSelected') is false
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is false
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is false
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is false
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is true
>+PASS axElement.isAttributeSettable('AXSelected') is false
>+PASS axElement.isAttributeSettable('AXSelected') is false
>+PASS axElement.isAttributeSettable('AXSelected') is false
>+
>diff --git a/LayoutTests/accessibility/set-selected-editable.html b/LayoutTests/accessibility/set-selected-editable.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..f9b001e24202246b93f3483b16807f646f2b9900
>--- /dev/null
>+++ b/LayoutTests/accessibility/set-selected-editable.html
>@@ -0,0 +1,102 @@
>+<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
>+<html>
>+<head>
>+<meta charset="utf-8">
>+<script src="../resources/js-test-pre.js"></script>
>+</head>
>+<body id="body">
>+
>+<div id="test0" role="table">
>+  <div id="test1" role="row">
>+    <div id="test2" role="columnheader">X</div>
>+    <div id="test3" role="columnheader">X</div>
>+  </div>
>+  <div id="test4" role="row">
>+    <div id="test5" role="rowheader">X</div>
>+    <div id="test6" role="cell">X</div>
>+  </div>
>+</div>
>+
>+<div id="test7" role="grid">
>+  <div id="test8" role="row">
>+    <div id="test9" role="columnheader">X</div>
>+    <div role="columnheader">X</div>
>+  </div>
>+  <div id="test10" role="row">
>+    <div id="test11" role="rowheader">X</div>
>+    <div id="test12" role="gridcell">X</div>
>+  </div>
>+</div>
>+
>+<div id="test13" role="treegrid">
>+  <div id="test14" role="row">
>+    <div id="test15" role="columnheader">X</div>
>+    <div id="test16" role="gridcell">X</div>
>+    <div id="test17" role="rowheader">X</div>
>+  </div>
>+</div>
>+
>+<ul id="test18" role="tree">
>+    <li id="test19" role="treeitem">tree item</li>
>+</ul>
>+
>+<ul id="test20" role="tablist">
>+<li id="test21" role="tab" tabindex="-1" class="">Crust</li>
>+</ul>
>+
>+<div id="test22" role="menuitemradio" aria-checked="true">Item one</div>
>+<div id="test23" role="menuitemcheckbox" aria-checked="false">Item four</div>
>+<div id="test24" role="menuitem">Item five</div>
>+
>+<input id="test25" type="radio">
>+<input id="test26" type="checkbox" />
>+<input id="test27" type="text" role="combobox">
>+
>+<p id="description"></p>
>+<div id="console"></div>
>+
>+<script>
>+    description("This tests that the accessibility Selected attribute can be settable only on certain elements.");
>+
>+    if (window.accessibilityController) {
>+          var expected = [
>+            "false", //0: table
>+            "true", //1: row
>+            "false", //2: columnheader
>+            "false", //3: columnheader
>+            "true", //4: row
>+            "true", //5: rowheader
>+            "true", //6: cell
>+            "false", //7: grid
>+            "true", //8: row
>+            "false", //9: columnheader
>+            "true", //10: row
>+            "true", //11: rowheader
>+            "true", //12: gridcell
>+            "true", //13: treegrid
>+            "true", //14: row
>+            "false", //15: columnheader
>+            "true", //16: gridcell
>+            "true", //17: rowheader
>+            "true", //18: tree
>+            "true", //19: treeitem
>+            "true", //20: tablist
>+            "true", //21: tab
>+            "true", //22: menuitemradio
>+            "true", //23: menuitemcheckbox
>+            "true", //24: menuitem
>+            "false", //25: radiobutton
>+            "false", //26: checkbox
>+            "false", //27: combobox
>+          ]
>+          for (var i = 0; i <= 27; i++) {
>+              var axElement = accessibilityController.accessibleElementById("test" + i);
>+              shouldBe("axElement.isAttributeSettable('AXSelected')", expected[i]);
>+          }
>+    }
>+
>+</script>
>+
>+<script src="../../resources/js-test-post.js"></script>
>+</body>
>+</html>

Actions: View | Formatted Diff | Diff

Attachments on bug 193371: 359438 | 359439 | 359445 | 359500 | 359516