<Swing Dev> Review Request JDK-8159152 Ctrl+F6, Ctrl+F5 doesn't work for iconified InternalFrame

Rajeev Chamyal rajeev.chamyal at oracle.com
Wed Jun 22 11:46:48 UTC 2016


Hello Alexandr,

 

Thanks for the review. HYPERLINK "https://bugs.openjdk.java.net/browse/JDK-4878528"JDK-4878528 keyboard focus test was failing with current fix.

I have updated the webrev to handle keyboard focus scenario. I have executed all the regression tests for JInternalFrame all are passing.

 

http://cr.openjdk.java.net/~rchamyal/8159152/webrev.01/

 

Regards,

Rajeev Chamyal

 

From: Alexandr Scherbatiy 
Sent: 21 June 2016 16:48
To: Rajeev Chamyal; Sergey Bylokhov; swing-dev at openjdk.java.net
Subject: Re: <Swing Dev> Review Request JDK-8159152 Ctrl+F6, Ctrl+F5 doesn't work for iconified InternalFrame

 

On 6/20/2016 5:59 AM, Rajeev Chamyal wrote:



Hello Alexandr,

 

TestJInternalFrameMinimize test passes after this fix.

  Could you also check that the initial issue where the call setComponentOrderCheckingEnabled(true/false) has been added to the DefaultDesktopManager.iconifyFrame(JInternalFrame) method?
    JDK-6325652 Iconified JInternalFrame does not restore when Ctrl+F5 is used
      https://bugs.openjdk.java.net/browse/JDK-6325652

  Thanks,
  Alexandr.




 

Regards,

Rajeev Chamyal

 

From: Alexandr Scherbatiy 
Sent: 17 June 2016 18:35
To: Rajeev Chamyal; Sergey Bylokhov; HYPERLINK "mailto:swing-dev at openjdk.java.net"swing-dev at openjdk.java.net
Subject: Re: <Swing Dev> Review Request JDK-8159152 Ctrl+F6, Ctrl+F5 doesn't work for iconified InternalFrame

 

On 6/17/2016 10:09 AM, Rajeev Chamyal wrote:




Hello All,

 

Please review the following webrev.

 

Bug: https://bugs.openjdk.java.net/browse/JDK-8159152 

Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Erchamyal/8159152/webrev.00/"http://cr.openjdk.java.net/~rchamyal/8159152/webrev.00/

 

Issue: Internal frame cache is not getting updated properly on iconifyFrame if there are less than 3 frames on the desktop.

Fix: Updated the iconifyFrame method so that frame cache updates properly on internal frame remove and icon addition to desktop.


  Could you check that the test test/javax/swing/JInternalFrame/8145060/TestJInternalFrameMinimize.java passes after the fix?

  Thanks,
  Alexandr.





 

Regards,

Rajeev Chamyal

 

 

 

 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20160622/b4c11c04/attachment.html>


More information about the swing-dev mailing list