RFR: 8307779: Relax the java.awt.Robot specification

Kevin Rushforth kcr at openjdk.org
Wed May 10 13:09:18 UTC 2023


On Tue, 9 May 2023 20:14:56 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:

>>> I do not understand why we need this method.
>> 
>> We need it in order to be able to revoke the saved permission.
>> 
>> For example, it may be necessary if the user has two monitors, but the user only allowed access to one of them.
>> If he later decides to get a screen capture of two monitors, with the current implementation it will be impossible to do so without this method.
>> 
>> Alternatively, we can replace it with a system property that says should we keep the permission or not.
>> It is discussed here https://github.com/openjdk/jdk/pull/13803#issuecomment-1540255849
>> 
>>> it needs a new bug ID.
>> 
>> done
>
> Updated according to your comments.
> I still left `revokeScreenCapturePermission` here for now, so that its documentation can be easily compared and aligned with the other documentation changes. 
> After finalization it will be moved to #13803 (if we decide to keep the method)

I agree with Phil (and Sergey, who mentioned the same earlier). I'm not sure I see enough motivation for this method to justify adding it. It seems more like a workaround for an implementation quirk of Wayland to me.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/13809#discussion_r1189882003



More information about the client-libs-dev mailing list