<Swing Dev> Review request for JDK-6288609 JInternalFrame.setDefaultCloseOperation() interferes with "close" behavior

[Alexander Scherbatiy]

On 10/29/2015 10:41 AM, Rajeev Chamyal wrote:
>
> Hello All,
>
> Please review the following fix for Jdk9:
>   
> *Bug*:https://bugs.openjdk.java.net/browse/JDK-6288609  
>
> *Webrev*:http://cr.openjdk.java.net/~rchamyal/6288609/webrev.00/ 
> <http://cr.openjdk.java.net/%7Erchamyal/6288609/webrev.00/>
>
> *Issue*: On disposing the Top level JInternalFrame focus is not 
> shifting to the JInternalFrame below it.
>
> *Cause*: Dispose method is changing the selection of currently closing 
> frame and then it calls the DefaultDeskTopManager:closeFrame which
>
> finds the JInternalFrame below the closing frame based on selection of 
> the closing frame and then shifts the focus to frame below it.
>
> Since selection is already changed by dispose method 
> DefaultDeskTopManager:closeFrame is unable to find reference to 
> previous frame.
>
> *Fix*: Removed the selection change code from Dispose method.

    Are there any cases that the JInternalFrame is still selected after 
the IS_CLOSED_PROPERTY is fired in the dispose() method so it is still 
necessary to set the selection to false?

   Thanks,
   Alexandr.

>   
> Regards,
> Rajeev Chamyal