Title: [271848] branches/safari-611-branch
- Revision
- 271848
- Author
- alanc...@apple.com
- Date
- 2021-01-25 14:13:00 -0800 (Mon, 25 Jan 2021)
Log Message
Cherry-pick r271536. rdar://problem/73477501
AX: increment/decrement synthetic arrow events don't work in ARIA slider examples
https://bugs.webkit.org/show_bug.cgi?id=220626
<rdar://problem/73228924>
Reviewed by Zalan Bujtas.
Source/WebCore:
keyCode is still expected to be filled in with standard codes for arrow keys.
Updated test: accessibility/keyevents-posted-for-increment-actions.html
* accessibility/AccessibilityNodeObject.cpp:
(WebCore::AccessibilityNodeObject::postKeyboardKeysForValueChange):
LayoutTests:
* accessibility/keyevents-posted-for-increment-actions-expected.txt:
* accessibility/keyevents-posted-for-increment-actions.html:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@271536 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-611-branch/LayoutTests/ChangeLog (271847 => 271848)
--- branches/safari-611-branch/LayoutTests/ChangeLog 2021-01-25 22:12:54 UTC (rev 271847)
+++ branches/safari-611-branch/LayoutTests/ChangeLog 2021-01-25 22:13:00 UTC (rev 271848)
@@ -1,5 +1,43 @@
2021-01-25 Alan Coon <alanc...@apple.com>
+ Cherry-pick r271536. rdar://problem/73477501
+
+ AX: increment/decrement synthetic arrow events don't work in ARIA slider examples
+ https://bugs.webkit.org/show_bug.cgi?id=220626
+ <rdar://problem/73228924>
+
+ Reviewed by Zalan Bujtas.
+
+ Source/WebCore:
+
+ keyCode is still expected to be filled in with standard codes for arrow keys.
+
+ Updated test: accessibility/keyevents-posted-for-increment-actions.html
+
+ * accessibility/AccessibilityNodeObject.cpp:
+ (WebCore::AccessibilityNodeObject::postKeyboardKeysForValueChange):
+
+ LayoutTests:
+
+ * accessibility/keyevents-posted-for-increment-actions-expected.txt:
+ * accessibility/keyevents-posted-for-increment-actions.html:
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@271536 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2021-01-15 Chris Fleizach <cfleiz...@apple.com>
+
+ AX: increment/decrement synthetic arrow events don't work in ARIA slider examples
+ https://bugs.webkit.org/show_bug.cgi?id=220626
+ <rdar://problem/73228924>
+
+ Reviewed by Zalan Bujtas.
+
+ * accessibility/keyevents-posted-for-increment-actions-expected.txt:
+ * accessibility/keyevents-posted-for-increment-actions.html:
+
+2021-01-25 Alan Coon <alanc...@apple.com>
+
Cherry-pick r271516. rdar://problem/73477422
[LFC][Integration] REGRESSION (r270123) facebook.com birthday dropdown do not work when creating new account
Modified: branches/safari-611-branch/LayoutTests/accessibility/keyevents-posted-for-increment-actions-expected.txt (271847 => 271848)
--- branches/safari-611-branch/LayoutTests/accessibility/keyevents-posted-for-increment-actions-expected.txt 2021-01-25 22:12:54 UTC (rev 271847)
+++ branches/safari-611-branch/LayoutTests/accessibility/keyevents-posted-for-increment-actions-expected.txt 2021-01-25 22:13:00 UTC (rev 271848)
@@ -5,18 +5,18 @@
Increment/Decrement - LTR
Horizontal orientation
-Keycode received: identifier: right key name: ArrowRight
-Keycode received: identifier: left key name: ArrowLeft
+Keycode received: identifier: right key name: ArrowRight key code: 39
+Keycode received: identifier: left key name: ArrowLeft key code: 37
Vertical orientation
-Keycode received: identifier: up key name: ArrowUp
-Keycode received: identifier: down key name: ArrowDown
+Keycode received: identifier: up key name: ArrowUp key code: 38
+Keycode received: identifier: down key name: ArrowDown key code: 40
Increment/Decrement - RTL
Horizontal orientation
-Keycode received: identifier: left key name: ArrowLeft
-Keycode received: identifier: right key name: ArrowRight
+Keycode received: identifier: left key name: ArrowLeft key code: 37
+Keycode received: identifier: right key name: ArrowRight key code: 39
Vertical orientation
-Keycode received: identifier: up key name: ArrowUp
-Keycode received: identifier: down key name: ArrowDown
+Keycode received: identifier: up key name: ArrowUp key code: 38
+Keycode received: identifier: down key name: ArrowDown key code: 40
PASS successfullyParsed is true
TEST COMPLETE
Modified: branches/safari-611-branch/LayoutTests/accessibility/keyevents-posted-for-increment-actions.html (271847 => 271848)
--- branches/safari-611-branch/LayoutTests/accessibility/keyevents-posted-for-increment-actions.html 2021-01-25 22:12:54 UTC (rev 271847)
+++ branches/safari-611-branch/LayoutTests/accessibility/keyevents-posted-for-increment-actions.html 2021-01-25 22:13:00 UTC (rev 271848)
@@ -20,7 +20,7 @@
var keyCount = 0;
function handleKeyDown(event) {
- debug("Keycode received: identifier: " + event.keyIdentifier + " key name: " + event.key);
+ debug("Keycode received: identifier: " + event.keyIdentifier + " key name: " + event.key + " key code: " + event.keyCode);
event.preventDefault();
event.stopPropagation();
Modified: branches/safari-611-branch/Source/WebCore/ChangeLog (271847 => 271848)
--- branches/safari-611-branch/Source/WebCore/ChangeLog 2021-01-25 22:12:54 UTC (rev 271847)
+++ branches/safari-611-branch/Source/WebCore/ChangeLog 2021-01-25 22:13:00 UTC (rev 271848)
@@ -1,5 +1,47 @@
2021-01-25 Alan Coon <alanc...@apple.com>
+ Cherry-pick r271536. rdar://problem/73477501
+
+ AX: increment/decrement synthetic arrow events don't work in ARIA slider examples
+ https://bugs.webkit.org/show_bug.cgi?id=220626
+ <rdar://problem/73228924>
+
+ Reviewed by Zalan Bujtas.
+
+ Source/WebCore:
+
+ keyCode is still expected to be filled in with standard codes for arrow keys.
+
+ Updated test: accessibility/keyevents-posted-for-increment-actions.html
+
+ * accessibility/AccessibilityNodeObject.cpp:
+ (WebCore::AccessibilityNodeObject::postKeyboardKeysForValueChange):
+
+ LayoutTests:
+
+ * accessibility/keyevents-posted-for-increment-actions-expected.txt:
+ * accessibility/keyevents-posted-for-increment-actions.html:
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@271536 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2021-01-15 Chris Fleizach <cfleiz...@apple.com>
+
+ AX: increment/decrement synthetic arrow events don't work in ARIA slider examples
+ https://bugs.webkit.org/show_bug.cgi?id=220626
+ <rdar://problem/73228924>
+
+ Reviewed by Zalan Bujtas.
+
+ keyCode is still expected to be filled in with standard codes for arrow keys.
+
+ Updated test: accessibility/keyevents-posted-for-increment-actions.html
+
+ * accessibility/AccessibilityNodeObject.cpp:
+ (WebCore::AccessibilityNodeObject::postKeyboardKeysForValueChange):
+
+2021-01-25 Alan Coon <alanc...@apple.com>
+
Cherry-pick r271530. rdar://problem/73478410
CRASH in MediaSourcePrivateAVFObjC::removeSourceBuffer()
Modified: branches/safari-611-branch/Source/WebCore/accessibility/AccessibilityNodeObject.cpp (271847 => 271848)
--- branches/safari-611-branch/Source/WebCore/accessibility/AccessibilityNodeObject.cpp 2021-01-25 22:12:54 UTC (rev 271847)
+++ branches/safari-611-branch/Source/WebCore/accessibility/AccessibilityNodeObject.cpp 2021-01-25 22:13:00 UTC (rev 271848)
@@ -1134,8 +1134,10 @@
bool vertical = orientation() == AccessibilityOrientation::Vertical;
bool isLTR = page()->userInterfaceLayoutDirection() == UserInterfaceLayoutDirection::LTR;
- keyInit.key = increase ? vertical ? "ArrowUp"_s : isLTR ? "ArrowRight"_s : "ArrowLeft"_s : vertical ? "ArrowDown"_s : isLTR ? "ArrowLeft"_s : "ArrowRight"_s;
- keyInit.keyIdentifier = increase ? vertical ? "up"_s : isLTR ? "right"_s : "left"_s : vertical ? "down"_s : isLTR ? "left"_s : "right"_s;
+ typedef enum { left = 37, up = 38, right = 39, down = 40 } keyCode;
+ keyInit.key = increase ? (vertical ? "ArrowUp"_s : (isLTR ? "ArrowRight"_s : "ArrowLeft"_s)) : (vertical ? "ArrowDown"_s : (isLTR ? "ArrowLeft"_s : "ArrowRight"_s));
+ keyInit.keyCode = increase ? (vertical ? keyCode::up : (isLTR ? keyCode::right : keyCode::left)) : (vertical ? keyCode::down : (isLTR ? keyCode::left : keyCode::right));
+ keyInit.keyIdentifier = increase ? (vertical ? "up"_s : (isLTR ? "right"_s : "left"_s)) : (vertical ? "down"_s : (isLTR ? "left"_s : "right"_s));
return dispatchSimulatedKeyboardUpDownEvent(this, keyInit);
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
