<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