RFR: 8307779: Relax the java.awt.Robot specification [v2]

[Alexander Zvegintsev]

On Wed, 10 May 2023 19:20:48 GMT, Phil Race <prr at openjdk.org> wrote:

>> Alexander Zvegintsev has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - rephrase wording about mouse pointer may not visually move
>>  - Remove mention of mouse cursor for screen capture
>
> src/java.desktop/share/classes/java/awt/Robot.java line 229:
> 
>> 227:     /**
>> 228:      * Moves mouse pointer to given screen coordinates.
>> 229:      * @implNote the mouse pointer may not visually move on some platforms,
> 
> This doesn't seem to be an implNote in the way that the JEP
> intends since it has nothing to do with anything we did in our implementation. I think it would have to be just spec, in which case I am not sure we should call out Linux+Wayland
> 
> See  https://openjdk.org/jeps/8068562
> 
> "Implementation Notes. This section contains informative notes about the implementation, such as advice to implementors, or performance characteristics that are specific to the implementation in this class of this version of the JDK. The information in this section is subject to change from release to release. These characteristics are also allowed to vary across platforms, vendors, and versions."

Updated.

> src/java.desktop/share/classes/java/awt/Robot.java line 466:
> 
>> 464:      * The {@link #revokeScreenCapturePermission()} can be used to revoke
>> 465:      * a previously granted permission.
>> 466:      * <p>
> 
> I would like to completely separate any decision on adding revokeScreenCapturePermission from
> this spec. relaxation.
> So we should remove the reference to it here into a separate PR about adding that API although I'm still not convinced we should add it.

Removed from this PR

> src/java.desktop/share/classes/java/awt/Robot.java line 478:
> 
>> 476:      * @throws  SecurityException if {@code readDisplayPixels} permission
>> 477:      *          is not granted, or user has not allowed any of his screens
>> 478:      *          to be captured.
> 
> "user" -> "the user"
> "his" -> gender neutral "their".
> Or maybe better ", or access to the screen is denied by the desktop environment"

Updated to the former for now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13809#discussion_r1190370654
PR Review Comment: https://git.openjdk.org/jdk/pull/13809#discussion_r1190370807
PR Review Comment: https://git.openjdk.org/jdk/pull/13809#discussion_r1190374866