<AWT Dev> <Swing Dev> [PATCH] 7168064: SwingUtilities.sharedOwnerFrame multiplies window close event

[Anthony Petrov]

Hi Jose,

I filed a bug for this issue:
http://bugs.sun.com/view_bug.do?bug_id=8015500

and published your patch as a webrev at:

http://cr.openjdk.java.net/~anthony/8-59-windowClosedEvents-8015500.0/

Please note that in JDK we use 4 spaces (and not TABs characters) for 
indenting lines. Could you please reformat your test and also use proper 
indentation for the two lines in Window.java that you're changing? Also, 
the copyright header in the test should be from the year 2013, not 2006. 
:) Please fix this and resend your patch.

Other than these minor suggestions, the proposed fix looks good to me.

Note that we need to get at least one more review from a reviewer on 
this mailing list before we can push this fix to a repository.

--
best regards,
Anthony

On 05/28/2013 03:35 PM, Jose Luis Martin wrote:
> Hi Anthony,
>
> Please, review the patch and test.
>
> Best Regards,
>
> -- Jose Luis Martin.
>
>
>
>
> El mié, 22-05-2013 a las 13:52 +0400, Anthony Petrov escribió:
>>
>>
>> If you and other AWT folks agree with that, could you prepare a patch
>> for this change and a jtreg-based regression test (it should be put
>> somewhere in test/java/awt/Window/, I think)?
>>
>> --
>> best regards,
>> Anthony
>>
>>
>