<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