On Tue, 3 Dec 2024 00:12:03 GMT, Phil Race <p...@openjdk.org> wrote:
>> test/jdk/lib/client/ExtendedRobot.java line 64:
>>
>>> 62: // syncDelay = AccessController.doPrivileged(new
>>> GetIntegerAction("java.awt.robotdelay", DEFAULT_SYNC_DELAY));
>>> 63: //}
>>> 64:
>>
>> Not related to this PR changes but since ExtendedRobot was mentioned here, I
>> went through Robot.java and it has few SecurityException in javadoc [(as
>> here)](https://github.com/openjdk/jdk/blob/db85090553ab14a84c3ed0a2604dd56c5b6e6982/src/java.desktop/share/classes/java/awt/Robot.java#L506C1-L507C43).
>>
>> Will changes to Robot.java be handled as separate PR + CSR ?
>
>> Not related to this PR changes but since ExtendedRobot was mentioned here, I
>> went through Robot.java and it has few SecurityException in javadoc [(as
>> here)](https://github.com/openjdk/jdk/blob/db85090553ab14a84c3ed0a2604dd56c5b6e6982/src/java.desktop/share/classes/java/awt/Robot.java#L506C1-L507C43).
>> Will changes to Robot.java be handled as separate PR + CSR ?
>
> We still use SecurityException in java.awt.Robot too. It is still needed for
> when the *desktop* denies access.
Makes sense. Thanks for clarifying.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22424#discussion_r1866797014
