RFR: 8306464: [testbug] CustomSecurityManagerTest full screen tests fails on MacOS intermittently

[Kevin Rushforth]

On Thu, 20 Apr 2023 14:32:58 GMT, Karthik P K <kpk at openjdk.org> wrote:

>> tests/system/src/test/java/test/robot/helloworld/CustomSecurityManagerTest.java line 208:
>> 
>>> 206: 
>>> 207:                 for (int col = 0; col < 2; col++) {
>>> 208:                     int x = col == 0 ? 1 : screenWidth.get() - 5;
>> 
>> For `col == 0` you might want to change the value from `1` to `4` for the same reason you changed the second value.
>
> If static final variable with a suitable value is added as mentioned above, same variable can be used here.

Taking my previous comment to the other query into account, I think this should be more like:


                    int x = col == 0 ? screenBound.minX + 4 : screenBounds.maxX - 5;


where `screenBounds` is the visual bounds of the screen.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1103#discussion_r1172766029