On Wed, 19 Jun 2024 08:30:42 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> On cancelling PageDialog, same PageFormat object should be returned which 
>> stopped working after 
>> [JDK-8307160](https://bugs.openjdk.org/browse/JDK-8307160).
>> Fix is made to reinstate "doIt" flag removed in JDK-8307160 so that correct 
>> value is returned from PageDialog.show action..
>> An automated printing testcase is created since the issue was caught by 
>> manual test and so having another manual test run the risk of not being 
>> executed during CI testing..
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Test fix

Have you ever noticed focus going out of dialog/frame when application starts 
in CI system? In this test, since you are directly using TAB to navigate till 
Cancel button, any changes that focus might not be on the Dialog on startup? I 
roughly remember a test where the focus was not on the Frame and we then 
setFocus or something was used......

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

PR Review: https://git.openjdk.org/jdk/pull/19786#pullrequestreview-2127529049

Reply via email to