RFR: 8294535 : Add screen capture functionality to PassFailJFrame [v3]

[lawrence.andrews]

On Wed, 9 Aug 2023 21:02:37 GMT, Harshitha Onkar <honkar at openjdk.org> wrote:

>> lawrence.andrews has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Changed enableScreenCapture method name to screenCapture shorter method name
>>  - Update full name
>
> test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 581:
> 
>> 579:         private int rows;
>> 580:         private int columns;
>> 581:         private boolean enableScreenCapture = false;
> 
> @lawrence-andrew  you missed updating var `enableScreenCapture` to `screenCapture` here. You can either update only the method name to screenCapture() and retain same var name (`enableScreenCapture`) or update two places where they are being used.

Its better to keep method name and var name separately.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14094#discussion_r1289349317