<i18n dev> RFR: 8345143: Remove uses of SecurityManager in the java.desktop module [v6]

Harshitha Onkar honkar at openjdk.org
Tue Dec 3 00:26:53 UTC 2024


On Tue, 3 Dec 2024 00:12:03 GMT, Phil Race <prr at 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


More information about the i18n-dev mailing list