RFR: 8280982: [Wayland] [XWayland] java.awt.Robot taking screenshots [v11]

[Phil Race]

On Wed, 7 Jun 2023 05:05:38 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Alexander Zvegintsev has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - move screencast-tokens.properties to a new location
>>  - fix failure of javax/swing/reliability/HangDuringStaticInitialization.java in headless environment
>
> src/java.desktop/unix/classes/sun/awt/screencast/TokenStorage.java line 145:
> 
>> 143:                     path,
>> 144:                     Set.of(PosixFilePermission.OWNER_READ,
>> 145:                             PosixFilePermission.OWNER_WRITE)
> 
> alignment issue..Posix..OWNER_WRITE should be placed below Posix..OENER_READ
> 
> Also, why we aren't adding OWNER_EXECUTE here too like done above?

Because execute is required on directories but not files unless they are programs.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13803#discussion_r1220882132