On Tue, 31 Mar 2026 02:08:21 GMT, Prasanta Sadhukhan <[email protected]>
wrote:
>> test/jdk/javax/swing/SwingUtilities/TestAcceleratorMask.java line 94:
>>
>>> 92: if ((mod & (InputEvent.SHIFT_DOWN_MASK |
>>> InputEvent.SHIFT_MASK)) != 0) {
>>> 93: return KeyEvent.VK_SHIFT;
>>> 94: }
>>
>> Since SFAICS we always return VK_ALT, what was the reason for checking the
>> other masks ?
>
> getSystemMnemonicKeyMask calls getFocusAcceleratorKeyMask which in LWCToolkit
> returns CTRL_MASK|ALT_MASK
Updated test
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30366#discussion_r3026280761
