From serb at openjdk.java.net Sun Jan 3 02:24:57 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Sun, 3 Jan 2021 02:24:57 GMT Subject: RFR: 8258924: javax/swing/JSplitPane/4201995/bug4201995.java fails in GTk L&F [v2] In-Reply-To: References: Message-ID: On Thu, 31 Dec 2020 07:57:10 GMT, Prasanta Sadhukhan wrote: >> This test tests whether SplitPane is opaque or not but in nimbus and subseqeuently in GTK L&F the opaque property is set to false [1] >> As nimbus is already handled in the test, it is modified to handle GTK L&F too. Along with this modification, the test is also updated to test all installed L&Fs. >> Mach5 job running for several iterations for all platform is ok. Link in JBS. >> >> [1] https://raw.githubusercontent.com/openjdk/jdk/master/src/java.desktop/share/classes/javax/swing/plaf/nimbus/skin.laf >> uiComponent opaque="false" type="javax.swing.JSplitPane" name="SplitPane" ui="SplitPaneUI" subregion="false" > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Continue to next L&F if current one is unsupported Marked as reviewed by serb (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/1888 From psadhukhan at openjdk.java.net Mon Jan 4 04:35:56 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 4 Jan 2021 04:35:56 GMT Subject: Integrated: 8258924: javax/swing/JSplitPane/4201995/bug4201995.java fails in GTk L&F In-Reply-To: References: Message-ID: On Thu, 24 Dec 2020 07:33:52 GMT, Prasanta Sadhukhan wrote: > This test tests whether SplitPane is opaque or not but in nimbus and subseqeuently in GTK L&F the opaque property is set to false [1] > As nimbus is already handled in the test, it is modified to handle GTK L&F too. Along with this modification, the test is also updated to test all installed L&Fs. > Mach5 job running for several iterations for all platform is ok. Link in JBS. > > [1] https://raw.githubusercontent.com/openjdk/jdk/master/src/java.desktop/share/classes/javax/swing/plaf/nimbus/skin.laf > uiComponent opaque="false" type="javax.swing.JSplitPane" name="SplitPane" ui="SplitPaneUI" subregion="false" This pull request has now been integrated. Changeset: a2a3f4a3 Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/a2a3f4a3 Stats: 26 lines in 2 files changed: 16 ins; 1 del; 9 mod 8258924: javax/swing/JSplitPane/4201995/bug4201995.java fails in GTk L&F Reviewed-by: serb ------------- PR: https://git.openjdk.java.net/jdk/pull/1888 From psadhukhan at openjdk.java.net Mon Jan 4 04:39:57 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 4 Jan 2021 04:39:57 GMT Subject: Withdrawn: 8251377: [macos11] JTabbedPane selected tab text is barely legible In-Reply-To: References: Message-ID: On Thu, 12 Nov 2020 09:58:36 GMT, Prasanta Sadhukhan wrote: > On macOS 11 (bigsur) beta, using the Swing Aqua Look and Feel, the text of the selected JTabbedPane tab title text is just a light gray outline of white text on a white background. The macOS 11 design inverted from dark background / light text to light background / dark text, > Proposed fix is to use dark color in light background so I used lightGray instead of "white" which is being used now, if we are on BigSur. This pull request has been closed without being integrated. ------------- PR: https://git.openjdk.java.net/jdk/pull/1182 From psadhukhan at openjdk.java.net Mon Jan 4 05:14:04 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 4 Jan 2021 05:14:04 GMT Subject: RFR: 8193942: Regression automated test '/open/test/jdk/javax/swing/JFrame/8175301/ScaledFrameBackgroundTest.java' fails Message-ID: This test use to fail in macos in mach5 nightly testing long time back. It used to fail in ubuntu17.10 due to unsupported wayland display mode. Recent run of this test on supported platforms does not fail. Modified the test slightly to be consistent with other test regarding robot delays, moved frame to center of screen. Mach5 job running several iterations of the test in all platforms is ok. Link in JBS. ------------- Commit messages: - 8193942: Regression automated test '/open/test/jdk/javax/swing/JFrame/8175301/ScaledFrameBackgroundTest.java' fails Changes: https://git.openjdk.java.net/jdk/pull/1923/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1923&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8193942 Stats: 5 lines in 2 files changed: 1 ins; 1 del; 3 mod Patch: https://git.openjdk.java.net/jdk/pull/1923.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1923/head:pull/1923 PR: https://git.openjdk.java.net/jdk/pull/1923 From psadhukhan at openjdk.java.net Mon Jan 4 06:15:07 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 4 Jan 2021 06:15:07 GMT Subject: RFR: 8196466: javax/swing/JFileChooser/8062561/bug8062561.java fails Message-ID: This test was marked for "windows only" in https://github.com/openjdk/jdk/commit/7cb9e5821e0e5bcc483bc7c587ea921e9b515e77 in JDK-8198333 but the test is redundantly problem listed for linux and mac. We can remove the test from ProblemList as it is being run in mach5 on windows without any issue and it will not run on linux,mac as 8062561 fix for which this test was created is windows specific. ------------- Commit messages: - 8196466: javax/swing/JFileChooser/8062561/bug8062561.java fails Changes: https://git.openjdk.java.net/jdk/pull/1924/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1924&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8196466 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/1924.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1924/head:pull/1924 PR: https://git.openjdk.java.net/jdk/pull/1924 From serb at openjdk.java.net Mon Jan 4 06:30:53 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Mon, 4 Jan 2021 06:30:53 GMT Subject: RFR: 8196466: javax/swing/JFileChooser/8062561/bug8062561.java fails In-Reply-To: References: Message-ID: On Mon, 4 Jan 2021 06:06:50 GMT, Prasanta Sadhukhan wrote: > This test was marked for "windows only" in https://github.com/openjdk/jdk/commit/7cb9e5821e0e5bcc483bc7c587ea921e9b515e77 in JDK-8198333 > but the test is redundantly problem listed for linux and mac. > We can remove the test from ProblemList as it is being run in mach5 on windows without any issue and it will not run on linux,mac as 8062561 fix for which this test was created is windows specific. Marked as reviewed by serb (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/1924 From psadhukhan at openjdk.java.net Mon Jan 4 06:38:57 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 4 Jan 2021 06:38:57 GMT Subject: Integrated: 8196466: javax/swing/JFileChooser/8062561/bug8062561.java fails In-Reply-To: References: Message-ID: On Mon, 4 Jan 2021 06:06:50 GMT, Prasanta Sadhukhan wrote: > This test was marked for "windows only" in https://github.com/openjdk/jdk/commit/7cb9e5821e0e5bcc483bc7c587ea921e9b515e77 in JDK-8198333 > but the test is redundantly problem listed for linux and mac. > We can remove the test from ProblemList as it is being run in mach5 on windows without any issue and it will not run on linux,mac as 8062561 fix for which this test was created is windows specific. This pull request has now been integrated. Changeset: d679caa2 Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/d679caa2 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod 8196466: javax/swing/JFileChooser/8062561/bug8062561.java fails Reviewed-by: serb ------------- PR: https://git.openjdk.java.net/jdk/pull/1924 From serb at openjdk.java.net Mon Jan 4 06:43:54 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Mon, 4 Jan 2021 06:43:54 GMT Subject: RFR: 8193942: Regression automated test '/open/test/jdk/javax/swing/JFrame/8175301/ScaledFrameBackgroundTest.java' fails In-Reply-To: References: Message-ID: On Mon, 4 Jan 2021 05:09:55 GMT, Prasanta Sadhukhan wrote: > This test use to fail in macos in mach5 nightly testing long time back. It used to fail in ubuntu17.10 due to unsupported wayland display mode. Recent run of this test on supported platforms does not fail. > Modified the test slightly to be consistent with other test regarding robot delays, moved frame to center of screen. > Mach5 job running several iterations of the test in all platforms is ok. Link in JBS. Please check that the test will work if the Windows uses 125% , see https://bugs.openjdk.java.net/browse/JDK-8193942?focusedCommentId=14150509&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-14150509 ------------- PR: https://git.openjdk.java.net/jdk/pull/1923 From psadhukhan at openjdk.java.net Mon Jan 4 07:01:56 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 4 Jan 2021 07:01:56 GMT Subject: RFR: 8193942: Regression automated test '/open/test/jdk/javax/swing/JFrame/8175301/ScaledFrameBackgroundTest.java' fails In-Reply-To: References: Message-ID: On Mon, 4 Jan 2021 06:40:42 GMT, Sergey Bylokhov wrote: >> This test use to fail in macos in mach5 nightly testing long time back. It used to fail in ubuntu17.10 due to unsupported wayland display mode. Recent run of this test on supported platforms does not fail. >> Modified the test slightly to be consistent with other test regarding robot delays, moved frame to center of screen. >> Mach5 job running several iterations of the test in all platforms is ok. Link in JBS. > > Please check that the test will work if the Windows uses 125% , see https://bugs.openjdk.java.net/browse/JDK-8193942?focusedCommentId=14150509&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-14150509 Yes, it works reason: User specified action: run main/othervm -Dsun.java2d.uiScale=1.25 -Dsun.java2d.d3d=true ScaledFrameBackgroundTest Mode: othervm [/othervm specified] Additional options from @modules: --add-modules java.desktop elapsed time (seconds): 6.049 ----------configuration:(3/43)---------- Boot Layer add modules: java.desktop ----------System.out:(0/0)---------- ----------System.err:(1/16)---------- STATUS:Passed. ------------- PR: https://git.openjdk.java.net/jdk/pull/1923 From github.com+70650887+skodanda at openjdk.java.net Mon Jan 4 09:15:14 2021 From: github.com+70650887+skodanda at openjdk.java.net (K Suman Rajkumaar) Date: Mon, 4 Jan 2021 09:15:14 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v2] In-Reply-To: References: Message-ID: > Hi All, Could you please review this fix for JDK16? > > Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. > > Fix: Rewritten the above applet based test to a regular java test. > > Best Regards, > K Suman Rajkumaar K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: Update bug8031573.java Corrected the jtreg tag from @run main bug8031573 to @run main/manual bug8031573 to make this test as a manual test. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1878/files - new: https://git.openjdk.java.net/jdk/pull/1878/files/9dbc094d..5f1ab6ea Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/1878.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1878/head:pull/1878 PR: https://git.openjdk.java.net/jdk/pull/1878 From kizune at openjdk.java.net Mon Jan 4 10:12:06 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Mon, 4 Jan 2021 10:12:06 GMT Subject: RFR: 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" Message-ID: <-y0HLftBY_vzkjcq-mZjJpVoGRzwt9AT3naP7_nZItQ=.31305f63-b5e4-4a7e-ad01-998828f629a1@github.com> 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" ------------- Commit messages: - 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" - Merge pull request #7 from openjdk/master - Merge pull request #6 from openjdk/master - Merge pull request #5 from openjdk/master - Merge pull request #4 from openjdk/master - Merge pull request #3 from openjdk/master - Merge pull request #2 from openjdk/master - Merge pull request #1 from openjdk/master Changes: https://git.openjdk.java.net/jdk/pull/1929/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1929&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8258643 Stats: 6 lines in 1 file changed: 1 ins; 0 del; 5 mod Patch: https://git.openjdk.java.net/jdk/pull/1929.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1929/head:pull/1929 PR: https://git.openjdk.java.net/jdk/pull/1929 From psadhukhan at openjdk.java.net Mon Jan 4 10:34:56 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 4 Jan 2021 10:34:56 GMT Subject: RFR: 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" In-Reply-To: <-y0HLftBY_vzkjcq-mZjJpVoGRzwt9AT3naP7_nZItQ=.31305f63-b5e4-4a7e-ad01-998828f629a1@github.com> References: <-y0HLftBY_vzkjcq-mZjJpVoGRzwt9AT3naP7_nZItQ=.31305f63-b5e4-4a7e-ad01-998828f629a1@github.com> Message-ID: <9daNtfQdavO1sgoceBdTa_vmkmwqv2ZgwWZs5MTb3l0=.9644c009-e539-469e-adc6-e0c499c6300e@github.com> On Mon, 4 Jan 2021 10:07:08 GMT, Alexander Zuev wrote: > 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" test/jdk/javax/swing/JComponent/7154030/bug7154030.java line 105: > 103: locy = bounds.y + insets.top; > 104: frw = bounds.width - insets.left - insets.right; > 105: frh = bounds.height - insets.top - insets.bottom; If the intent is to do away with window decorations, can't we just call frame.setUndecorated instead of this snippet? Also, it will be better to post a mach5 job in JBS running several iterations as it fails in mach5 primarily. ------------- PR: https://git.openjdk.java.net/jdk/pull/1929 From serb at openjdk.java.net Mon Jan 4 11:24:59 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Mon, 4 Jan 2021 11:24:59 GMT Subject: RFR: 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" In-Reply-To: <9daNtfQdavO1sgoceBdTa_vmkmwqv2ZgwWZs5MTb3l0=.9644c009-e539-469e-adc6-e0c499c6300e@github.com> References: <-y0HLftBY_vzkjcq-mZjJpVoGRzwt9AT3naP7_nZItQ=.31305f63-b5e4-4a7e-ad01-998828f629a1@github.com> <9daNtfQdavO1sgoceBdTa_vmkmwqv2ZgwWZs5MTb3l0=.9644c009-e539-469e-adc6-e0c499c6300e@github.com> Message-ID: On Mon, 4 Jan 2021 10:32:30 GMT, Prasanta Sadhukhan wrote: >> 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" > > test/jdk/javax/swing/JComponent/7154030/bug7154030.java line 105: > >> 103: locy = bounds.y + insets.top; >> 104: frw = bounds.width - insets.left - insets.right; >> 105: frh = bounds.height - insets.top - insets.bottom; > > If the intent is to do away with window decorations, can't we just call frame.setUndecorated instead of this snippet? > Also, it will be better to post a mach5 job in JBS running several iterations as it fails in mach5 primarily. Or even check the location of the button only? ------------- PR: https://git.openjdk.java.net/jdk/pull/1929 From kizune at openjdk.java.net Mon Jan 4 20:14:11 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Mon, 4 Jan 2021 20:14:11 GMT Subject: RFR: 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" [v2] In-Reply-To: <-y0HLftBY_vzkjcq-mZjJpVoGRzwt9AT3naP7_nZItQ=.31305f63-b5e4-4a7e-ad01-998828f629a1@github.com> References: <-y0HLftBY_vzkjcq-mZjJpVoGRzwt9AT3naP7_nZItQ=.31305f63-b5e4-4a7e-ad01-998828f629a1@github.com> Message-ID: > 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" Alexander Zuev has updated the pull request incrementally with one additional commit since the last revision: Imports fixed. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1929/files - new: https://git.openjdk.java.net/jdk/pull/1929/files/94f6f859..1a8a0a19 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1929&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1929&range=00-01 Stats: 6 lines in 1 file changed: 0 ins; 5 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/1929.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1929/head:pull/1929 PR: https://git.openjdk.java.net/jdk/pull/1929 From kizune at openjdk.java.net Mon Jan 4 20:30:55 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Mon, 4 Jan 2021 20:30:55 GMT Subject: RFR: 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" [v2] In-Reply-To: References: <-y0HLftBY_vzkjcq-mZjJpVoGRzwt9AT3naP7_nZItQ=.31305f63-b5e4-4a7e-ad01-998828f629a1@github.com> <9daNtfQdavO1sgoceBdTa_vmkmwqv2ZgwWZs5MTb3l0=.9644c009-e539-469e-adc6-e0c499c6300e@github.com> Message-ID: On Mon, 4 Jan 2021 11:22:37 GMT, Sergey Bylokhov wrote: >> test/jdk/javax/swing/JComponent/7154030/bug7154030.java line 105: >> >>> 103: locy = bounds.y + insets.top; >>> 104: frw = bounds.width - insets.left - insets.right; >>> 105: frh = bounds.height - insets.top - insets.bottom; >> >> If the intent is to do away with window decorations, can't we just call frame.setUndecorated instead of this snippet? >> Also, it will be better to post a mach5 job in JBS running several iterations as it fails in mach5 primarily. > > Or even check the location of the button only? This is the simple fix and i don't want to make the analysis are too small because this will make artifacts harder to analyze. Test runs are linked in the bug comment. ------------- PR: https://git.openjdk.java.net/jdk/pull/1929 From prr at openjdk.java.net Mon Jan 4 21:11:58 2021 From: prr at openjdk.java.net (Phil Race) Date: Mon, 4 Jan 2021 21:11:58 GMT Subject: Integrated: 8257809: JNI warnings from Toolkit JPEG image decoding In-Reply-To: References: Message-ID: On Sun, 20 Dec 2020 01:37:47 GMT, Phil Race wrote: > The fix is to reverse the order of acquisition to get dst before src so that the call to GetArrayLength() comes first. > This also necessitates moving the RELEASE_ARRAYS() call on an error condition to the new "2nd" block. > > The new regression test passes on all platforms and all the other headless tests passed too. > So do the automated headful tests. This pull request has now been integrated. Changeset: e6f99260 Author: Phil Race URL: https://git.openjdk.java.net/jdk/commit/e6f99260 Stats: 126 lines in 4 files changed: 116 ins; 10 del; 0 mod 8257809: JNI warnings from Toolkit JPEG image decoding Reviewed-by: serb ------------- PR: https://git.openjdk.java.net/jdk/pull/1850 From serb at openjdk.java.net Mon Jan 4 23:03:53 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Mon, 4 Jan 2021 23:03:53 GMT Subject: RFR: 8193942: Regression automated test '/open/test/jdk/javax/swing/JFrame/8175301/ScaledFrameBackgroundTest.java' fails In-Reply-To: References: Message-ID: On Mon, 4 Jan 2021 06:59:10 GMT, Prasanta Sadhukhan wrote: > reason: User specified action: run main/othervm -Dsun.java2d.uiScale=1.25 -Dsun.java2d.d3d=true You have set it via the property, but the bug was reported on the system where the 125% dpi was set in the windows settings and the test was run as-is. ------------- PR: https://git.openjdk.java.net/jdk/pull/1923 From psadhukhan at openjdk.java.net Tue Jan 5 04:15:54 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Tue, 5 Jan 2021 04:15:54 GMT Subject: RFR: 8193942: Regression automated test '/open/test/jdk/javax/swing/JFrame/8175301/ScaledFrameBackgroundTest.java' fails In-Reply-To: References: Message-ID: On Mon, 4 Jan 2021 23:01:25 GMT, Sergey Bylokhov wrote: >> Yes, it works >> >> reason: User specified action: run main/othervm -Dsun.java2d.uiScale=1.25 -Dsun.java2d.d3d=true ScaledFrameBackgroundTest >> Mode: othervm [/othervm specified] >> Additional options from @modules: --add-modules java.desktop >> elapsed time (seconds): 6.049 >> ----------configuration:(3/43)---------- >> Boot Layer >> add modules: java.desktop >> >> ----------System.out:(0/0)---------- >> ----------System.err:(1/16)---------- >> STATUS:Passed. > >> reason: User specified action: run main/othervm -Dsun.java2d.uiScale=1.25 -Dsun.java2d.d3d=true > > You have set it via the property, but the bug was reported on the system where the 125% dpi was set in the windows settings and the test was run as-is. I have tried that as well. In fact, my windows display setting is 125% and it worked without this property change in uiscale. ------------- PR: https://git.openjdk.java.net/jdk/pull/1923 From serb at openjdk.java.net Tue Jan 5 06:50:54 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Tue, 5 Jan 2021 06:50:54 GMT Subject: RFR: 8193942: Regression automated test '/open/test/jdk/javax/swing/JFrame/8175301/ScaledFrameBackgroundTest.java' fails In-Reply-To: References: Message-ID: On Mon, 4 Jan 2021 05:09:55 GMT, Prasanta Sadhukhan wrote: > This test use to fail in macos in mach5 nightly testing long time back. It used to fail in ubuntu17.10 due to unsupported wayland display mode. Recent run of this test on supported platforms does not fail. > Modified the test slightly to be consistent with other test regarding robot delays, moved frame to center of screen. > Mach5 job running several iterations of the test in all platforms is ok. Link in JBS. Thank you for your confirmation. Looks fine. ------------- Marked as reviewed by serb (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/1923 From aivanov at openjdk.java.net Tue Jan 5 08:38:01 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 5 Jan 2021 08:38:01 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test In-Reply-To: References: Message-ID: <5Kct1QSD0z1McCmq3_CxPbjHNpFJZ56L2o-1Pw9CdDc=.f2aab69c-27ad-45ad-8660-71b30d76237a@github.com> On Wed, 23 Dec 2020 11:50:41 GMT, K Suman Rajkumaar wrote: > Hi All, Could you please review this fix for JDK16? > > Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. > > Fix: Rewritten the above applet based test to a regular java test. > > Best Regards, > K Suman Rajkumaar You should remove the HTML which hosts the applet. ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From aivanov at openjdk.java.net Tue Jan 5 08:38:01 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 5 Jan 2021 08:38:01 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v2] In-Reply-To: References: Message-ID: On Mon, 4 Jan 2021 09:15:14 GMT, K Suman Rajkumaar wrote: >> Hi All, Could you please review this fix for JDK16? >> >> Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. >> >> Fix: Rewritten the above applet based test to a regular java test. >> >> Best Regards, >> K Suman Rajkumaar > > K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: > > Update bug8031573.java > > Corrected the jtreg tag from @run main bug8031573 to @run main/manual bug8031573 to make this test as a manual test. Changes requested by aivanov (Reviewer). test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 60: > 58: + "If the display does not support HiDPI mode press PASS.\n" > 59: + "1. Run the test on HiDPI Display.\n" > 60: + "2. Press the Menu in the applet.\n" There's no applet. Shall we say ?Open the menu? instead? test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 61: > 59: + "1. Run the test on HiDPI Display.\n" > 60: + "2. Press the Menu in the applet.\n" > 61: + "3. Check that the icon on the JCheckBoxMenuItem is smooth If so, press PASS, else press FAIL.\n"; Suggestion: + "3. Check that the icon on the JCheckBoxMenuItem is smooth.\n" + " If so, press PASS, otherwise press FAIL."; test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 68: > 66: > 67: if (!latch.await(5, TimeUnit.MINUTES)) { > 68: frame.dispose(); The `dispose()` method should be called on EDT. test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 78: > 76: private static void createTestGUI() { > 77: frame = new JFrame("bug8031573"); > 78: frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); I propose to omit this line: the test should never call `System.exit()`. test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 105: > 103: latch.countDown(); > 104: frame.dispose(); > 105: throw new RuntimeException("Test Failed!"); Throwing the exception on EDT is redundant and does not make the test fail; moreover you throw the exception in `main()` if `passed` is `false`. test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 118: > 116: public void windowClosing(WindowEvent e) { > 117: latch.countDown(); > 118: } Add here `frame.dispose();` and then the frame will close without `frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);` and the app will exit even when launched as a stand-alone app. test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 52: > 50: public class bug8031573 { > 51: > 52: private static JFrame frame; `frame` should also be `volatile` as it's accessed from multiple threads: main thread and EDT. ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From turbanoff at gmail.com Tue Jan 5 11:40:27 2021 From: turbanoff at gmail.com (Andrey Turbanov) Date: Tue, 5 Jan 2021 14:40:27 +0300 Subject: Suspicious field assignment in StrikeMetrics Message-ID: Hello. I've found suspicious code in constructor sun.font.StrikeMetrics#StrikeMetrics() StrikeMetrics() { ascentX = ascentY = Integer.MAX_VALUE; descentX = descentY = leadingX = leadingY = Integer.MIN_VALUE; baselineX = baselineX = maxAdvanceX = maxAdvanceY = Integer.MIN_VALUE; } As you can see, basLineX field is assigned twice. Looks like it's supposed to be like this: baselineX = baselineY = maxAdvanceX = maxAdvanceY = Integer.MIN_VALUE; Found by SpotBugs https://spotbugs.readthedocs.io/en/stable/bugDescriptions.html#sa-double-assignment-of-field-sa-field-double-assignment Andrey Turbanov From vladislav.volodin at sap.com Tue Jan 5 12:14:33 2021 From: vladislav.volodin at sap.com (Volodin, Vladislav) Date: Tue, 5 Jan 2021 12:14:33 +0000 Subject: Suspicious field assignment in StrikeMetrics In-Reply-To: References: Message-ID: Hello Andrey, Nice catch! You can create a bug and prepare a pull request, so maintainers can review it. What I have just personally noticed, all fields have the type float, so probably instead of "Integer.MIN_VALUE", I would recommend to use "-Float.MAX_VALUE". But since this code is about 13-years old, I don't know if reviewers can accept your PR without additional tests. Kind regards, Vlad -----Original Message----- From: swing-dev On Behalf Of Andrey Turbanov Sent: Dienstag, 5. Januar 2021 12:40 To: swing-dev at openjdk.java.net Subject: Suspicious field assignment in StrikeMetrics Hello. I've found suspicious code in constructor sun.font.StrikeMetrics#StrikeMetrics() StrikeMetrics() { ascentX = ascentY = Integer.MAX_VALUE; descentX = descentY = leadingX = leadingY = Integer.MIN_VALUE; baselineX = baselineX = maxAdvanceX = maxAdvanceY = Integer.MIN_VALUE; } As you can see, basLineX field is assigned twice. Looks like it's supposed to be like this: baselineX = baselineY = maxAdvanceX = maxAdvanceY = Integer.MIN_VALUE; Found by SpotBugs https://spotbugs.readthedocs.io/en/stable/bugDescriptions.html#sa-double-assignment-of-field-sa-field-double-assignment Andrey Turbanov From psadhukhan at openjdk.java.net Tue Jan 5 12:59:59 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Tue, 5 Jan 2021 12:59:59 GMT Subject: RFR: 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" [v2] In-Reply-To: References: <-y0HLftBY_vzkjcq-mZjJpVoGRzwt9AT3naP7_nZItQ=.31305f63-b5e4-4a7e-ad01-998828f629a1@github.com> Message-ID: On Mon, 4 Jan 2021 20:14:11 GMT, Alexander Zuev wrote: >> 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" > > Alexander Zuev has updated the pull request incrementally with one additional commit since the last revision: > > Imports fixed. Marked as reviewed by psadhukhan (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/1929 From turbanoff at gmail.com Tue Jan 5 14:23:05 2021 From: turbanoff at gmail.com (Andrey Turbanov) Date: Tue, 5 Jan 2021 17:23:05 +0300 Subject: Suspicious field assignment in StrikeMetrics In-Reply-To: References: Message-ID: I don't have a JBS account yet, so I can't create a bug. I hope someone, who read it can create it. Andrey Turbanov ??, 5 ???. 2021 ?. ? 15:14, Volodin, Vladislav : > > Hello Andrey, > > Nice catch! You can create a bug and prepare a pull request, so maintainers can review it. > What I have just personally noticed, all fields have the type float, so probably instead of "Integer.MIN_VALUE", I would recommend to use "-Float.MAX_VALUE". But since this code is about 13-years old, I don't know if reviewers can accept your PR without additional tests. > > Kind regards, > Vlad > > -----Original Message----- > From: swing-dev On Behalf Of Andrey Turbanov > Sent: Dienstag, 5. Januar 2021 12:40 > To: swing-dev at openjdk.java.net > Subject: Suspicious field assignment in StrikeMetrics > > Hello. > I've found suspicious code in constructor sun.font.StrikeMetrics#StrikeMetrics() > > StrikeMetrics() { > ascentX = ascentY = Integer.MAX_VALUE; > descentX = descentY = leadingX = leadingY = Integer.MIN_VALUE; > baselineX = baselineX = maxAdvanceX = maxAdvanceY = Integer.MIN_VALUE; > } > > As you can see, basLineX field is assigned twice. > Looks like it's supposed to be like this: > > baselineX = baselineY = maxAdvanceX = maxAdvanceY = Integer.MIN_VALUE; > > > > Found by SpotBugs > https://spotbugs.readthedocs.io/en/stable/bugDescriptions.html#sa-double-assignment-of-field-sa-field-double-assignment > > Andrey Turbanov From kizune at openjdk.java.net Tue Jan 5 14:48:55 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Tue, 5 Jan 2021 14:48:55 GMT Subject: Integrated: 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" In-Reply-To: <-y0HLftBY_vzkjcq-mZjJpVoGRzwt9AT3naP7_nZItQ=.31305f63-b5e4-4a7e-ad01-998828f629a1@github.com> References: <-y0HLftBY_vzkjcq-mZjJpVoGRzwt9AT3naP7_nZItQ=.31305f63-b5e4-4a7e-ad01-998828f629a1@github.com> Message-ID: On Mon, 4 Jan 2021 10:07:08 GMT, Alexander Zuev wrote: > 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" This pull request has now been integrated. Changeset: fc3b45c0 Author: Alexander Zuev URL: https://git.openjdk.java.net/jdk/commit/fc3b45c0 Stats: 12 lines in 1 file changed: 1 ins; 5 del; 6 mod 8258643: javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" Reviewed-by: psadhukhan ------------- PR: https://git.openjdk.java.net/jdk/pull/1929 From trebari at openjdk.java.net Wed Jan 6 05:51:00 2021 From: trebari at openjdk.java.net (Tejpal Rebari) Date: Wed, 6 Jan 2021 05:51:00 GMT Subject: RFR: 8048109: JToggleButton does not fire actionPerformed under certain conditions [v4] In-Reply-To: References: <-cE9eTaeK1Tb0uVNh4ICINADEJ7ca-22xQBAPcwoyAE=.42736629-8faa-471f-91c2-6d61cf913182@github.com> Message-ID: <25o4tfqvGxXD1sCck6teaaGO8iuWtOVY2YUZ-yi-8yM=.030e72ee-29a8-46c6-b4e1-18d70e8fa6b8@github.com> On Wed, 23 Dec 2020 23:47:29 GMT, Sergey Bylokhov wrote: >> yeah , i have checked in windows and ubuntu native apps and it is matching native behaviour. >> The issue is seen in Motif, Nimbus, GTK and windows which sets consumeEventOnClose to true. >> It is not seen in Metal And Aqua which doesn't set this variable and the value from BasicLookAndFeel.java is used which is false. > > Can you recheck that using the next usecase: > - Create menu in the frame > - Add jbutton to the frame exactly in the place where the menu popup is opened > - Open menu and click some menu item, will the jbutton be clicked? Open menu and click some menu item, will the jbutton be clicked? No, the JButton is not clicked in this use case. ------------- PR: https://git.openjdk.java.net/jdk/pull/600 From psadhukhan at openjdk.java.net Wed Jan 6 06:48:54 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 6 Jan 2021 06:48:54 GMT Subject: Integrated: 8193942: Regression automated test '/open/test/jdk/javax/swing/JFrame/8175301/ScaledFrameBackgroundTest.java' fails In-Reply-To: References: Message-ID: <865CDi3f3d59xRliHjmZtEmfsCYcFX9XViYP5hPT5EA=.f366a48a-a985-44c3-86ca-baba38ecef36@github.com> On Mon, 4 Jan 2021 05:09:55 GMT, Prasanta Sadhukhan wrote: > This test use to fail in macos in mach5 nightly testing long time back. It used to fail in ubuntu17.10 due to unsupported wayland display mode. Recent run of this test on supported platforms does not fail. > Modified the test slightly to be consistent with other test regarding robot delays, moved frame to center of screen. > Mach5 job running several iterations of the test in all platforms is ok. Link in JBS. This pull request has now been integrated. Changeset: 32538b5b Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/32538b5b Stats: 5 lines in 2 files changed: 1 ins; 1 del; 3 mod 8193942: Regression automated test '/open/test/jdk/javax/swing/JFrame/8175301/ScaledFrameBackgroundTest.java' fails Reviewed-by: serb ------------- PR: https://git.openjdk.java.net/jdk/pull/1923 From psadhukhan at openjdk.java.net Wed Jan 6 07:24:09 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 6 Jan 2021 07:24:09 GMT Subject: RFR: 8233555: [TESTBUG] JRadioButton tests failing on MacoS Message-ID: These radiobutton tests were failing on macos in nightly testing. However recent run of these test are working fine on local mac10.14.6, 10.15.7 systems as well as mach5 macos systems, so we can deproblemlist these tests. Mach5 job link is in JBS. ------------- Commit messages: - 8233555: [TESTBUG] JRadioButton tests failing on MacoS Changes: https://git.openjdk.java.net/jdk/pull/1957/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1957&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8233555 Stats: 3 lines in 1 file changed: 0 ins; 3 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/1957.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1957/head:pull/1957 PR: https://git.openjdk.java.net/jdk/pull/1957 From github.com+70650887+skodanda at openjdk.java.net Wed Jan 6 08:22:12 2021 From: github.com+70650887+skodanda at openjdk.java.net (K Suman Rajkumaar) Date: Wed, 6 Jan 2021 08:22:12 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v3] In-Reply-To: References: Message-ID: > Hi All, Could you please review this fix for JDK16? > > Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. > > Fix: Rewritten the above applet based test to a regular java test. > > Best Regards, > K Suman Rajkumaar K Suman Rajkumaar has updated the pull request incrementally with three additional commits since the last revision: - Update bug8031573.java - Update bug8031573.java - Incorporated the review comments from Alexey. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1878/files - new: https://git.openjdk.java.net/jdk/pull/1878/files/5f1ab6ea..9020f7bf Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=01-02 Stats: 8 lines in 1 file changed: 2 ins; 2 del; 4 mod Patch: https://git.openjdk.java.net/jdk/pull/1878.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1878/head:pull/1878 PR: https://git.openjdk.java.net/jdk/pull/1878 From github.com+70650887+skodanda at openjdk.java.net Wed Jan 6 08:26:56 2021 From: github.com+70650887+skodanda at openjdk.java.net (K Suman Rajkumaar) Date: Wed, 6 Jan 2021 08:26:56 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v2] In-Reply-To: References: Message-ID: On Tue, 5 Jan 2021 08:25:00 GMT, Alexey Ivanov wrote: >> K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: >> >> Update bug8031573.java >> >> Corrected the jtreg tag from @run main bug8031573 to @run main/manual bug8031573 to make this test as a manual test. > > test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 68: > >> 66: >> 67: if (!latch.await(5, TimeUnit.MINUTES)) { >> 68: frame.dispose(); > > The `dispose()` method should be called on EDT. If I remove the frame.dispose(), the frame doesn't gets disposed even after timeout. ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From serb at openjdk.java.net Wed Jan 6 08:47:54 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Wed, 6 Jan 2021 08:47:54 GMT Subject: RFR: 8233555: [TESTBUG] JRadioButton tests failing on MacoS In-Reply-To: References: Message-ID: On Wed, 6 Jan 2021 07:18:46 GMT, Prasanta Sadhukhan wrote: > These radiobutton tests were failing on macos in nightly testing. However recent run of these test are working fine on local mac10.14.6, 10.15.7 systems as well as mach5 macos systems, so we can deproblemlist these tests. Mach5 job link is in JBS. At least two of those tests were failed due to JDK-8226892. If they work fine now does it mean that some other fixes resolved the problem? ------------- PR: https://git.openjdk.java.net/jdk/pull/1957 From psadhukhan at openjdk.java.net Wed Jan 6 08:54:54 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 6 Jan 2021 08:54:54 GMT Subject: RFR: 8233555: [TESTBUG] JRadioButton tests failing on MacoS In-Reply-To: References: Message-ID: <1Lj05iOcMpRAPysvAfXJbMQ2C4xu0fr0w3mrwSyLOls=.aad57d05-ca88-48fe-8eef-c0552dc98059@github.com> On Wed, 6 Jan 2021 08:45:19 GMT, Sergey Bylokhov wrote: >> These radiobutton tests were failing on macos in nightly testing. However recent run of these test are working fine on local mac10.14.6, 10.15.7 systems as well as mach5 macos systems, so we can deproblemlist these tests. Mach5 job link is in JBS. > > At least two of those tests were failed due to JDK-8226892. If they work fine now does it mean that some other fixes resolved the problem? Not sure but the code added by JDK-8226892 is reworked by JDK-8249548 and we dont see the issue now as can be seen by mach5 run. ------------- PR: https://git.openjdk.java.net/jdk/pull/1957 From serb at openjdk.java.net Wed Jan 6 09:00:56 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Wed, 6 Jan 2021 09:00:56 GMT Subject: RFR: 8233555: [TESTBUG] JRadioButton tests failing on MacoS In-Reply-To: <1Lj05iOcMpRAPysvAfXJbMQ2C4xu0fr0w3mrwSyLOls=.aad57d05-ca88-48fe-8eef-c0552dc98059@github.com> References: <1Lj05iOcMpRAPysvAfXJbMQ2C4xu0fr0w3mrwSyLOls=.aad57d05-ca88-48fe-8eef-c0552dc98059@github.com> Message-ID: On Wed, 6 Jan 2021 08:51:44 GMT, Prasanta Sadhukhan wrote: > Not sure but the code added by JDK-8226892 is reworked by JDK-8249548 and we dont see the issue now as can be seen by mach5 run. So the tests will fail on the builds before jdk16 b14? ------------- PR: https://git.openjdk.java.net/jdk/pull/1957 From psadhukhan at openjdk.java.net Wed Jan 6 09:22:54 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 6 Jan 2021 09:22:54 GMT Subject: RFR: 8233555: [TESTBUG] JRadioButton tests failing on MacoS In-Reply-To: References: <1Lj05iOcMpRAPysvAfXJbMQ2C4xu0fr0w3mrwSyLOls=.aad57d05-ca88-48fe-8eef-c0552dc98059@github.com> Message-ID: On Wed, 6 Jan 2021 08:57:42 GMT, Sergey Bylokhov wrote: >> Not sure but the code added by JDK-8226892 is reworked by JDK-8249548 and we dont see the issue now as can be seen by mach5 run. > >> Not sure but the code added by JDK-8226892 is reworked by JDK-8249548 and we dont see the issue now as can be seen by mach5 run. > > So the tests will fail on the builds before jdk16 b14? Yes, maybe not all but bug8033699 fails before 16b14 ------------- PR: https://git.openjdk.java.net/jdk/pull/1957 From pbansal at openjdk.java.net Wed Jan 6 09:22:55 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Wed, 6 Jan 2021 09:22:55 GMT Subject: RFR: 8233555: [TESTBUG] JRadioButton tests failing on MacoS In-Reply-To: References: <1Lj05iOcMpRAPysvAfXJbMQ2C4xu0fr0w3mrwSyLOls=.aad57d05-ca88-48fe-8eef-c0552dc98059@github.com> Message-ID: On Wed, 6 Jan 2021 09:17:50 GMT, Prasanta Sadhukhan wrote: >>> Not sure but the code added by JDK-8226892 is reworked by JDK-8249548 and we dont see the issue now as can be seen by mach5 run. >> >> So the tests will fail on the builds before jdk16 b14? > > Yes, maybe not all but bug8033699 fails before 16b14 The JDK-8249548 may be backed out soon as it has caused some other issue JDK-8259237. I am working on it. ------------- PR: https://git.openjdk.java.net/jdk/pull/1957 From github.com+70650887+skodanda at openjdk.java.net Wed Jan 6 09:46:10 2021 From: github.com+70650887+skodanda at openjdk.java.net (K Suman Rajkumaar) Date: Wed, 6 Jan 2021 09:46:10 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v4] In-Reply-To: References: Message-ID: <2ERHBWQCI41fV0E3qwGRQvz7ktEoDN7oaNCbJvYjzik=.b7ec58f0-5cdf-4a24-b828-919fc492b91f@github.com> > Hi All, Could you please review this fix for JDK16? > > Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. > > Fix: Rewritten the above applet based test to a regular java test. > > Best Regards, > K Suman Rajkumaar K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: Deleted the bug8031573.html ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1878/files - new: https://git.openjdk.java.net/jdk/pull/1878/files/9020f7bf..0e675a02 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=03 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=02-03 Stats: 40 lines in 1 file changed: 0 ins; 40 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/1878.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1878/head:pull/1878 PR: https://git.openjdk.java.net/jdk/pull/1878 From trebari at openjdk.java.net Wed Jan 6 12:48:03 2021 From: trebari at openjdk.java.net (Tejpal Rebari) Date: Wed, 6 Jan 2021 12:48:03 GMT Subject: RFR: 8049700: Deprecate obsolete classes and methods in javax/swing/plaf/basic Message-ID: Please review the following fix for jdk17. In this fix i have deprecated and marked for removal following classes and methods public void intervalAdded(ListDataEvent e) public void intervalRemoved(ListDataEvent e) protected boolean lt(File a, File b) in BasicDirectoryModel.java inner class PropertyChangeHandler, VSBChangeListener, HSBChangeListener, ViewportChangeHandler in BasicScrollPaneUI.java inner class MouseInputHandler in BasicMenuItemUI.java method BasicToolBarUI.java#createFloatingFrame >From 8049700 not deprecated the paintText(Graphics g, JComponent c, Rectangle textRect, String text) method in BasicButtonUI as AquaButtonUI, MetalButtonUI and MetalToggleButtonUI overrides it. Similarly not deprecated ChangeHandler of BasicMenuUI as AquaMenuUI and MotifMenuUI uses this class. ------------- Commit messages: - initial fix Changes: https://git.openjdk.java.net/jdk/pull/1958/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1958&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8049700 Stats: 38 lines in 4 files changed: 38 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/1958.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1958/head:pull/1958 PR: https://git.openjdk.java.net/jdk/pull/1958 From aivanov at openjdk.java.net Wed Jan 6 13:03:57 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 6 Jan 2021 13:03:57 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v4] In-Reply-To: <2ERHBWQCI41fV0E3qwGRQvz7ktEoDN7oaNCbJvYjzik=.b7ec58f0-5cdf-4a24-b828-919fc492b91f@github.com> References: <2ERHBWQCI41fV0E3qwGRQvz7ktEoDN7oaNCbJvYjzik=.b7ec58f0-5cdf-4a24-b828-919fc492b91f@github.com> Message-ID: On Wed, 6 Jan 2021 09:46:10 GMT, K Suman Rajkumaar wrote: >> Hi All, Could you please review this fix for JDK16? >> >> Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. >> >> Fix: Rewritten the above applet based test to a regular java test. >> >> Best Regards, >> K Suman Rajkumaar > > K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: > > Deleted the bug8031573.html Changes requested by aivanov (Reviewer). test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 120: > 118: } > 119: }); > 120: frame.setSize(760, 250); Doesn't `pack()` work? test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 121: > 119: }); > 120: frame.setSize(760, 250); > 121: frame.setLocation(0, 250); Shall the frame rather be centred on the screen by using `setLocationRelativeTo(null)`? ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From aivanov at openjdk.java.net Wed Jan 6 13:03:58 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 6 Jan 2021 13:03:58 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v2] In-Reply-To: References: Message-ID: <8ozS5FWXgm7GTKAvvMLVTZTF-j51w_7rIkYwqQaENOE=.71272433-32ee-42a9-adef-5036c12ec03d@github.com> On Wed, 6 Jan 2021 08:24:04 GMT, K Suman Rajkumaar wrote: >> test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 68: >> >>> 66: >>> 67: if (!latch.await(5, TimeUnit.MINUTES)) { >>> 68: frame.dispose(); >> >> The `dispose()` method should be called on EDT. > > If I remove the frame.dispose(), the frame doesn't gets disposed even after timeout. You're right, in my previous comment I forgot about timeout. Yet it should still be called on EDT. However, to clean up the code a bit, I propose calling `dispose()` in finally block. Thus there'll be only one place where frame is disposed of instead of being scattered in multiple event handlers: try { // Create GUI // Check conditions and throw exceptions on failure } finally { SwingUtilities.invokeAndWait(() -> { if (frame != null) { frame.dispose(); } } } In this case, frame will be accessed from one thread only, EDT, and does not need to be volatile. Since the frame is always disposed of, other calls to `frame.dispose()` can be removed. ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From github.com+70650887+skodanda at openjdk.java.net Wed Jan 6 14:28:29 2021 From: github.com+70650887+skodanda at openjdk.java.net (K Suman Rajkumaar) Date: Wed, 6 Jan 2021 14:28:29 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v5] In-Reply-To: References: Message-ID: > Hi All, Could you please review this fix for JDK16? > > Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. > > Fix: Rewritten the above applet based test to a regular java test. > > Best Regards, > K Suman Rajkumaar K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: Updated with another set of review comments from Alexey Updated with another set of review comments from Alexey. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1878/files - new: https://git.openjdk.java.net/jdk/pull/1878/files/0e675a02..72c48ff5 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=04 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=03-04 Stats: 19 lines in 1 file changed: 8 ins; 3 del; 8 mod Patch: https://git.openjdk.java.net/jdk/pull/1878.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1878/head:pull/1878 PR: https://git.openjdk.java.net/jdk/pull/1878 From aivanov at openjdk.java.net Wed Jan 6 15:36:58 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 6 Jan 2021 15:36:58 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v5] In-Reply-To: References: Message-ID: On Wed, 6 Jan 2021 14:28:29 GMT, K Suman Rajkumaar wrote: >> Hi All, Could you please review this fix for JDK16? >> >> Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. >> >> Fix: Rewritten the above applet based test to a regular java test. >> >> Best Regards, >> K Suman Rajkumaar > > K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: > > Updated with another set of review comments from Alexey > > Updated with another set of review comments from Alexey. Changes requested by aivanov (Reviewer). test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 125: > 123: }); > 124: frame.setSize(760, 250); > 125: frame.pack(); `setSize` is redundant when you call `pack`. The latter will also set size of the frame based on the preferred size determined by the layout manager. ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From github.com+70650887+skodanda at openjdk.java.net Wed Jan 6 16:25:16 2021 From: github.com+70650887+skodanda at openjdk.java.net (K Suman Rajkumaar) Date: Wed, 6 Jan 2021 16:25:16 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v6] In-Reply-To: References: Message-ID: > Hi All, Could you please review this fix for JDK16? > > Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. > > Fix: Rewritten the above applet based test to a regular java test. > > Best Regards, > K Suman Rajkumaar K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: Removed the setSize() ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1878/files - new: https://git.openjdk.java.net/jdk/pull/1878/files/72c48ff5..af8de106 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=05 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=04-05 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/1878.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1878/head:pull/1878 PR: https://git.openjdk.java.net/jdk/pull/1878 From aivanov at openjdk.java.net Wed Jan 6 19:54:56 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 6 Jan 2021 19:54:56 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v6] In-Reply-To: References: Message-ID: On Wed, 6 Jan 2021 16:25:16 GMT, K Suman Rajkumaar wrote: >> Hi All, Could you please review this fix for JDK16? >> >> Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. >> >> Fix: Rewritten the above applet based test to a regular java test. >> >> Best Regards, >> K Suman Rajkumaar > > K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: > > Removed the setSize() Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From prr at openjdk.java.net Wed Jan 6 20:46:00 2021 From: prr at openjdk.java.net (Phil Race) Date: Wed, 6 Jan 2021 20:46:00 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v6] In-Reply-To: References: Message-ID: On Wed, 6 Jan 2021 19:52:35 GMT, Alexey Ivanov wrote: >> K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: >> >> Removed the setSize() > > Marked as reviewed by aivanov (Reviewer). "Hi All, Could you please review this fix for JDK16?" This is for JDK 17, not JDK 16, right ? I can't imagine why it would be for 16 at this juncture .. ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From prr at openjdk.java.net Wed Jan 6 20:46:01 2021 From: prr at openjdk.java.net (Phil Race) Date: Wed, 6 Jan 2021 20:46:01 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v6] In-Reply-To: References: Message-ID: On Wed, 6 Jan 2021 16:25:16 GMT, K Suman Rajkumaar wrote: >> Hi All, Could you please review this fix for JDK16? >> >> Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. >> >> Fix: Rewritten the above applet based test to a regular java test. >> >> Best Regards, >> K Suman Rajkumaar > > K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: > > Removed the setSize() test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java line 58: > 56: public static final String INSTRUCTIONS = "INSTRUCTIONS:\n\n" > 57: + "Verify that high resolution system icons are used JCheckBoxMenuItem on HiDPI displays.\n" > 58: + "If the display does not support HiDPI mode press PASS.\n" "used" -> "used for" ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From serb at openjdk.java.net Thu Jan 7 00:25:00 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Thu, 7 Jan 2021 00:25:00 GMT Subject: RFR: 8233555: [TESTBUG] JRadioButton tests failing on MacoS In-Reply-To: References: Message-ID: On Wed, 6 Jan 2021 07:18:46 GMT, Prasanta Sadhukhan wrote: > These radiobutton tests were failing on macos in nightly testing. However recent run of these test are working fine on local mac10.14.6, 10.15.7 systems as well as mach5 macos systems, so we can deproblemlist these tests. Mach5 job link is in JBS. Marked as reviewed by serb (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/1957 From serb at openjdk.java.net Thu Jan 7 00:25:01 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Thu, 7 Jan 2021 00:25:01 GMT Subject: RFR: 8233555: [TESTBUG] JRadioButton tests failing on MacoS In-Reply-To: References: <1Lj05iOcMpRAPysvAfXJbMQ2C4xu0fr0w3mrwSyLOls=.aad57d05-ca88-48fe-8eef-c0552dc98059@github.com> Message-ID: On Wed, 6 Jan 2021 09:20:38 GMT, Pankaj Bansal wrote: > The JDK-8249548 may be backed out soon as it has caused some other issue JDK-8259237. I am working on it. So please take care of these tests, so we do not break them again. ------------- PR: https://git.openjdk.java.net/jdk/pull/1957 From psadhukhan at openjdk.java.net Thu Jan 7 04:01:57 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Thu, 7 Jan 2021 04:01:57 GMT Subject: Integrated: 8233555: [TESTBUG] JRadioButton tests failing on MacoS In-Reply-To: References: Message-ID: On Wed, 6 Jan 2021 07:18:46 GMT, Prasanta Sadhukhan wrote: > These radiobutton tests were failing on macos in nightly testing. However recent run of these test are working fine on local mac10.14.6, 10.15.7 systems as well as mach5 macos systems, so we can deproblemlist these tests. Mach5 job link is in JBS. This pull request has now been integrated. Changeset: 227f99d3 Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/227f99d3 Stats: 3 lines in 1 file changed: 0 ins; 3 del; 0 mod 8233555: [TESTBUG] JRadioButton tests failing on MacoS Reviewed-by: serb ------------- PR: https://git.openjdk.java.net/jdk/pull/1957 From github.com+70650887+skodanda at openjdk.java.net Thu Jan 7 05:54:07 2021 From: github.com+70650887+skodanda at openjdk.java.net (K Suman Rajkumaar) Date: Thu, 7 Jan 2021 05:54:07 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v7] In-Reply-To: References: Message-ID: > Hi All, Could you please review this fix for JDK16? > > Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. > > Fix: Rewritten the above applet based test to a regular java test. > > Best Regards, > K Suman Rajkumaar K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: Changed "used" to "used for" as per Phil's review comment. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1878/files - new: https://git.openjdk.java.net/jdk/pull/1878/files/af8de106..b51bf3bd Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=06 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1878&range=05-06 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/1878.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1878/head:pull/1878 PR: https://git.openjdk.java.net/jdk/pull/1878 From aivanov at openjdk.java.net Thu Jan 7 15:56:57 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Thu, 7 Jan 2021 15:56:57 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v6] In-Reply-To: References: Message-ID: On Wed, 6 Jan 2021 20:42:55 GMT, Phil Race wrote: > "Hi All, Could you please review this fix for JDK16?" > This is for JDK 17, not JDK 16, right ? I can't imagine why it would be for 16 at this juncture .. No, it's not meant for JDK 16 ? it's for jdk-dev, the main development line which is JDK 17 at this very moment. ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From psadhukhan at openjdk.java.net Fri Jan 8 11:08:05 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 8 Jan 2021 11:08:05 GMT Subject: RFR: 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 Message-ID: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> This test was failing on mach5 nightly on ubuntu systems long time back. Modified the test to add some delays and call robot.waitForIdle() to make it more stable and the resultant test passes on all mach5 systems including linux consisting of ubuntu 19.10, 18.04,20.10,20.04 so we can remove the test from problemlist. mach5 job running for several iterations is posted in JBS. ------------- Commit messages: - 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 Changes: https://git.openjdk.java.net/jdk/pull/2003/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2003&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8225045 Stats: 13 lines in 2 files changed: 11 ins; 1 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/2003.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2003/head:pull/2003 PR: https://git.openjdk.java.net/jdk/pull/2003 From serb at openjdk.java.net Sat Jan 9 00:19:05 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Sat, 9 Jan 2021 00:19:05 GMT Subject: RFR: 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 In-Reply-To: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> References: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> Message-ID: On Fri, 8 Jan 2021 11:03:36 GMT, Prasanta Sadhukhan wrote: > This test was failing on mach5 nightly on ubuntu systems long time back. Modified the test to add some delays and call robot.waitForIdle() to make it more stable and the resultant test passes on all mach5 systems including linux consisting of ubuntu 19.10, 18.04,20.10,20.04 so we can remove the test from problemlist. > mach5 job running for several iterations is posted in JBS. test/jdk/javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java line 66: > 64: public static void main(String[] args) throws Exception { > 65: robot = new Robot(); > 66: robot.setAutoDelay(100); Why do you need an autodeley here? It seems the test does not use any of mouseXXX/keyXXX methods. ------------- PR: https://git.openjdk.java.net/jdk/pull/2003 From pbansal at openjdk.java.net Sat Jan 9 12:53:11 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Sat, 9 Jan 2021 12:53:11 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. Message-ID: Please review a fix for jdk16 Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. Before fix for JDK-8249548 JToggleButton: For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. JRadioButton: For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" After fix for JDK-8249548 JToggleButton: For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". JRadioButton: For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". After current fix: JToggleButton: For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. JRadioButton: For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. ------------- Commit messages: - 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. Changes: https://git.openjdk.java.net/jdk16/pull/99/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8259237 Stats: 375 lines in 4 files changed: 350 ins; 20 del; 5 mod Patch: https://git.openjdk.java.net/jdk16/pull/99.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/99/head:pull/99 PR: https://git.openjdk.java.net/jdk16/pull/99 From serb at openjdk.java.net Sun Jan 10 02:58:00 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Sun, 10 Jan 2021 02:58:00 GMT Subject: RFR: 8048109: JToggleButton does not fire actionPerformed under certain conditions [v4] In-Reply-To: References: Message-ID: On Thu, 10 Dec 2020 16:06:00 GMT, Tejpal Rebari wrote: >> Please review the following fix for jdk16. >> >> Issue : There is a JToggleButton that will post/take down a JPopupMenu when the button is selected. If the button is selected and the menu is not posted the action listener will post the menu. If the button is selected and the menu is displayed the action listener will take the menu down. The use case is: >> 1 - select button >> 2 - menu posted >> 3 - select button >> 4 - menu taken down >> >> With MetalLookAndFeel the above use case works fine, but with WindowsLookAndFeel the second button selection does not fire the actionPerformed event, button needs to be selected third time for the menu to be taken down. >> >> The issue is that the button must be selected twice after the menu is posted to have the actionPerformed event to fire when using the Windows look and feel. >> >> Fix : MouseGrabber is not removed while uninstalling the listeners in the BasicPopupMenuUI. >> By removing the mouseGrabber in the uninstallListeners() methods fixes this issue. >> >> Added a test to test the same in all the LookAndFeels > > Tejpal Rebari has updated the pull request incrementally with one additional commit since the last revision: > > changed mode of files OK, let's see how it will work. ------------- Marked as reviewed by serb (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/600 From trebari at openjdk.java.net Sun Jan 10 12:42:02 2021 From: trebari at openjdk.java.net (Tejpal Rebari) Date: Sun, 10 Jan 2021 12:42:02 GMT Subject: Integrated: 8048109: JToggleButton does not fire actionPerformed under certain conditions In-Reply-To: References: Message-ID: On Mon, 12 Oct 2020 08:01:43 GMT, Tejpal Rebari wrote: > Please review the following fix for jdk16. > > Issue : There is a JToggleButton that will post/take down a JPopupMenu when the button is selected. If the button is selected and the menu is not posted the action listener will post the menu. If the button is selected and the menu is displayed the action listener will take the menu down. The use case is: > 1 - select button > 2 - menu posted > 3 - select button > 4 - menu taken down > > With MetalLookAndFeel the above use case works fine, but with WindowsLookAndFeel the second button selection does not fire the actionPerformed event, button needs to be selected third time for the menu to be taken down. > > The issue is that the button must be selected twice after the menu is posted to have the actionPerformed event to fire when using the Windows look and feel. > > Fix : MouseGrabber is not removed while uninstalling the listeners in the BasicPopupMenuUI. > By removing the mouseGrabber in the uninstallListeners() methods fixes this issue. > > Added a test to test the same in all the LookAndFeels This pull request has now been integrated. Changeset: 65ca5c66 Author: Tejpal Rebari URL: https://git.openjdk.java.net/jdk/commit/65ca5c66 Stats: 169 lines in 5 files changed: 165 ins; 0 del; 4 mod 8048109: JToggleButton does not fire actionPerformed under certain conditions Reviewed-by: serb, psadhukhan, vdyakov ------------- PR: https://git.openjdk.java.net/jdk/pull/600 From herrick at openjdk.java.net Mon Jan 11 01:56:56 2021 From: herrick at openjdk.java.net (Andy Herrick) Date: Mon, 11 Jan 2021 01:56:56 GMT Subject: Withdrawn: JDK-8189198: Add "forRemoval = true" to Applet APIs In-Reply-To: References: Message-ID: On Mon, 9 Nov 2020 13:56:33 GMT, Andy Herrick wrote: > JDK-8189198: Add "forRemoval = true" to Applet APIs This pull request has been closed without being integrated. ------------- PR: https://git.openjdk.java.net/jdk/pull/1127 From psadhukhan at openjdk.java.net Mon Jan 11 04:50:09 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 11 Jan 2021 04:50:09 GMT Subject: RFR: 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 [v2] In-Reply-To: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> References: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> Message-ID: > This test was failing on mach5 nightly on ubuntu systems long time back. Modified the test to add some delays and call robot.waitForIdle() to make it more stable and the resultant test passes on all mach5 systems including linux consisting of ubuntu 19.10, 18.04,20.10,20.04 so we can remove the test from problemlist. > mach5 job running for several iterations is posted in JBS. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: Remove unneeded setAutoDelay ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/2003/files - new: https://git.openjdk.java.net/jdk/pull/2003/files/9c02b54d..c9c12efc Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2003&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2003&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/2003.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2003/head:pull/2003 PR: https://git.openjdk.java.net/jdk/pull/2003 From trebari at openjdk.java.net Mon Jan 11 05:33:56 2021 From: trebari at openjdk.java.net (Tejpal Rebari) Date: Mon, 11 Jan 2021 05:33:56 GMT Subject: RFR: 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 [v2] In-Reply-To: References: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> Message-ID: <7Ias379b9ALHZBOMtmS_iJAj2zXxaGjp4s2hMj5TDS8=.4395ff0f-f56c-47b6-87e8-6651012c7ecc@github.com> On Mon, 11 Jan 2021 04:50:09 GMT, Prasanta Sadhukhan wrote: >> This test was failing on mach5 nightly on ubuntu systems long time back. Modified the test to add some delays and call robot.waitForIdle() to make it more stable and the resultant test passes on all mach5 systems including linux consisting of ubuntu 19.10, 18.04,20.10,20.04 so we can remove the test from problemlist. >> mach5 job running for several iterations is posted in JBS. > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Remove unneeded setAutoDelay test/jdk/javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java line 220: > 218: if (!bufferedImagesEqual(imageIconImage, iconImage)) { > 219: ImageIO.write(imageIconImage, "png", new File("imageicon-fail.png")); > 220: ImageIO.write(iconImage, "png", new File("iconImage-fail.png")); Is this for debugging purpose when the test fails ? ------------- PR: https://git.openjdk.java.net/jdk/pull/2003 From serb at openjdk.java.net Mon Jan 11 05:44:55 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Mon, 11 Jan 2021 05:44:55 GMT Subject: RFR: 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 [v2] In-Reply-To: References: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> Message-ID: On Mon, 11 Jan 2021 04:50:09 GMT, Prasanta Sadhukhan wrote: >> This test was failing on mach5 nightly on ubuntu systems long time back. Modified the test to add some delays and call robot.waitForIdle() to make it more stable and the resultant test passes on all mach5 systems including linux consisting of ubuntu 19.10, 18.04,20.10,20.04 so we can remove the test from problemlist. >> mach5 job running for several iterations is posted in JBS. > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Remove unneeded setAutoDelay Marked as reviewed by serb (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2003 From pbansal at openjdk.java.net Mon Jan 11 06:00:03 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Mon, 11 Jan 2021 06:00:03 GMT Subject: RFR: 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 [v2] In-Reply-To: References: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> Message-ID: On Mon, 11 Jan 2021 04:50:09 GMT, Prasanta Sadhukhan wrote: >> This test was failing on mach5 nightly on ubuntu systems long time back. Modified the test to add some delays and call robot.waitForIdle() to make it more stable and the resultant test passes on all mach5 systems including linux consisting of ubuntu 19.10, 18.04,20.10,20.04 so we can remove the test from problemlist. >> mach5 job running for several iterations is posted in JBS. > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Remove unneeded setAutoDelay Marked as reviewed by pbansal (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2003 From serb at openjdk.java.net Mon Jan 11 08:19:19 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Mon, 11 Jan 2021 08:19:19 GMT Subject: RFR: 8259522: Apply java.io.Serial annotations in java.desktop Message-ID: Please review the application of @java.io.Serial annotation (JDK-8202385) to types in the desktop module to enable stricter compile-time checking of serialization-related declarations. This annotation can be applied to these methods in the module: private void writeObject(java.io.ObjectOutputStream stream) throws IOException private void readObject(java.io.ObjectInputStream stream) throws IOException, ClassNotFoundException private void readObjectNoData() throws ObjectStreamException ANY-ACCESS-MODIFIER Object writeReplace() throws ObjectStreamException ANY-ACCESS-MODIFIER Object readResolve() throws ObjectStreamException private static final ObjectStreamField[] serialPersistentFields private static final long serialVersionUID Notes: - I have tried to update the comments for serialVersionUID as accurately as possible, but mostly based on the source code history and bugs in JBS where that field was added - Some of the readObject/writeObject methods in the javax.swing package does not have a spec, because this package and some others are excluded from the serialization specification. A similar fix was implemented for java.base module as well: http://mail.openjdk.java.net/pipermail/core-libs-dev/2019-August/062046.html ------------- Commit messages: - Update TextAttribute.java - Update readResolve - Update writeReplace - Update readObject - Update writeObject - Update serialPersistentFields - Update serialVersionUID Changes: https://git.openjdk.java.net/jdk/pull/2020/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2020&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8259522 Stats: 3426 lines in 343 files changed: 2268 ins; 287 del; 871 mod Patch: https://git.openjdk.java.net/jdk/pull/2020.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2020/head:pull/2020 PR: https://git.openjdk.java.net/jdk/pull/2020 From psadhukhan at openjdk.java.net Mon Jan 11 08:22:59 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 11 Jan 2021 08:22:59 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. In-Reply-To: References: Message-ID: On Sat, 9 Jan 2021 12:47:48 GMT, Pankaj Bansal wrote: > Please review a fix for jdk16 > > Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. > > Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. > > This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". > > Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. > > Before fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. > JRadioButton: > For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. > For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. > For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" > > After fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > JRadioButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > > After current fix: > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. > JRadioButton: > For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. > For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" > > I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. I guess you did not run jtreg job with latest problemlist as it did not run javax/swing/JRadioButton/ButtonGroupFocus/ButtonGroupFocusTest.java javax/swing/JRadioButton/8075609/bug8075609.java javax/swing/JRadioButton/8033699/bug8033699.java Please see this test pass with your change as some of them used to fail after JDK-8226892 which was reworked by fix of JDK-8249548 src/java.desktop/macosx/classes/com/apple/laf/AquaButtonUI.java line 228: > 226: if (listener != null) { > 227: b.removeMouseListener(listener); > 228: b.removeMouseListener(listener); Why are we removing same listener twice? ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Mon Jan 11 08:37:57 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Mon, 11 Jan 2021 08:37:57 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. In-Reply-To: References: Message-ID: On Mon, 11 Jan 2021 08:05:21 GMT, Prasanta Sadhukhan wrote: >> Please review a fix for jdk16 >> >> Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. >> >> Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. >> >> This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". >> >> Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. >> >> Before fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. >> JRadioButton: >> For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. >> For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. >> For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" >> >> After fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> JRadioButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> >> After current fix: >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. >> JRadioButton: >> For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. >> For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" >> >> I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. > > src/java.desktop/macosx/classes/com/apple/laf/AquaButtonUI.java line 228: > >> 226: if (listener != null) { >> 227: b.removeMouseListener(listener); >> 228: b.removeMouseListener(listener); > > Why are we removing same listener twice? I just rolled back the changes I did in this file in JDK-8249548. Looks like this was has been there for some time. See the file in commit made in this file before JDK-8249548 https://github.com/openjdk/jdk/blob/c18080fef714949221c8ddabc4e23d409575c3d3/src/java.desktop/macosx/classes/com/apple/laf/AquaButtonUI.java. But yes, it should not be done and I will remove it and update the PR. ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Mon Jan 11 09:52:18 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Mon, 11 Jan 2021 09:52:18 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v2] In-Reply-To: References: Message-ID: > Please review a fix for jdk16 > > Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. > > Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. > > This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". > > Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. > > Before fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. > JRadioButton: > For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. > For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. > For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" > > After fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > JRadioButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > > After current fix: > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. > JRadioButton: > For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. > For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" > > I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: Remove duplicate call to remove listener ------------- Changes: - all: https://git.openjdk.java.net/jdk16/pull/99/files - new: https://git.openjdk.java.net/jdk16/pull/99/files/903ffcbd..724f5999 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod Patch: https://git.openjdk.java.net/jdk16/pull/99.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/99/head:pull/99 PR: https://git.openjdk.java.net/jdk16/pull/99 From kizune at openjdk.java.net Mon Jan 11 11:19:04 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Mon, 11 Jan 2021 11:19:04 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v2] In-Reply-To: References: Message-ID: On Mon, 11 Jan 2021 08:20:39 GMT, Prasanta Sadhukhan wrote: >> Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: >> >> Remove duplicate call to remove listener > > I guess you did not run jtreg job with latest problemlist as it did not run > javax/swing/JRadioButton/ButtonGroupFocus/ButtonGroupFocusTest.java > javax/swing/JRadioButton/8075609/bug8075609.java > javax/swing/JRadioButton/8033699/bug8033699.java > Please see this test pass with your change as some of them used to fail after JDK-8226892 which was reworked by fix of JDK-8249548 Quick question: why we special casing JRadioButton case? Is there any reason we should call the action on them? ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Mon Jan 11 12:01:04 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Mon, 11 Jan 2021 12:01:04 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v2] In-Reply-To: References: Message-ID: On Mon, 11 Jan 2021 11:16:02 GMT, Alexander Zuev wrote: > Quick question: why we special casing JRadioButton case? Is there any reason we should call the action on them? This was the behaviour which was there before JDK-8249548 for JRadioButton and I am just reverting to it. As explained in the PR description, the behaviour before JDK-8249548 was as below Before fix for JDK-8249548 JToggleButton: For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. JRadioButton: For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Mon Jan 11 13:30:15 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Mon, 11 Jan 2021 13:30:15 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v3] In-Reply-To: References: Message-ID: > Please review a fix for jdk16 > > Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. > > Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. > > This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". > > Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. > > Before fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. > JRadioButton: > For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. > For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. > For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" > > After fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > JRadioButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > > After current fix: > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. > JRadioButton: > For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. > For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" > > I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: Remove duplicate code for AquaL&F ------------- Changes: - all: https://git.openjdk.java.net/jdk16/pull/99/files - new: https://git.openjdk.java.net/jdk16/pull/99/files/724f5999..de3db58a Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=01-02 Stats: 337 lines in 2 files changed: 19 ins; 317 del; 1 mod Patch: https://git.openjdk.java.net/jdk16/pull/99.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/99/head:pull/99 PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Mon Jan 11 13:40:00 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Mon, 11 Jan 2021 13:40:00 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v3] In-Reply-To: References: Message-ID: On Mon, 11 Jan 2021 11:57:49 GMT, Pankaj Bansal wrote: >> Quick question: why we special casing JRadioButton case? Is there any reason we should call the action on them? > >> Quick question: why we special casing JRadioButton case? Is there any reason we should call the action on them? > > This was the behaviour which was there before JDK-8249548 for JRadioButton and I am just reverting to it. As explained in the PR description, the behaviour before JDK-8249548 was as below > > Before fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. > JRadioButton: > For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. > For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. > For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" I have removed the duplicate code for AquaL&F. I think during the fix made for JDK-8226892, it was a mistake to not change the AquaL&F code. So, the behaviour of JRadioButton should be similar across L&Fs and the duplicate code in AquaL&F is not needed. A mach5 job with all jtreg/jck tests is being run and I will will update when it finishes. ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From kizune at openjdk.java.net Mon Jan 11 18:17:05 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Mon, 11 Jan 2021 18:17:05 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v3] In-Reply-To: References: Message-ID: On Mon, 11 Jan 2021 13:30:15 GMT, Pankaj Bansal wrote: >> Please review a fix for jdk16 >> >> Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. >> >> Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. >> >> This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". >> >> Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. >> >> Before fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. >> JRadioButton: >> For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. >> For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. >> For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" >> >> After fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> JRadioButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> >> After current fix: >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. >> JRadioButton: >> For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. >> For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" >> >> I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. > > Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: > > Remove duplicate code for AquaL&F Looks ok now. ------------- Marked as reviewed by kizune (Reviewer). PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Mon Jan 11 18:24:00 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Mon, 11 Jan 2021 18:24:00 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v3] In-Reply-To: References: Message-ID: On Mon, 11 Jan 2021 08:20:39 GMT, Prasanta Sadhukhan wrote: > I guess you did not run jtreg job with latest problemlist as it did not run > javax/swing/JRadioButton/ButtonGroupFocus/ButtonGroupFocusTest.java > javax/swing/JRadioButton/8075609/bug8075609.java > javax/swing/JRadioButton/8033699/bug8033699.java > Please see this test pass with your change as some of them used to fail after JDK-8226892 which was reworked by fix of JDK-8249548 I have updated the PR with some commits and ran the full jtreg/jck tests including the tests you have mentioned. The fix is not breaking anything. ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From psadhukhan at openjdk.java.net Tue Jan 12 04:02:05 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Tue, 12 Jan 2021 04:02:05 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v3] In-Reply-To: References: Message-ID: <1G-TYyhdBLCszc12GMj_LRKqkWbU4U8ADN3qf_NcVVY=.14d71d20-57e4-4ffa-9c79-67a522f5afd6@github.com> On Mon, 11 Jan 2021 13:30:15 GMT, Pankaj Bansal wrote: >> Please review a fix for jdk16 >> >> Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. >> >> Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. >> >> This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". >> >> Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. >> >> Before fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. >> JRadioButton: >> For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. >> For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. >> For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" >> >> After fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> JRadioButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> >> After current fix: >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. >> JRadioButton: >> For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. >> For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" >> >> I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. > > Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: > > Remove duplicate code for AquaL&F test/jdk/javax/swing/ButtonGroup/TestButtonGroupFocusTraversal.java line 32: > 30: * @run main TestButtonGroupFocusTraversal > 31: */ > 32: We need to append this bugid as product code is changed for which this testcase is modified ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Tue Jan 12 04:18:13 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Tue, 12 Jan 2021 04:18:13 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v4] In-Reply-To: References: Message-ID: > Please review a fix for jdk16 > > Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. > > Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. > > This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". > > Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. > > Before fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. > JRadioButton: > For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. > For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. > For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" > > After fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > JRadioButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > > After current fix: > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. > JRadioButton: > For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. > For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" > > I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: Add current bugid ------------- Changes: - all: https://git.openjdk.java.net/jdk16/pull/99/files - new: https://git.openjdk.java.net/jdk16/pull/99/files/de3db58a..6c745600 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=03 - incr: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=02-03 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk16/pull/99.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/99/head:pull/99 PR: https://git.openjdk.java.net/jdk16/pull/99 From psadhukhan at openjdk.java.net Tue Jan 12 04:18:13 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Tue, 12 Jan 2021 04:18:13 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v4] In-Reply-To: References: Message-ID: On Tue, 12 Jan 2021 04:15:31 GMT, Pankaj Bansal wrote: >> Please review a fix for jdk16 >> >> Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. >> >> Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. >> >> This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". >> >> Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. >> >> Before fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. >> JRadioButton: >> For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. >> For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. >> For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" >> >> After fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> JRadioButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> >> After current fix: >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. >> JRadioButton: >> For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. >> For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" >> >> I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. > > Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: > > Add current bugid Marked as reviewed by psadhukhan (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Tue Jan 12 04:18:14 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Tue, 12 Jan 2021 04:18:14 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v3] In-Reply-To: <1G-TYyhdBLCszc12GMj_LRKqkWbU4U8ADN3qf_NcVVY=.14d71d20-57e4-4ffa-9c79-67a522f5afd6@github.com> References: <1G-TYyhdBLCszc12GMj_LRKqkWbU4U8ADN3qf_NcVVY=.14d71d20-57e4-4ffa-9c79-67a522f5afd6@github.com> Message-ID: On Tue, 12 Jan 2021 03:58:44 GMT, Prasanta Sadhukhan wrote: >> Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: >> >> Remove duplicate code for AquaL&F > > test/jdk/javax/swing/ButtonGroup/TestButtonGroupFocusTraversal.java line 32: > >> 30: * @run main TestButtonGroupFocusTraversal >> 31: */ >> 32: > > We need to append this bugid as product code is changed for which this testcase is modified Done, please have a look. ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From serb at openjdk.java.net Tue Jan 12 04:28:58 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Tue, 12 Jan 2021 04:28:58 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v4] In-Reply-To: References: Message-ID: On Tue, 12 Jan 2021 04:14:52 GMT, Prasanta Sadhukhan wrote: >> Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: >> >> Add current bugid > > Marked as reviewed by psadhukhan (Reviewer). The spec of the update method seems outdated: * Find the new toggle button that focus needs to be * moved to in the group, select the button BTW I am not sure why the Aqua is so specific and requires an additional press while the same radio buttons do not require that. Is it possible that some fix was implemented in the shared code so now all L&F skip the "space"? ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Tue Jan 12 04:41:58 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Tue, 12 Jan 2021 04:41:58 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v4] In-Reply-To: References: Message-ID: On Tue, 12 Jan 2021 04:26:19 GMT, Sergey Bylokhov wrote: > The spec of the update method seems outdated: > * Find the new toggle button that focus needs to be > * moved to in the group, select the button > > BTW I am not sure why the Aqua is so specific and requires an additional press while the same radio buttons do not require that. Is it possible that some fix was implemented in the shared code so now all L&F skip the "space"? The AquaL&F also does not need the additional "space" press for JRadioButton. During the fix of JDK-8226892, code was added for other L&F, so that the additional "space" press is not needed. But same code was not added for AquaL&F. So, the additional key press was needed. Now, the additional press should not be needed for AquaL&F also for JRadioButton. So now, for all L&Fs, the additional key press is needed for JToggleButton and is not needed for JRadioButton. I did not add radio button to spec as RadioButton are also ToggleButton only. But yes, I will highlight that. So I will update the spec as below. Hope that is ok. * Find the new toggle/radio button that focus needs to be * moved to in the group, select the button ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From serb at openjdk.java.net Tue Jan 12 05:10:01 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Tue, 12 Jan 2021 05:10:01 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v4] In-Reply-To: References: Message-ID: On Tue, 12 Jan 2021 04:39:39 GMT, Pankaj Bansal wrote: > I did not add radio button to spec as RadioButton are also ToggleButton only. But yes, I will highlight that. So I will update the spec as below. Hope that is ok. > * Find the new toggle/radio button that focus needs to be > * moved to in the group, select the button Also please add a comment of why we need to use the setPressed/setArmed it seems unrelated to the selection of the button, and is not described anywhere. ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From trebari at openjdk.java.net Tue Jan 12 05:21:03 2021 From: trebari at openjdk.java.net (Tejpal Rebari) Date: Tue, 12 Jan 2021 05:21:03 GMT Subject: RFR: 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 [v2] In-Reply-To: References: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> Message-ID: <93Y3ruAIsL7iXnTq7UvXbq4uIOsU1CNlJs5MJzgR_a8=.00ba6f9d-2d12-45cb-a7a8-b30488d60f7d@github.com> On Mon, 11 Jan 2021 04:50:09 GMT, Prasanta Sadhukhan wrote: >> This test was failing on mach5 nightly on ubuntu systems long time back. Modified the test to add some delays and call robot.waitForIdle() to make it more stable and the resultant test passes on all mach5 systems including linux consisting of ubuntu 19.10, 18.04,20.10,20.04 so we can remove the test from problemlist. >> mach5 job running for several iterations is posted in JBS. > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Remove unneeded setAutoDelay Marked as reviewed by trebari (Committer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2003 From pbansal at openjdk.java.net Tue Jan 12 05:56:16 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Tue, 12 Jan 2021 05:56:16 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v5] In-Reply-To: References: Message-ID: > Please review a fix for jdk16 > > Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. > > Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. > > This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". > > Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. > > Before fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. > JRadioButton: > For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. > For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. > For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" > > After fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > JRadioButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > > After current fix: > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. > JRadioButton: > For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. > For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" > > I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: Update Spec ------------- Changes: - all: https://git.openjdk.java.net/jdk16/pull/99/files - new: https://git.openjdk.java.net/jdk16/pull/99/files/6c745600..fca7f7a4 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=04 - incr: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=03-04 Stats: 4 lines in 1 file changed: 3 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk16/pull/99.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/99/head:pull/99 PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Tue Jan 12 05:56:16 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Tue, 12 Jan 2021 05:56:16 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v4] In-Reply-To: References: Message-ID: <40WXxYHrQoinLbEVbHlp58UUv5Bdzg5x_wFptPAOibM=.2cb48050-c47b-4806-858d-705fdbdc4d31@github.com> On Tue, 12 Jan 2021 05:07:10 GMT, Sergey Bylokhov wrote: > > I did not add radio button to spec as RadioButton are also ToggleButton only. But yes, I will highlight that. So I will update the spec as below. Hope that is ok. > > > > * Find the new toggle/radio button that focus needs to be > > * moved to in the group, select the button > > Also please add a comment of why we need to use the setPressed/setArmed it seems unrelated to the selection of the button, and is not described anywhere. Done, please have a look. ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From serb at openjdk.java.net Tue Jan 12 06:24:01 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Tue, 12 Jan 2021 06:24:01 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v5] In-Reply-To: References: Message-ID: <0VUkqD1QzgxChCwQY-u3_I-bJOX6-LlrRLCxaKkDqSA=.a9ed2a08-b06a-4b3c-ae61-bd7d9a2ebd69@github.com> On Tue, 12 Jan 2021 05:56:16 GMT, Pankaj Bansal wrote: >> Please review a fix for jdk16 >> >> Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. >> >> Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. >> >> This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". >> >> Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. >> >> Before fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. >> JRadioButton: >> For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. >> For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. >> For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" >> >> After fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> JRadioButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> >> After current fix: >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. >> JRadioButton: >> For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. >> For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" >> >> I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. > > Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: > > Update Spec test/jdk/javax/swing/ButtonGroup/TestButtonGroupFocusTraversal.java line 80: > 78: toggleButton2 = new JToggleButton("2"); > 79: radioButton1 = new JRadioButton("1"); > 80: radioButton2 = new JRadioButton("2"); Just to be safe please check the JCheckBox, it also extends the JToggleButton and we should not break it. ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Tue Jan 12 07:08:16 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Tue, 12 Jan 2021 07:08:16 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v6] In-Reply-To: References: Message-ID: > Please review a fix for jdk16 > > Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. > > Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. > > This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". > > Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. > > Before fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. > JRadioButton: > For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. > For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. > For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" > > After fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > JRadioButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > > After current fix: > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. > JRadioButton: > For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. > For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" > > I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: Add checkbox in test ------------- Changes: - all: https://git.openjdk.java.net/jdk16/pull/99/files - new: https://git.openjdk.java.net/jdk16/pull/99/files/fca7f7a4..9962d615 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=05 - incr: https://webrevs.openjdk.java.net/?repo=jdk16&pr=99&range=04-05 Stats: 57 lines in 1 file changed: 57 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jdk16/pull/99.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/99/head:pull/99 PR: https://git.openjdk.java.net/jdk16/pull/99 From serb at openjdk.java.net Tue Jan 12 09:29:59 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Tue, 12 Jan 2021 09:29:59 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v6] In-Reply-To: References: Message-ID: On Tue, 12 Jan 2021 07:08:16 GMT, Pankaj Bansal wrote: >> Please review a fix for jdk16 >> >> Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. >> >> Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. >> >> This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". >> >> Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. >> >> Before fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. >> JRadioButton: >> For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. >> For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. >> For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" >> >> After fix for JDK-8249548 >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> JRadioButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". >> >> After current fix: >> JToggleButton: >> For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. >> JRadioButton: >> For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. >> For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" >> >> I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. > > Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision: > > Add checkbox in test Marked as reviewed by serb (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From serb at openjdk.java.net Tue Jan 12 09:30:00 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Tue, 12 Jan 2021 09:30:00 GMT Subject: [jdk16] RFR: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. [v4] In-Reply-To: <40WXxYHrQoinLbEVbHlp58UUv5Bdzg5x_wFptPAOibM=.2cb48050-c47b-4806-858d-705fdbdc4d31@github.com> References: <40WXxYHrQoinLbEVbHlp58UUv5Bdzg5x_wFptPAOibM=.2cb48050-c47b-4806-858d-705fdbdc4d31@github.com> Message-ID: On Tue, 12 Jan 2021 05:52:33 GMT, Pankaj Bansal wrote: >>> I did not add radio button to spec as RadioButton are also ToggleButton only. But yes, I will highlight that. So I will update the spec as below. Hope that is ok. >>> * Find the new toggle/radio button that focus needs to be >>> * moved to in the group, select the button >> >> Also please add a comment of why we need to use the setPressed/setArmed it seems unrelated to the selection of the button, and is not described anywhere. > >> > I did not add radio button to spec as RadioButton are also ToggleButton only. But yes, I will highlight that. So I will update the spec as below. Hope that is ok. >> > >> > * Find the new toggle/radio button that focus needs to be >> > * moved to in the group, select the button >> >> Also please add a comment of why we need to use the setPressed/setArmed it seems unrelated to the selection of the button, and is not described anywhere. > > Done, please have a look. Just small summary info for the future: we have a JToggleButton class and two subclasses: JCheckBox and JRadioButton. 1. It was decided(but not strictly specified) to generate an action when the selection is changed by the arrow keys for JRadioButton. It is just common sense. 2. It was decided to not generate an action when the selection is changed by arrow keys for JCheckBox. Since the user should have the ability to move over different checkboxes without toggling them. 3. The parent class JToggleButton is a root cause of the current bug under discussion. We could implement it in any way, we can generate an action when the user presses the arrow key, but the opposite just a safer solution now. ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From pbansal at openjdk.java.net Tue Jan 12 09:48:56 2021 From: pbansal at openjdk.java.net (Pankaj Bansal) Date: Tue, 12 Jan 2021 09:48:56 GMT Subject: [jdk16] Integrated: 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. In-Reply-To: References: Message-ID: On Sat, 9 Jan 2021 12:47:48 GMT, Pankaj Bansal wrote: > Please review a fix for jdk16 > > Issue: In SwingSet2, if the user navigates through the demos, the demo gets selected/starts on pressing left/right key only. There is no need to press the "space" key. Earlier, on pressing the left/right key, only demo icon used to get focused and user needed to press the "space" to actually select a demo. > > Cause: The SwingSet2 has JToggleButtons added to a ButtonGroup. Each JToggleButton has an AbstractAction set on it, which loads the demo. Earlier, when the user pressed Left/Right button, only the selected button used to change. The Action set on JToggleButton used to perform only on pressing the "Space" button. Now, the Action is performed on navigating the JToggleButtons using Left/Right keys without the need to press the "space" key. > > This issue is a side effect of fix done for https://bugs.openjdk.java.net/browse/JDK-8249548. The issues fixed in JDK-8249548 were not present in JRadioButton as there was code available to handle it in AquaButtonRadioUI and BasicRadioButtonUI. The fix done for JDK-8249548 moved duplicate code from AquaButtonRadioUI and BasicRadioButtonUI and moved it to BasicButtonUI, so this code is available for JToggleButton and JRadioButton for all L&Fs. This was wrong as there is a difference in behaviour for JRadioButtons added to ButtonGroup for AquaL&F and other L&Fs. In AquaL&F, the AbstractAction set on JRadioButton is not performed on button selection and user has to press "space". In other L&Fs, the AbstractAction is performed on selection of Button itself without the need to press "space". > > Fix: The current change fixes the issue in present bug and keeps the fixes done in JDK-8249548. Following is the behaviour of JToggleButton and JRadioButton for different L&Fs before JDK-8249548 fix, after JDK-8249548 fix and after current fix. > > Before fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can not navigate/select the buttons added to ButtonGroup properly as mentioned in the JDK-8249548. > JRadioButton: > For Synth L&F (Nimbus L&F), user can not navigate/select the buttons added to ButtonGroup. > For AquaL&F, user can select the buttons added to ButtonGroup by pressing left/right key and needs to press the "space" to perform the set Action. > For Other L&Fs, user can select/navigate the buttons and the set Action is also performed without pressing "space" > > After fix for JDK-8249548 > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > JRadioButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups and the AbstractAction set is performed without the need to press "space". > > After current fix: > JToggleButton: > For all L&Fs, user can navigate/select the buttons added to ButtonGroups. User needs to press "space" to perform the set AbstractAction. > JRadioButton: > For AquaL&F, the behaviour before JDK-8249548 is restored, so user needs to press the "space" to perform the set Action. > For all other L&Fs (including Synth L&F), the behaviour is same. User can navigate/select the buttons added to ButtonGroups and set Action is performed without pressing "space" > > I have run mach5 jobs with full jtreg/jck and all looks good. Links in JBS. The test TestButtonGroupFocusTraversal.java is modified such that it fails without current fix and passes after the fix. The fix also should be verified by running SwingSet2. This pull request has now been integrated. Changeset: 28ff2de1 Author: Pankaj Bansal URL: https://git.openjdk.java.net/jdk16/commit/28ff2de1 Stats: 99 lines in 2 files changed: 92 ins; 1 del; 6 mod 8259237: Demo selection changes with left/right arrow key. No need to press space for selection. Reviewed-by: psadhukhan, kizune, serb ------------- PR: https://git.openjdk.java.net/jdk16/pull/99 From aivanov at openjdk.java.net Tue Jan 12 20:25:56 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 12 Jan 2021 20:25:56 GMT Subject: RFR: 8259522: Apply java.io.Serial annotations in java.desktop In-Reply-To: References: Message-ID: On Mon, 11 Jan 2021 06:21:52 GMT, Sergey Bylokhov wrote: > Please review the application of @java.io.Serial annotation (JDK-8202385) to types in the desktop module to enable stricter compile-time checking of serialization-related declarations. > > This annotation can be applied to these methods in the module: > > private void writeObject(java.io.ObjectOutputStream stream) throws IOException > private void readObject(java.io.ObjectInputStream stream) throws IOException, ClassNotFoundException > private void readObjectNoData() throws ObjectStreamException > ANY-ACCESS-MODIFIER Object writeReplace() throws ObjectStreamException > ANY-ACCESS-MODIFIER Object readResolve() throws ObjectStreamException > private static final ObjectStreamField[] serialPersistentFields > private static final long serialVersionUID > > Notes: > - I have tried to update the comments for serialVersionUID as accurately as possible, but mostly based on the source code history and bugs in JBS where that field was added > - Some of the readObject/writeObject methods in the javax.swing package does not have a spec, because this package and some others are excluded from the serialization specification. > > A similar fix was implemented for java.base module as well: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2019-August/062046.html Marked as reviewed by aivanov (Reviewer). src/java.desktop/share/classes/com/sun/media/sound/InvalidDataException.java line 42: > 40: */ > 41: @Serial > 42: private static final long serialVersionUID = 1L; This is the standard wording, yet should it mention the serialization is between the same versions only because the value of serialVersionUID is not unique? src/java.desktop/share/classes/java/awt/image/RasterFormatException.java line 28: > 26: package java.awt.image; > 27: > 28: Suggestion: In the majority of classes, there's only one empty line between package declaration and the first import statement. src/java.desktop/share/classes/java/awt/image/ImagingOpException.java line 28: > 26: package java.awt.image; > 27: > 28: Suggestion: ------------- PR: https://git.openjdk.java.net/jdk/pull/2020 From serb at openjdk.java.net Tue Jan 12 20:42:18 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Tue, 12 Jan 2021 20:42:18 GMT Subject: RFR: 8259522: Apply java.io.Serial annotations in java.desktop [v2] In-Reply-To: References: Message-ID: <7X46pwNuTDGbEN3-jFa2QY2qUtMCK2dAu-BiPdoMK14=.19bc926b-62ba-4f79-8f32-be27898077a8@github.com> > Please review the application of @java.io.Serial annotation (JDK-8202385) to types in the desktop module to enable stricter compile-time checking of serialization-related declarations. > > This annotation can be applied to these methods in the module: > > private void writeObject(java.io.ObjectOutputStream stream) throws IOException > private void readObject(java.io.ObjectInputStream stream) throws IOException, ClassNotFoundException > private void readObjectNoData() throws ObjectStreamException > ANY-ACCESS-MODIFIER Object writeReplace() throws ObjectStreamException > ANY-ACCESS-MODIFIER Object readResolve() throws ObjectStreamException > private static final ObjectStreamField[] serialPersistentFields > private static final long serialVersionUID > > Notes: > - I have tried to update the comments for serialVersionUID as accurately as possible, but mostly based on the source code history and bugs in JBS where that field was added > - Some of the readObject/writeObject methods in the javax.swing package does not have a spec, because this package and some others are excluded from the serialization specification. > > A similar fix was implemented for java.base module as well: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2019-August/062046.html Sergey Bylokhov has updated the pull request incrementally with two additional commits since the last revision: - Update src/java.desktop/share/classes/java/awt/image/ImagingOpException.java Co-authored-by: Aleksei Ivanov <70774172+aivanov-jdk at users.noreply.github.com> - Update src/java.desktop/share/classes/java/awt/image/RasterFormatException.java Co-authored-by: Aleksei Ivanov <70774172+aivanov-jdk at users.noreply.github.com> ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/2020/files - new: https://git.openjdk.java.net/jdk/pull/2020/files/2e50a258..502c1a40 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2020&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2020&range=00-01 Stats: 4 lines in 2 files changed: 0 ins; 4 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/2020.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2020/head:pull/2020 PR: https://git.openjdk.java.net/jdk/pull/2020 From serb at openjdk.java.net Tue Jan 12 20:42:19 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Tue, 12 Jan 2021 20:42:19 GMT Subject: RFR: 8259522: Apply java.io.Serial annotations in java.desktop [v2] In-Reply-To: References: Message-ID: On Tue, 12 Jan 2021 11:29:24 GMT, Alexey Ivanov wrote: >> Sergey Bylokhov has updated the pull request incrementally with two additional commits since the last revision: >> >> - Update src/java.desktop/share/classes/java/awt/image/ImagingOpException.java >> >> Co-authored-by: Aleksei Ivanov <70774172+aivanov-jdk at users.noreply.github.com> >> - Update src/java.desktop/share/classes/java/awt/image/RasterFormatException.java >> >> Co-authored-by: Aleksei Ivanov <70774172+aivanov-jdk at users.noreply.github.com> > > src/java.desktop/share/classes/com/sun/media/sound/InvalidDataException.java line 42: > >> 40: */ >> 41: @Serial >> 42: private static final long serialVersionUID = 1L; > > This is the standard wording, yet should it mention the serialization is between the same versions only because the value of serialVersionUID is not unique? I think it is "quite" unique, the "1L" is used from the beginning. It is just different from the value which could be generated by the "serialver" but still should work fine. ------------- PR: https://git.openjdk.java.net/jdk/pull/2020 From aivanov at openjdk.java.net Tue Jan 12 21:58:14 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 12 Jan 2021 21:58:14 GMT Subject: RFR: 8259522: Apply java.io.Serial annotations in java.desktop [v2] In-Reply-To: References: Message-ID: On Tue, 12 Jan 2021 20:36:21 GMT, Sergey Bylokhov wrote: >> src/java.desktop/share/classes/com/sun/media/sound/InvalidDataException.java line 42: >> >>> 40: */ >>> 41: @Serial >>> 42: private static final long serialVersionUID = 1L; >> >> This is the standard wording, yet should it mention the serialization is between the same versions only because the value of serialVersionUID is not unique? > > I think it is "quite" unique, the "1L" is used from the beginning. It is just different from the value which could be generated by the "serialver" but still should work fine. Okay. There are several classes which extend this one, all of them use the same 1L value which makes it not "quite" unique. Changing the values at this time presents more risks than benefits. ------------- PR: https://git.openjdk.java.net/jdk/pull/2020 From psadhukhan at openjdk.java.net Wed Jan 13 05:56:00 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 13 Jan 2021 05:56:00 GMT Subject: RFR: 8049700: Deprecate obsolete classes and methods in javax/swing/plaf/basic In-Reply-To: References: Message-ID: On Wed, 6 Jan 2021 12:43:41 GMT, Tejpal Rebari wrote: > Please review the following fix for jdk17. > In this fix i have deprecated and marked for removal following classes and methods > public void intervalAdded(ListDataEvent e) > public void intervalRemoved(ListDataEvent e) > protected boolean lt(File a, File b) in BasicDirectoryModel.java > > inner class PropertyChangeHandler, VSBChangeListener, HSBChangeListener, > ViewportChangeHandler in BasicScrollPaneUI.java > inner class MouseInputHandler in BasicMenuItemUI.java > method BasicToolBarUI.java#createFloatingFrame > > From 8049700 not deprecated the paintText(Graphics g, JComponent c, Rectangle textRect, String text) method in BasicButtonUI as AquaButtonUI, MetalButtonUI and MetalToggleButtonUI overrides it. > Similarly not deprecated ChangeHandler of BasicMenuUI as AquaMenuUI and MotifMenuUI uses this class. Please elaborate as to why this methods are to be deprecated. It will be useful if you give alternate methods to be used in the javadoc in @deprecated tag which are supposed to be called by user once these are removed. ------------- PR: https://git.openjdk.java.net/jdk/pull/1958 From psadhukhan at openjdk.java.net Wed Jan 13 06:13:57 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 13 Jan 2021 06:13:57 GMT Subject: RFR: 8259522: Apply java.io.Serial annotations in java.desktop [v2] In-Reply-To: <7X46pwNuTDGbEN3-jFa2QY2qUtMCK2dAu-BiPdoMK14=.19bc926b-62ba-4f79-8f32-be27898077a8@github.com> References: <7X46pwNuTDGbEN3-jFa2QY2qUtMCK2dAu-BiPdoMK14=.19bc926b-62ba-4f79-8f32-be27898077a8@github.com> Message-ID: <0BeCSRlVfit-78buTvO8-tci6rZDhwsML4JXKzujLM8=.ff5ba622-5c99-4925-b690-5b348d0d0e18@github.com> On Tue, 12 Jan 2021 20:42:18 GMT, Sergey Bylokhov wrote: >> Please review the application of @java.io.Serial annotation (JDK-8202385) to types in the desktop module to enable stricter compile-time checking of serialization-related declarations. >> >> This annotation can be applied to these methods in the module: >> >> private void writeObject(java.io.ObjectOutputStream stream) throws IOException >> private void readObject(java.io.ObjectInputStream stream) throws IOException, ClassNotFoundException >> private void readObjectNoData() throws ObjectStreamException >> ANY-ACCESS-MODIFIER Object writeReplace() throws ObjectStreamException >> ANY-ACCESS-MODIFIER Object readResolve() throws ObjectStreamException >> private static final ObjectStreamField[] serialPersistentFields >> private static final long serialVersionUID >> >> Notes: >> - I have tried to update the comments for serialVersionUID as accurately as possible, but mostly based on the source code history and bugs in JBS where that field was added >> - Some of the readObject/writeObject methods in the javax.swing package does not have a spec, because this package and some others are excluded from the serialization specification. >> >> A similar fix was implemented for java.base module as well: >> http://mail.openjdk.java.net/pipermail/core-libs-dev/2019-August/062046.html > > Sergey Bylokhov has updated the pull request incrementally with two additional commits since the last revision: > > - Update src/java.desktop/share/classes/java/awt/image/ImagingOpException.java > > Co-authored-by: Aleksei Ivanov <70774172+aivanov-jdk at users.noreply.github.com> > - Update src/java.desktop/share/classes/java/awt/image/RasterFormatException.java > > Co-authored-by: Aleksei Ivanov <70774172+aivanov-jdk at users.noreply.github.com> Why do we add serialVersionUID in some classes like DefaultMutableTreeNode.java but not in other swing classes? Also, if this change is for stricter compile-time checking, shouldn't we remove @SuppressWarnings("serial") check? ------------- PR: https://git.openjdk.java.net/jdk/pull/2020 From psadhukhan at openjdk.java.net Wed Jan 13 07:01:01 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 13 Jan 2021 07:01:01 GMT Subject: Integrated: 8256019: JLabel HTML text does not support translucent text colors In-Reply-To: References: Message-ID: On Wed, 11 Nov 2020 08:00:09 GMT, Prasanta Sadhukhan wrote: > Issue is a JLabel with a translucent foreground color properly renders plain text, but with HTML text, the alpha component is discarded and the text is rendered using an opaque color. > As per https://www.w3schools.com/cssref/func_rgba.asp, CSS supports rgba() to support alpha and render translucent text color > but support for rgba() is not present in JDK html text rendering. > > Added support for rgba() to render translucent text color. This pull request has now been integrated. Changeset: 44c83794 Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/44c83794 Stats: 196 lines in 2 files changed: 190 ins; 0 del; 6 mod 8256019: JLabel HTML text does not support translucent text colors Reviewed-by: serb ------------- PR: https://git.openjdk.java.net/jdk/pull/1158 From psadhukhan at openjdk.java.net Wed Jan 13 07:03:56 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 13 Jan 2021 07:03:56 GMT Subject: RFR: 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 [v2] In-Reply-To: <7Ias379b9ALHZBOMtmS_iJAj2zXxaGjp4s2hMj5TDS8=.4395ff0f-f56c-47b6-87e8-6651012c7ecc@github.com> References: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> <7Ias379b9ALHZBOMtmS_iJAj2zXxaGjp4s2hMj5TDS8=.4395ff0f-f56c-47b6-87e8-6651012c7ecc@github.com> Message-ID: On Mon, 11 Jan 2021 05:31:15 GMT, Tejpal Rebari wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: >> >> Remove unneeded setAutoDelay > > test/jdk/javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java line 220: > >> 218: if (!bufferedImagesEqual(imageIconImage, iconImage)) { >> 219: ImageIO.write(imageIconImage, "png", new File("imageicon-fail.png")); >> 220: ImageIO.write(iconImage, "png", new File("iconImage-fail.png")); > > Is this for debugging purpose when the test fails ? yes ------------- PR: https://git.openjdk.java.net/jdk/pull/2003 From psadhukhan at openjdk.java.net Wed Jan 13 07:03:57 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 13 Jan 2021 07:03:57 GMT Subject: Integrated: 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 In-Reply-To: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> References: <_Z-JJqhyCnKAUaKRK370cz9rCtIdrdL3bSBtDM-atKA=.c33c5d38-607b-4c72-849c-be0e6442e2c9@github.com> Message-ID: On Fri, 8 Jan 2021 11:03:36 GMT, Prasanta Sadhukhan wrote: > This test was failing on mach5 nightly on ubuntu systems long time back. Modified the test to add some delays and call robot.waitForIdle() to make it more stable and the resultant test passes on all mach5 systems including linux consisting of ubuntu 19.10, 18.04,20.10,20.04 so we can remove the test from problemlist. > mach5 job running for several iterations is posted in JBS. This pull request has now been integrated. Changeset: a483869a Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/a483869a Stats: 13 lines in 2 files changed: 11 ins; 2 del; 0 mod 8225045: javax/swing/JInternalFrame/8146321/JInternalFrameIconTest.java fails on linux-x64 Reviewed-by: serb, pbansal, trebari ------------- PR: https://git.openjdk.java.net/jdk/pull/2003 From serb at openjdk.java.net Wed Jan 13 07:07:54 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Wed, 13 Jan 2021 07:07:54 GMT Subject: RFR: 8259522: Apply java.io.Serial annotations in java.desktop [v2] In-Reply-To: <0BeCSRlVfit-78buTvO8-tci6rZDhwsML4JXKzujLM8=.ff5ba622-5c99-4925-b690-5b348d0d0e18@github.com> References: <7X46pwNuTDGbEN3-jFa2QY2qUtMCK2dAu-BiPdoMK14=.19bc926b-62ba-4f79-8f32-be27898077a8@github.com> <0BeCSRlVfit-78buTvO8-tci6rZDhwsML4JXKzujLM8=.ff5ba622-5c99-4925-b690-5b348d0d0e18@github.com> Message-ID: On Wed, 13 Jan 2021 06:10:53 GMT, Prasanta Sadhukhan wrote: > Why do we add serialVersionUID in some classes like DefaultMutableTreeNode.java but not in other swing classes? Most Swing classes are marked by the specific "Warning" that "Same-version serialization only" is supported. (I think such a warning is missed in a few classes). So generally the serialVersionUID field is not needed in such classes, but if present this provides a small benefit -> this UID is not generated at runtime. For example, the DefaultMutableTreeNode was updated by the JDK-5017904 fix, which was unrelated to serialization but was targeted for the performance issue. > Also, if this change is for stricter compile-time checking, shouldn't we remove @SuppressWarnings("serial") check? At some point, we probably can remove it but will need to fix all serialization warnings which were disabled. ------------- PR: https://git.openjdk.java.net/jdk/pull/2020 From psadhukhan at openjdk.java.net Wed Jan 13 07:34:56 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 13 Jan 2021 07:34:56 GMT Subject: RFR: 8259522: Apply java.io.Serial annotations in java.desktop [v2] In-Reply-To: <7X46pwNuTDGbEN3-jFa2QY2qUtMCK2dAu-BiPdoMK14=.19bc926b-62ba-4f79-8f32-be27898077a8@github.com> References: <7X46pwNuTDGbEN3-jFa2QY2qUtMCK2dAu-BiPdoMK14=.19bc926b-62ba-4f79-8f32-be27898077a8@github.com> Message-ID: On Tue, 12 Jan 2021 20:42:18 GMT, Sergey Bylokhov wrote: >> Please review the application of @java.io.Serial annotation (JDK-8202385) to types in the desktop module to enable stricter compile-time checking of serialization-related declarations. >> >> This annotation can be applied to these methods in the module: >> >> private void writeObject(java.io.ObjectOutputStream stream) throws IOException >> private void readObject(java.io.ObjectInputStream stream) throws IOException, ClassNotFoundException >> private void readObjectNoData() throws ObjectStreamException >> ANY-ACCESS-MODIFIER Object writeReplace() throws ObjectStreamException >> ANY-ACCESS-MODIFIER Object readResolve() throws ObjectStreamException >> private static final ObjectStreamField[] serialPersistentFields >> private static final long serialVersionUID >> >> Notes: >> - I have tried to update the comments for serialVersionUID as accurately as possible, but mostly based on the source code history and bugs in JBS where that field was added >> - Some of the readObject/writeObject methods in the javax.swing package does not have a spec, because this package and some others are excluded from the serialization specification. >> >> A similar fix was implemented for java.base module as well: >> http://mail.openjdk.java.net/pipermail/core-libs-dev/2019-August/062046.html > > Sergey Bylokhov has updated the pull request incrementally with two additional commits since the last revision: > > - Update src/java.desktop/share/classes/java/awt/image/ImagingOpException.java > > Co-authored-by: Aleksei Ivanov <70774172+aivanov-jdk at users.noreply.github.com> > - Update src/java.desktop/share/classes/java/awt/image/RasterFormatException.java > > Co-authored-by: Aleksei Ivanov <70774172+aivanov-jdk at users.noreply.github.com> Marked as reviewed by psadhukhan (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2020 From psadhukhan at openjdk.java.net Wed Jan 13 12:49:05 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 13 Jan 2021 12:49:05 GMT Subject: RFR: 8202880: Test javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java fails Message-ID: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> This test was unstable in linux in nightly testing. Modified to move the frame to center of screen so that the left-taskbar of linux doesn't interfere with the mouse movement along with delay after frame is visible to make it more stable. Mach5 job running for several iterations on all platforms is ok. Link in JBS. ------------- Commit messages: - 8202880: Test javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java fails Changes: https://git.openjdk.java.net/jdk/pull/2061/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2061&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8202880 Stats: 48 lines in 2 files changed: 9 ins; 22 del; 17 mod Patch: https://git.openjdk.java.net/jdk/pull/2061.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2061/head:pull/2061 PR: https://git.openjdk.java.net/jdk/pull/2061 From aivanov at openjdk.java.net Wed Jan 13 15:48:03 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 13 Jan 2021 15:48:03 GMT Subject: RFR: 8202880: Test javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java fails In-Reply-To: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> References: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> Message-ID: On Wed, 13 Jan 2021 10:41:29 GMT, Prasanta Sadhukhan wrote: > This test was unstable in linux in nightly testing. Modified to move the frame to center of screen so that the left-taskbar of linux doesn't interfere with the mouse movement along with delay after frame is visible to make it more stable. > Mach5 job running for several iterations on all platforms is ok. Link in JBS. Copyright year in the test and in the ProblemList.txt, since it exists there, should be updated. test/jdk/javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java line 55: > 53: private static JMenuItem paste; > 54: private static JMenuItem delete; > 55: private static JMenuItem selectAll; These can be local variables; as well as the menuBar. They're not accessed anywhere else but createGUI(). test/jdk/javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java line 183: > 181: frame.pack(); > 182: frame.setVisible(true); > 183: frame.setLocationRelativeTo(null); Shall `setLocationRelativeTo` be called before `setVisible(true)`? test/jdk/javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java line 82: > 80: robot.delay(1000); > 81: > 82: System.out.println("popmenu visible " + menu.isPopupMenuVisible()); `menu.isPopupMenuVisible()` should also be called on EDT. ------------- PR: https://git.openjdk.java.net/jdk/pull/2061 From github.com+70650887+skodanda at openjdk.java.net Wed Jan 13 16:30:05 2021 From: github.com+70650887+skodanda at openjdk.java.net (K Suman Rajkumaar) Date: Wed, 13 Jan 2021 16:30:05 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v6] In-Reply-To: References: Message-ID: On Thu, 7 Jan 2021 15:54:12 GMT, Alexey Ivanov wrote: >> "Hi All, Could you please review this fix for JDK16?" >> This is for JDK 17, not JDK 16, right ? I can't imagine why it would be for 16 at this juncture .. > >> "Hi All, Could you please review this fix for JDK16?" >> This is for JDK 17, not JDK 16, right ? I can't imagine why it would be for 16 at this juncture .. > > No, it's not meant for JDK 16 ? it's for jdk-dev, the main development line which is JDK 17 at this very moment. Dear Phil/Sergey, I have taken care of the review comments. Please have a look. Best Regards, K Suman Rajkumaar ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From serb at openjdk.java.net Wed Jan 13 21:17:09 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Wed, 13 Jan 2021 21:17:09 GMT Subject: RFR: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test [v7] In-Reply-To: References: Message-ID: On Thu, 7 Jan 2021 05:54:07 GMT, K Suman Rajkumaar wrote: >> Hi All, Could you please review this fix for JDK16? >> >> Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. >> >> Fix: Rewritten the above applet based test to a regular java test. >> >> Best Regards, >> K Suman Rajkumaar > > K Suman Rajkumaar has updated the pull request incrementally with one additional commit since the last revision: > > Changed "used" to "used for" as per Phil's review comment. Marked as reviewed by serb (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From github.com+1247730+stanio at openjdk.java.net Wed Jan 13 21:41:13 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Wed, 13 Jan 2021 21:41:13 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes Message-ID: Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. **Current behavior**

Foo

Bar
  1. Baz
Qux
**Expected behavior** All text should be displayed with a font size of the computed `` font-size ? 1.5. [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 ------------- Commit messages: - 8257664: Fix font-size inheritance with percentage values Changes: https://git.openjdk.java.net/jdk/pull/1759/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1759&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8257664 Stats: 132 lines in 2 files changed: 132 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/1759.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1759/head:pull/1759 PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Wed Jan 13 21:41:13 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Wed, 13 Jan 2021 21:41:13 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes In-Reply-To: References: Message-ID: <9BOPWTWFGoZdPM6XZhrIsehHu7FagIIAkzpsIR6udcc=.c3a81f47-68de-4905-b7bc-ff7a829958c9@github.com> On Sun, 13 Dec 2020 14:51:47 GMT, Stanimir Stamenkov wrote: > Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. > > _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. > > **Current behavior** > > > > > >

Foo

> >
Bar
> >
    >
  1. Baz
  2. >
> > > > > >
Qux
> > > > **Expected behavior** > > All text should be displayed with a font size of the computed `` font-size ? 1.5. > > [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 I'll need a few days to find a printer and a scanner for submitting OCA. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Wed Jan 13 21:41:14 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Wed, 13 Jan 2021 21:41:14 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes In-Reply-To: <9BOPWTWFGoZdPM6XZhrIsehHu7FagIIAkzpsIR6udcc=.c3a81f47-68de-4905-b7bc-ff7a829958c9@github.com> References: <9BOPWTWFGoZdPM6XZhrIsehHu7FagIIAkzpsIR6udcc=.c3a81f47-68de-4905-b7bc-ff7a829958c9@github.com> Message-ID: <8I8iiGtAjkh4TPzYWZS4gzpdXaIenmX7plLzcR_0OYo=.843ede15-987e-4ea6-9c14-184f64a39cdc@github.com> On Sun, 13 Dec 2020 14:58:07 GMT, Stanimir Stamenkov wrote: >> Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. >> >> _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. >> >> **Current behavior** >> >> >> >> >> >>

Foo

>> >>
Bar
>> >>
    >>
  1. Baz
  2. >>
>> >> >> >> >> >>
Qux
>> >> >> >> **Expected behavior** >> >> All text should be displayed with a font size of the computed `` font-size ? 1.5. >> >> [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 > > I'll need a few days to find a printer and a scanner for submitting OCA. I'll try to get my OCA done this week. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Wed Jan 13 21:41:14 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Wed, 13 Jan 2021 21:41:14 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes In-Reply-To: <8I8iiGtAjkh4TPzYWZS4gzpdXaIenmX7plLzcR_0OYo=.843ede15-987e-4ea6-9c14-184f64a39cdc@github.com> References: <9BOPWTWFGoZdPM6XZhrIsehHu7FagIIAkzpsIR6udcc=.c3a81f47-68de-4905-b7bc-ff7a829958c9@github.com> <8I8iiGtAjkh4TPzYWZS4gzpdXaIenmX7plLzcR_0OYo=.843ede15-987e-4ea6-9c14-184f64a39cdc@github.com> Message-ID: <9lvhXeoEpJbEESHDc_q4NOXZ6aAzD4lgkCDT0hJ2WsQ=.4a460617-19e2-4787-80f1-697a2ad3f528@github.com> On Mon, 11 Jan 2021 07:52:25 GMT, Stanimir Stamenkov wrote: >> I'll need a few days to find a printer and a scanner for submitting OCA. > > I'll try to get my OCA done this week. I've just emailed my scanned OCA. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From robilad at openjdk.java.net Wed Jan 13 21:49:03 2021 From: robilad at openjdk.java.net (Dalibor Topic) Date: Wed, 13 Jan 2021 21:49:03 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes In-Reply-To: <9lvhXeoEpJbEESHDc_q4NOXZ6aAzD4lgkCDT0hJ2WsQ=.4a460617-19e2-4787-80f1-697a2ad3f528@github.com> References: <9BOPWTWFGoZdPM6XZhrIsehHu7FagIIAkzpsIR6udcc=.c3a81f47-68de-4905-b7bc-ff7a829958c9@github.com> <8I8iiGtAjkh4TPzYWZS4gzpdXaIenmX7plLzcR_0OYo=.843ede15-987e-4ea6-9c14-184f64a39cdc@github.com> <9lvhXeoEpJbEESHDc_q4NOXZ6aAzD4lgkCDT0hJ2WsQ=.4a460617-19e2-4787-80f1-697a2ad3f528@github.com> Message-ID: On Wed, 13 Jan 2021 16:12:26 GMT, Stanimir Stamenkov wrote: >> I'll try to get my OCA done this week. > > I've just emailed my scanned OCA. Thank you. Your OCA has been processed and your account has been marked as verified. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From kizune at openjdk.java.net Wed Jan 13 22:31:15 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Wed, 13 Jan 2021 22:31:15 GMT Subject: [jdk16] RFR: 8258643: [TESTBUG] javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" Message-ID: Backport from mainline. ------------- Commit messages: - 8258643: [TESTBUG] javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" Changes: https://git.openjdk.java.net/jdk16/pull/119/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk16&pr=119&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8258643 Stats: 12 lines in 1 file changed: 1 ins; 5 del; 6 mod Patch: https://git.openjdk.java.net/jdk16/pull/119.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/119/head:pull/119 PR: https://git.openjdk.java.net/jdk16/pull/119 From jwilhelm at openjdk.java.net Thu Jan 14 01:25:23 2021 From: jwilhelm at openjdk.java.net (Jesper Wilhelmsson) Date: Thu, 14 Jan 2021 01:25:23 GMT Subject: RFR: Merge jdk16 Message-ID: <-aqBh7X3owjZhc7NJ9IadTg_pDoBEcSVULP6D9ZQlwc=.2f9552f6-95ee-4bdd-8591-e2fea0f2ee47@github.com> Forwardport JDK 16 -> JDK 17 ------------- Commit messages: - Merge - 8259719: ProblemList runtime/cds/appcds/jigsaw/modulepath/ModulePathAndCP_JFR.java on Windows - 8259720: ProblemList java/awt/Focus/AppletInitialFocusTest/AppletInitialFocusTest1.java on Windows - 8259722: ProblemList two jdk/jfr/startupargs tests on Windows - 8231461: static/instance overload leads to 'unexpected static method found in unbound lookup' when resolving method reference - 8259063: Possible deadlock with vtable/itable creation vs concurrent class unloading - 8259657: typo in generated HELP page prevents localization - 8258272: LoadVectorMaskedNode can't be replaced by zero con - 8259560: Zero m68k: "static assertion failed: align" after JDK-8252049 - 8258985: Parallel WeakProcessor may use too few threads - ... and 11 more: https://git.openjdk.java.net/jdk/compare/c7e2174b...64cae854 The webrevs contain the adjustments done while merging with regards to each parent branch: - master: https://webrevs.openjdk.java.net/?repo=jdk&pr=2072&range=00.0 - jdk16: https://webrevs.openjdk.java.net/?repo=jdk&pr=2072&range=00.1 Changes: https://git.openjdk.java.net/jdk/pull/2072/files Stats: 991 lines in 48 files changed: 764 ins; 123 del; 104 mod Patch: https://git.openjdk.java.net/jdk/pull/2072.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2072/head:pull/2072 PR: https://git.openjdk.java.net/jdk/pull/2072 From jwilhelm at openjdk.java.net Thu Jan 14 01:33:06 2021 From: jwilhelm at openjdk.java.net (Jesper Wilhelmsson) Date: Thu, 14 Jan 2021 01:33:06 GMT Subject: Integrated: Merge jdk16 In-Reply-To: <-aqBh7X3owjZhc7NJ9IadTg_pDoBEcSVULP6D9ZQlwc=.2f9552f6-95ee-4bdd-8591-e2fea0f2ee47@github.com> References: <-aqBh7X3owjZhc7NJ9IadTg_pDoBEcSVULP6D9ZQlwc=.2f9552f6-95ee-4bdd-8591-e2fea0f2ee47@github.com> Message-ID: On Thu, 14 Jan 2021 01:20:13 GMT, Jesper Wilhelmsson wrote: > Forwardport JDK 16 -> JDK 17 This pull request has now been integrated. Changeset: 51e14f2e Author: Jesper Wilhelmsson URL: https://git.openjdk.java.net/jdk/commit/51e14f2e Stats: 991 lines in 48 files changed: 764 ins; 123 del; 104 mod Merge ------------- PR: https://git.openjdk.java.net/jdk/pull/2072 From ljiang at openjdk.java.net Thu Jan 14 14:05:11 2021 From: ljiang at openjdk.java.net (Leo Jiang) Date: Thu, 14 Jan 2021 14:05:11 GMT Subject: [jdk16] RFR: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 Message-ID: This is the changes for JDK 16 msg drop 10. ------------- Commit messages: - JDK-8259732: JDK 16 L10n resource file update - msg drop 10 Changes: https://git.openjdk.java.net/jdk16/pull/123/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk16&pr=123&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8259732 Stats: 215 lines in 30 files changed: 118 ins; 16 del; 81 mod Patch: https://git.openjdk.java.net/jdk16/pull/123.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/123/head:pull/123 PR: https://git.openjdk.java.net/jdk16/pull/123 From ljiang at openjdk.java.net Thu Jan 14 14:13:02 2021 From: ljiang at openjdk.java.net (Leo Jiang) Date: Thu, 14 Jan 2021 14:13:02 GMT Subject: [jdk16] RFR: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 In-Reply-To: References: Message-ID: On Thu, 14 Jan 2021 14:00:00 GMT, Leo Jiang wrote: > This is the changes for JDK 16 msg drop 10. [webrev.tar.gz](https://github.com/openjdk/jdk16/files/5814846/webrev.tar.gz) Since they're Unicode escape sequences in the l10n resource files, so I attached a human readable webrev, created by `git webrev` and converted. Pls find this to help your review. ------------- PR: https://git.openjdk.java.net/jdk16/pull/123 From ljiang at openjdk.java.net Thu Jan 14 14:27:25 2021 From: ljiang at openjdk.java.net (Leo Jiang) Date: Thu, 14 Jan 2021 14:27:25 GMT Subject: [jdk16] RFR: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 [v2] In-Reply-To: References: Message-ID: > This is the changes for JDK 16 msg drop 10. Leo Jiang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: - Merge branch 'master' into msgdrop - JDK-8259732: JDK 16 L10n resource file update - msg drop 10 ------------- Changes: - all: https://git.openjdk.java.net/jdk16/pull/123/files - new: https://git.openjdk.java.net/jdk16/pull/123/files/230117b4..d72f444a Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk16&pr=123&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk16&pr=123&range=00-01 Stats: 718 lines in 32 files changed: 616 ins; 38 del; 64 mod Patch: https://git.openjdk.java.net/jdk16/pull/123.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/123/head:pull/123 PR: https://git.openjdk.java.net/jdk16/pull/123 From ljiang at openjdk.java.net Thu Jan 14 15:38:09 2021 From: ljiang at openjdk.java.net (Leo Jiang) Date: Thu, 14 Jan 2021 15:38:09 GMT Subject: [jdk16] RFR: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 In-Reply-To: References: Message-ID: On Thu, 14 Jan 2021 14:10:12 GMT, Leo Jiang wrote: >> This is the changes for JDK 16 msg drop 10. > > [webrev.tar.gz](https://github.com/openjdk/jdk16/files/5814846/webrev.tar.gz) > > Since they're Unicode escape sequences in the l10n resource files, so I attached a human readable webrev, created by `git webrev` and converted. Pls find this to help your review. @naotoj Would you have time to take a look at this change? Very appreciated! ------------- PR: https://git.openjdk.java.net/jdk16/pull/123 From naoto at openjdk.java.net Thu Jan 14 17:22:09 2021 From: naoto at openjdk.java.net (Naoto Sato) Date: Thu, 14 Jan 2021 17:22:09 GMT Subject: [jdk16] RFR: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 [v2] In-Reply-To: References: Message-ID: On Thu, 14 Jan 2021 14:27:25 GMT, Leo Jiang wrote: >> This is the changes for JDK 16 msg drop 10. > > Leo Jiang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: > > - Merge branch 'master' into msgdrop > - JDK-8259732: JDK 16 L10n resource file update - msg drop 10 src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/standard.properties line 518: > 516: doclet.footer_specified=\ > 517: The -footer option is no longer supported and will be ignored.\n\ > 518: It may be removed in a future release. I believe this is to fix no newline at the end (unrelated to l10n changes). Do we need to change the copyright year for this? ------------- PR: https://git.openjdk.java.net/jdk16/pull/123 From serb at openjdk.java.net Fri Jan 15 00:31:01 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Fri, 15 Jan 2021 00:31:01 GMT Subject: Integrated: 8259522: Apply java.io.Serial annotations in java.desktop In-Reply-To: References: Message-ID: On Mon, 11 Jan 2021 06:21:52 GMT, Sergey Bylokhov wrote: > Please review the application of @java.io.Serial annotation (JDK-8202385) to types in the desktop module to enable stricter compile-time checking of serialization-related declarations. > > This annotation can be applied to these methods in the module: > > private void writeObject(java.io.ObjectOutputStream stream) throws IOException > private void readObject(java.io.ObjectInputStream stream) throws IOException, ClassNotFoundException > private void readObjectNoData() throws ObjectStreamException > ANY-ACCESS-MODIFIER Object writeReplace() throws ObjectStreamException > ANY-ACCESS-MODIFIER Object readResolve() throws ObjectStreamException > private static final ObjectStreamField[] serialPersistentFields > private static final long serialVersionUID > > Notes: > - I have tried to update the comments for serialVersionUID as accurately as possible, but mostly based on the source code history and bugs in JBS where that field was added > - Some of the readObject/writeObject methods in the javax.swing package does not have a spec, because this package and some others are excluded from the serialization specification. > > A similar fix was implemented for java.base module as well: > http://mail.openjdk.java.net/pipermail/core-libs-dev/2019-August/062046.html This pull request has now been integrated. Changeset: 978bed6c Author: Sergey Bylokhov URL: https://git.openjdk.java.net/jdk/commit/978bed6c Stats: 3424 lines in 343 files changed: 2264 ins; 287 del; 873 mod 8259522: Apply java.io.Serial annotations in java.desktop Reviewed-by: aivanov, psadhukhan ------------- PR: https://git.openjdk.java.net/jdk/pull/2020 From ljiang at openjdk.java.net Fri Jan 15 02:02:11 2021 From: ljiang at openjdk.java.net (Leo Jiang) Date: Fri, 15 Jan 2021 02:02:11 GMT Subject: [jdk16] RFR: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 [v2] In-Reply-To: References: Message-ID: <1Bb8sf6zFDnQSM5kRQIlrgBNwEpO-IvxsPmN05F0QFs=.13eb3035-71d5-4310-98e5-d9989e51cf60@github.com> On Thu, 14 Jan 2021 17:19:11 GMT, Naoto Sato wrote: >> Leo Jiang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: >> >> - Merge branch 'master' into msgdrop >> - JDK-8259732: JDK 16 L10n resource file update - msg drop 10 > > src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/standard.properties line 518: > >> 516: doclet.footer_specified=\ >> 517: The -footer option is no longer supported and will be ignored.\n\ >> 518: It may be removed in a future release. > > I believe this is to fix no newline at the end (unrelated to l10n changes). Do we need to change the copyright year for this? Actually I was correcting the L217, changed {) -> {}. But 2 days ago, Jonathan Gibbons fixed it in another commit 6bb6093. I found his fix after running the merge. Looks both of us forgot to update the copyright year. Any suggestion? doclet.help.systemProperties.body=\ The {0} page lists references to system properties. ------------- PR: https://git.openjdk.java.net/jdk16/pull/123 From trebari at openjdk.java.net Fri Jan 15 05:16:01 2021 From: trebari at openjdk.java.net (Tejpal Rebari) Date: Fri, 15 Jan 2021 05:16:01 GMT Subject: [jdk16] RFR: 8258643: [TESTBUG] javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" In-Reply-To: References: Message-ID: On Wed, 13 Jan 2021 22:26:47 GMT, Alexander Zuev wrote: > Backport from mainline. Marked as reviewed by trebari (Committer). ------------- PR: https://git.openjdk.java.net/jdk16/pull/119 From psadhukhan at openjdk.java.net Fri Jan 15 05:38:05 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 15 Jan 2021 05:38:05 GMT Subject: [jdk16] RFR: 8258643: [TESTBUG] javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" In-Reply-To: References: Message-ID: On Wed, 13 Jan 2021 22:26:47 GMT, Alexander Zuev wrote: > Backport from mainline. Marked as reviewed by psadhukhan (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk16/pull/119 From serb at openjdk.java.net Fri Jan 15 06:15:04 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Fri, 15 Jan 2021 06:15:04 GMT Subject: [jdk16] RFR: 8258643: [TESTBUG] javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" In-Reply-To: References: Message-ID: <9ApFtEJnjQjbHNB0t2M4z9ik_luPXD63A5xJQL9WirA=.9f591c04-a491-4202-98db-a36e9a53ab8e@github.com> On Fri, 15 Jan 2021 05:35:06 GMT, Prasanta Sadhukhan wrote: >> Backport from mainline. > > Marked as reviewed by psadhukhan (Reviewer). Looks like this test still fails in the mainline as well: https://bugs.openjdk.java.net/browse/JDK-8259650 ------------- PR: https://git.openjdk.java.net/jdk16/pull/119 From kizune at openjdk.java.net Fri Jan 15 06:22:04 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Fri, 15 Jan 2021 06:22:04 GMT Subject: [jdk16] RFR: 8258643: [TESTBUG] javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" In-Reply-To: <9ApFtEJnjQjbHNB0t2M4z9ik_luPXD63A5xJQL9WirA=.9f591c04-a491-4202-98db-a36e9a53ab8e@github.com> References: <9ApFtEJnjQjbHNB0t2M4z9ik_luPXD63A5xJQL9WirA=.9f591c04-a491-4202-98db-a36e9a53ab8e@github.com> Message-ID: On Fri, 15 Jan 2021 06:12:15 GMT, Sergey Bylokhov wrote: > Looks like this test still fails in the mainline as well: https://bugs.openjdk.java.net/browse/JDK-8259650 Yes but the reason is different. Now it seems like component's screenshot taken when component is not fully painted. Looking into it. ------------- PR: https://git.openjdk.java.net/jdk16/pull/119 From psadhukhan at openjdk.java.net Fri Jan 15 06:33:19 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 15 Jan 2021 06:33:19 GMT Subject: RFR: 8202880: Test javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java fails [v2] In-Reply-To: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> References: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> Message-ID: <7UBZ96Nh6YdPKEF0C06wkbrPdLLHSeKLgdEf-i4yHrc=.52da70b4-7c8e-4f6d-9fc3-1637e509e34e@github.com> > This test was unstable in linux in nightly testing. Modified to move the frame to center of screen so that the left-taskbar of linux doesn't interfere with the mouse movement along with delay after frame is visible to make it more stable. > Mach5 job running for several iterations on all platforms is ok. Link in JBS. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: Address review comments ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/2061/files - new: https://git.openjdk.java.net/jdk/pull/2061/files/ca7e5589..bc7ef81f Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2061&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2061&range=00-01 Stats: 15 lines in 2 files changed: 6 ins; 7 del; 2 mod Patch: https://git.openjdk.java.net/jdk/pull/2061.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2061/head:pull/2061 PR: https://git.openjdk.java.net/jdk/pull/2061 From psadhukhan at openjdk.java.net Fri Jan 15 09:42:07 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 15 Jan 2021 09:42:07 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes In-Reply-To: References: Message-ID: On Sun, 13 Dec 2020 14:51:47 GMT, Stanimir Stamenkov wrote: > Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. > > _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. > > **Current behavior** > > > > > >

Foo

> >
Bar
> >
    >
  1. Baz
  2. >
> > > > > >
Qux
> > > > **Expected behavior** > > All text should be displayed with a font size of the computed `` font-size ? 1.5. > > [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 test/jdk/javax/swing/text/html/StyleSheet/bug8257665.java line 4: > 2: * > 3: */ > 4: You can copy the copyright from any of the test present in this directory, just change the year to 2021 test/jdk/javax/swing/text/html/StyleSheet/bug8257665.java line 14: > 12: /* > 13: * @test > 14: * @key headless No need to specify headless tag. If it's not @key headful, then it is implicitly headless test/jdk/javax/swing/text/html/StyleSheet/bug8257665.java line 17: > 15: * @bug 8257665 > 16: * @summary Tests inherited font-size with parent percentage specification. > 17: * @run main bug8257665 We normally dont use bugid as name for the test. You probably can name it TestWrongCSSFontSize.java src/java.desktop/share/classes/javax/swing/text/html/StyleSheet.java line 3423: > 3421: */ > 3422: static final Object FONT_SIZE_INHERIT = > 3423: new CSS().new FontSize().parseCssValue("100%"); Do we need to instantiate new CSS object here? Can't we reuse "css" variable which is already initialized? ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+70650887+skodanda at openjdk.java.net Fri Jan 15 09:43:04 2021 From: github.com+70650887+skodanda at openjdk.java.net (K Suman Rajkumaar) Date: Fri, 15 Jan 2021 09:43:04 GMT Subject: Integrated: JDK-8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test In-Reply-To: References: Message-ID: On Wed, 23 Dec 2020 11:50:41 GMT, K Suman Rajkumaar wrote: > Hi All, Could you please review this fix for JDK16? > > Problem Description: The test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java is applet based. > > Fix: Rewritten the above applet based test to a regular java test. > > Best Regards, > K Suman Rajkumaar This pull request has now been integrated. Changeset: b01a15e4 Author: K Suman Rajkumaar <70650887+skodanda at users.noreply.github.com> Committer: Alexey Ivanov URL: https://git.openjdk.java.net/jdk/commit/b01a15e4 Stats: 130 lines in 2 files changed: 68 ins; 44 del; 18 mod 8258884: [TEST_BUG] Convert applet-based test open/test/jdk/javax/swing/JMenuItem/8031573/bug8031573.java to a regular java test Reviewed-by: aivanov, serb ------------- PR: https://git.openjdk.java.net/jdk/pull/1878 From github.com+70650887+skodanda at openjdk.java.net Fri Jan 15 10:38:10 2021 From: github.com+70650887+skodanda at openjdk.java.net (K Suman Rajkumaar) Date: Fri, 15 Jan 2021 10:38:10 GMT Subject: RFR: JDK-8259818: [TEST_BUG] Convert applet-based test test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java to a regular java Message-ID: Dear All, Hi All, Could you please review this fix for JDK17? Problem Description: The test open/test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java is applet based. Fix: Rewritten the above applet based test as a regular java test. Best Regards, K Suman Rajkumaar ------------- Commit messages: - Removed white spaces - Rewritten the test test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java as a regular java application. Changes: https://git.openjdk.java.net/jdk/pull/2094/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2094&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8259818 Stats: 135 lines in 2 files changed: 77 ins; 37 del; 21 mod Patch: https://git.openjdk.java.net/jdk/pull/2094.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2094/head:pull/2094 PR: https://git.openjdk.java.net/jdk/pull/2094 From github.com+1247730+stanio at openjdk.java.net Fri Jan 15 12:47:28 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Fri, 15 Jan 2021 12:47:28 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v2] In-Reply-To: References: Message-ID: > Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. > > _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. > > **Current behavior** > > > > > >

Foo

> >
Bar
> >
    >
  1. Baz
  2. >
> > > > > >
Qux
> > > > **Expected behavior** > > All text should be displayed with a font size of the computed `` font-size ? 1.5. > > [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: fixup! 8257664: Fix font-size inheritance with percentage values - Renamed bug8257665.java -> TestWrongCSSFontSize.java - Added full copyright statement - Removed redundant @key headless - Corrected bug no. 8257665 -> 8257664 ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1759/files - new: https://git.openjdk.java.net/jdk/pull/1759/files/ea857fcf..c5b6d41c Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1759&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1759&range=00-01 Stats: 250 lines in 2 files changed: 134 ins; 116 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/1759.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1759/head:pull/1759 PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Fri Jan 15 12:47:29 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Fri, 15 Jan 2021 12:47:29 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v2] In-Reply-To: References: Message-ID: On Fri, 15 Jan 2021 09:32:15 GMT, Prasanta Sadhukhan wrote: >> Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: >> >> fixup! 8257664: Fix font-size inheritance with percentage values >> >> - Renamed bug8257665.java -> TestWrongCSSFontSize.java >> - Added full copyright statement >> - Removed redundant @key headless >> - Corrected bug no. 8257665 -> 8257664 > > test/jdk/javax/swing/text/html/StyleSheet/bug8257665.java line 17: > >> 15: * @bug 8257665 >> 16: * @summary Tests inherited font-size with parent percentage specification. >> 17: * @run main bug8257665 > > We normally dont use bugid as name for the test. You probably can name it TestWrongCSSFontSize.java Renamed. I had mimicked the existing `bug4936917.java`, initially. I also had the `@bug` number wrong. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Fri Jan 15 12:56:05 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Fri, 15 Jan 2021 12:56:05 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v2] In-Reply-To: References: Message-ID: On Fri, 15 Jan 2021 09:38:16 GMT, Prasanta Sadhukhan wrote: >> Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: >> >> fixup! 8257664: Fix font-size inheritance with percentage values >> >> - Renamed bug8257665.java -> TestWrongCSSFontSize.java >> - Added full copyright statement >> - Removed redundant @key headless >> - Corrected bug no. 8257665 -> 8257664 > > src/java.desktop/share/classes/javax/swing/text/html/StyleSheet.java line 3423: > >> 3421: */ >> 3422: static final Object FONT_SIZE_INHERIT = >> 3423: new CSS().new FontSize().parseCssValue("100%"); > > Do we need to instantiate new CSS object here? Can't we reuse "css" variable which is already initialized? I hadn't previously realized the `ViewAttributeSet` class is not static, so it should be possible to reuse the instance `css`. Not sure if a separate instance per style sheet is necessary, but I'll prepare a revision. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From aivanov at openjdk.java.net Fri Jan 15 13:24:04 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Fri, 15 Jan 2021 13:24:04 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes In-Reply-To: References: <9BOPWTWFGoZdPM6XZhrIsehHu7FagIIAkzpsIR6udcc=.c3a81f47-68de-4905-b7bc-ff7a829958c9@github.com> <8I8iiGtAjkh4TPzYWZS4gzpdXaIenmX7plLzcR_0OYo=.843ede15-987e-4ea6-9c14-184f64a39cdc@github.com> <9lvhXeoEpJbEESHDc_q4NOXZ6aAzD4lgkCDT0hJ2WsQ=.4a460617-19e2-4787-80f1-697a2ad3f528@github.com> Message-ID: <6gfK8xPDLEjtARTMW97tt3ud4gGuh8F9Kx8IOiFoIJk=.f17715d2-f11f-49f8-b2ab-e43bc4a4a445@github.com> On Wed, 13 Jan 2021 21:46:45 GMT, Dalibor Topic wrote: >> I've just emailed my scanned OCA. > > Thank you. Your OCA has been processed and your account has been marked as verified. Providing a way to display the rendered HTML, especially when the test fails, for visual inspection could be a bonus. This feature could be activated with a parameter passed to the test. It's not a requirement, just a suggestion. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From aivanov at openjdk.java.net Fri Jan 15 13:24:07 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Fri, 15 Jan 2021 13:24:07 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v2] In-Reply-To: References: Message-ID: On Fri, 15 Jan 2021 09:31:38 GMT, Prasanta Sadhukhan wrote: >> Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: >> >> fixup! 8257664: Fix font-size inheritance with percentage values >> >> - Renamed bug8257665.java -> TestWrongCSSFontSize.java >> - Added full copyright statement >> - Removed redundant @key headless >> - Corrected bug no. 8257665 -> 8257664 > > test/jdk/javax/swing/text/html/StyleSheet/bug8257665.java line 14: > >> 12: /* >> 13: * @test >> 14: * @key headless > > No need to specify headless tag. If it's not @key headful, then it is implicitly headless I believe the test is _headful_ because it uses `SwingUtilities.invokeAndWait` even though it does not show any UI. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Fri Jan 15 15:35:04 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Fri, 15 Jan 2021 15:35:04 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v2] In-Reply-To: References: Message-ID: <4kOQwc_o73RnC2NkOs5V_74VJf_NaQG_gCGutpR0pZ4=.8307e31e-9ca8-4427-a859-9dc3b0063b0e@github.com> On Fri, 15 Jan 2021 13:18:44 GMT, Alexey Ivanov wrote: >> test/jdk/javax/swing/text/html/StyleSheet/bug8257665.java line 14: >> >>> 12: /* >>> 13: * @test >>> 14: * @key headless >> >> No need to specify headless tag. If it's not @key headful, then it is implicitly headless > > I believe the test is _headful_ because it uses `SwingUtilities.invokeAndWait` even though it does not show any UI. I've used `SwingUtilities.invokeAndWait` just to be on the safe side that "everything Swing should be executed on the EDT" but I guess I could drop it as no UI ever shown? ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From psadhukhan at openjdk.java.net Fri Jan 15 15:49:01 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 15 Jan 2021 15:49:01 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v2] In-Reply-To: <4kOQwc_o73RnC2NkOs5V_74VJf_NaQG_gCGutpR0pZ4=.8307e31e-9ca8-4427-a859-9dc3b0063b0e@github.com> References: <4kOQwc_o73RnC2NkOs5V_74VJf_NaQG_gCGutpR0pZ4=.8307e31e-9ca8-4427-a859-9dc3b0063b0e@github.com> Message-ID: On Fri, 15 Jan 2021 15:31:54 GMT, Stanimir Stamenkov wrote: >> I believe the test is _headful_ because it uses `SwingUtilities.invokeAndWait` even though it does not show any UI. > > I've used `SwingUtilities.invokeAndWait` just to be on the safe side that "everything Swing should be executed on the EDT" but I guess I could drop it as no UI ever shown? I am not sure that if we use invokeAndWait(), it needs to be headful..There are many test where we use invokeAndWait but they are headless. But it will be good to see what the test is doing visually so you probably can make it headful. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From psadhukhan at openjdk.java.net Fri Jan 15 15:49:06 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 15 Jan 2021 15:49:06 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v2] In-Reply-To: References: Message-ID: On Fri, 15 Jan 2021 12:47:28 GMT, Stanimir Stamenkov wrote: >> Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. >> >> _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. >> >> **Current behavior** >> >> >> >> >> >>

Foo

>> >>
Bar
>> >>
    >>
  1. Baz
  2. >>
>> >> >> >> >> >>
Qux
>> >> >> >> **Expected behavior** >> >> All text should be displayed with a font size of the computed `` font-size ? 1.5. >> >> [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 > > Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: > > fixup! 8257664: Fix font-size inheritance with percentage values > > - Renamed bug8257665.java -> TestWrongCSSFontSize.java > - Added full copyright statement > - Removed redundant @key headless > - Corrected bug no. 8257665 -> 8257664 test/jdk/javax/swing/text/html/StyleSheet/TestWrongCSSFontSize.java line 119: > 117: > 118: public static void main(String[] args) throws Throwable { > 119: bug8257665 test = new bug8257665(); I guess this needs to be changed now that the testname is modified. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From psadhukhan at openjdk.java.net Fri Jan 15 15:50:05 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 15 Jan 2021 15:50:05 GMT Subject: RFR: 8202880: Test javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java fails [v2] In-Reply-To: References: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> Message-ID: On Wed, 13 Jan 2021 15:45:22 GMT, Alexey Ivanov wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: >> >> Address review comments > > Copyright year in the test and in the ProblemList.txt, since it exists there, should be updated. @aivanov-jdk any more comment on this? ------------- PR: https://git.openjdk.java.net/jdk/pull/2061 From aivanov at openjdk.java.net Fri Jan 15 16:37:15 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Fri, 15 Jan 2021 16:37:15 GMT Subject: RFR: 8202880: Test javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java fails [v2] In-Reply-To: <7UBZ96Nh6YdPKEF0C06wkbrPdLLHSeKLgdEf-i4yHrc=.52da70b4-7c8e-4f6d-9fc3-1637e509e34e@github.com> References: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> <7UBZ96Nh6YdPKEF0C06wkbrPdLLHSeKLgdEf-i4yHrc=.52da70b4-7c8e-4f6d-9fc3-1637e509e34e@github.com> Message-ID: On Fri, 15 Jan 2021 06:33:19 GMT, Prasanta Sadhukhan wrote: >> This test was unstable in linux in nightly testing. Modified to move the frame to center of screen so that the left-taskbar of linux doesn't interfere with the mouse movement along with delay after frame is visible to make it more stable. >> Mach5 job running for several iterations on all platforms is ok. Link in JBS. > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Address review comments Changes requested by aivanov (Reviewer). test/jdk/javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java line 93: > 91: JMenuItem delete; > 92: JMenuItem selectAll; > 93: JMenuBar menuBar; Why at the top of the method instead of the first usage? What about `undo`, `redo`, `cut`, `copy`? I cannot see they're used anywhere else but this method too. test/jdk/javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java line 80: > 78: robot.mouseWheel(1); > 79: robot.waitForIdle(); > 80: if (!menu.isPopupMenuVisible()) { This still calls `menu.isPopupMenuVisible()` on main thread, doesn't it? ------------- PR: https://git.openjdk.java.net/jdk/pull/2061 From psadhukhan at openjdk.java.net Fri Jan 15 17:06:22 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 15 Jan 2021 17:06:22 GMT Subject: RFR: 8202880: Test javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java fails [v3] In-Reply-To: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> References: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> Message-ID: > This test was unstable in linux in nightly testing. Modified to move the frame to center of screen so that the left-taskbar of linux doesn't interfere with the mouse movement along with delay after frame is visible to make it more stable. > Mach5 job running for several iterations on all platforms is ok. Link in JBS. Prasanta Sadhukhan has updated the pull request incrementally with three additional commits since the last revision: - Review fix - Revert "Address review comments" This reverts commit cb12e5bf447b0922fe601c11bbceb2200904b1c3. - Address review comments ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/2061/files - new: https://git.openjdk.java.net/jdk/pull/2061/files/bc7ef81f..9f3bbbf6 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2061&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2061&range=01-02 Stats: 23 lines in 1 file changed: 5 ins; 9 del; 9 mod Patch: https://git.openjdk.java.net/jdk/pull/2061.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2061/head:pull/2061 PR: https://git.openjdk.java.net/jdk/pull/2061 From github.com+1247730+stanio at openjdk.java.net Fri Jan 15 17:54:26 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Fri, 15 Jan 2021 17:54:26 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v3] In-Reply-To: References: Message-ID: > Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. > > _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. > > **Current behavior** > > > > > >

Foo

> >
Bar
> >
    >
  1. Baz
  2. >
> > > > > >
Qux
> > > > **Expected behavior** > > All text should be displayed with a font size of the computed `` font-size ? 1.5. > > [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: fixup! 8257664: Fix font-size inheritance with percentage values - Correct test type/ctor name in main() - Add program option to save image capture of the editor ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1759/files - new: https://git.openjdk.java.net/jdk/pull/1759/files/c5b6d41c..117e01ff Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1759&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1759&range=01-02 Stats: 28 lines in 1 file changed: 24 ins; 2 del; 2 mod Patch: https://git.openjdk.java.net/jdk/pull/1759.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1759/head:pull/1759 PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Fri Jan 15 17:58:03 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Fri, 15 Jan 2021 17:58:03 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes In-Reply-To: <6gfK8xPDLEjtARTMW97tt3ud4gGuh8F9Kx8IOiFoIJk=.f17715d2-f11f-49f8-b2ab-e43bc4a4a445@github.com> References: <9BOPWTWFGoZdPM6XZhrIsehHu7FagIIAkzpsIR6udcc=.c3a81f47-68de-4905-b7bc-ff7a829958c9@github.com> <8I8iiGtAjkh4TPzYWZS4gzpdXaIenmX7plLzcR_0OYo=.843ede15-987e-4ea6-9c14-184f64a39cdc@github.com> <9lvhXeoEpJbEESHDc_q4NOXZ6aAzD4lgkCDT0hJ2WsQ=.4a460617-19e2-4787-80f1-697a2ad3f528@github.com> <6gfK8xPDLEjtARTMW97tt3ud4gGuh8F9Kx8IOiFoIJk=.f17715d2-f11f-49f8-b2ab-e43bc4a4a445@github.com> Message-ID: On Fri, 15 Jan 2021 13:21:29 GMT, Alexey Ivanov wrote: >> Thank you. Your OCA has been processed and your account has been marked as verified. > > Providing a way to display the rendered HTML, especially when the test fails, for visual inspection could be a bonus. This feature could be activated with a parameter passed to the test. It's not a requirement, just a suggestion. Added an option to save editor image capture to a file. Here's the current (wrong) output: ![wrongCssFontSize](https://user-images.githubusercontent.com/1247730/104761222-3f4c1500-576b-11eb-8ff4-cc67e5d341e2.png) All text should use the same font size ? here's a side-by-side (wrong-right) comparison using the workaround I've referenced in the PR description: ![wrong-right](https://user-images.githubusercontent.com/1247730/104761383-7f12fc80-576b-11eb-95ba-68a0edf8e60a.png) ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Fri Jan 15 18:07:25 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Fri, 15 Jan 2021 18:07:25 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v4] In-Reply-To: References: <4kOQwc_o73RnC2NkOs5V_74VJf_NaQG_gCGutpR0pZ4=.8307e31e-9ca8-4427-a859-9dc3b0063b0e@github.com> Message-ID: On Fri, 15 Jan 2021 15:46:14 GMT, Prasanta Sadhukhan wrote: >> I've used `SwingUtilities.invokeAndWait` just to be on the safe side that "everything Swing should be executed on the EDT" but I guess I could drop it as no UI ever shown? > > I am not sure that if we use invokeAndWait(), it needs to be headful..There are many test where we use invokeAndWait but they are headless. But it will be good to see what the test is doing visually so you probably can make it headful. Please, see if the option I've added to save an image capture of the editor component appears sufficient for the visual inspection. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Fri Jan 15 18:11:05 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Fri, 15 Jan 2021 18:11:05 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v4] In-Reply-To: References: Message-ID: On Fri, 15 Jan 2021 12:53:43 GMT, Stanimir Stamenkov wrote: >> src/java.desktop/share/classes/javax/swing/text/html/StyleSheet.java line 3423: >> >>> 3421: */ >>> 3422: static final Object FONT_SIZE_INHERIT = >>> 3423: new CSS().new FontSize().parseCssValue("100%"); >> >> Do we need to instantiate new CSS object here? Can't we reuse "css" variable which is already initialized? > > I hadn't previously realized the `ViewAttributeSet` class is not static, so it should be possible to reuse the instance `css`. Not sure if a separate instance per style sheet is necessary, but I'll prepare a revision. I have now made it use the instance `StyleSheet.css` variable. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Fri Jan 15 18:07:25 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Fri, 15 Jan 2021 18:07:25 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v4] In-Reply-To: References: Message-ID: > Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. > > _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. > > **Current behavior** > > > > > >

Foo

> >
Bar
> >
    >
  1. Baz
  2. >
> > > > > >
Qux
> > > > **Expected behavior** > > All text should be displayed with a font size of the computed `` font-size ? 1.5. > > [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: fixup! 8257664: Fix font-size inheritance with percentage values Init the "fontSizeInherit" value using the instance "StyleSheet.css". ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1759/files - new: https://git.openjdk.java.net/jdk/pull/1759/files/117e01ff..497b9556 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1759&range=03 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1759&range=02-03 Stats: 25 lines in 1 file changed: 15 ins; 9 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/1759.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1759/head:pull/1759 PR: https://git.openjdk.java.net/jdk/pull/1759 From aivanov at openjdk.java.net Fri Jan 15 20:09:06 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Fri, 15 Jan 2021 20:09:06 GMT Subject: RFR: 8202880: Test javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java fails [v3] In-Reply-To: References: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> Message-ID: <2DEBY1PTfoWULLp7Kua2C889eG7-IhRFg1Pob9fFBl0=.b12510ca-928d-4e93-b9ee-c8df26535560@github.com> On Fri, 15 Jan 2021 17:06:22 GMT, Prasanta Sadhukhan wrote: >> This test was unstable in linux in nightly testing. Modified to move the frame to center of screen so that the left-taskbar of linux doesn't interfere with the mouse movement along with delay after frame is visible to make it more stable. >> Mach5 job running for several iterations on all platforms is ok. Link in JBS. > > Prasanta Sadhukhan has updated the pull request incrementally with three additional commits since the last revision: > > - Review fix > - Revert "Address review comments" > > This reverts commit cb12e5bf447b0922fe601c11bbceb2200904b1c3. > - Address review comments Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2061 From aivanov at openjdk.java.net Sat Jan 16 00:21:07 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Sat, 16 Jan 2021 00:21:07 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v4] In-Reply-To: References: Message-ID: On Fri, 15 Jan 2021 18:07:25 GMT, Stanimir Stamenkov wrote: >> Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. >> >> _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. >> >> **Current behavior** >> >> >> >> >> >>

Foo

>> >>
Bar
>> >>
    >>
  1. Baz
  2. >>
>> >> >> >> >> >>
Qux
>> >> >> >> **Expected behavior** >> >> All text should be displayed with a font size of the computed `` font-size ? 1.5. >> >> [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 > > Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: > > fixup! 8257664: Fix font-size inheritance with percentage values > > Init the "fontSizeInherit" value using the instance "StyleSheet.css". Looks good to me. The test runs successfully on headless hosts in mach5. As for the visual inspection, I thought about creating a frame with JEditorPane but saving an image is will also work on headless systems. That is to say, saving the image is even better. src/java.desktop/share/classes/javax/swing/text/html/StyleSheet.java line 1737: > 1735: * property values, Cascading, and Inheritance > 1736: */ > 1737: Object fontSizeInherit() { The method can be private, right? ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From serb at openjdk.java.net Sat Jan 16 00:45:03 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Sat, 16 Jan 2021 00:45:03 GMT Subject: RFR: JDK-8259818: [TEST_BUG] Convert applet-based test test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java to a regular java In-Reply-To: References: Message-ID: On Fri, 15 Jan 2021 07:59:24 GMT, K Suman Rajkumaar wrote: > Dear All, > > Hi All, Could you please review this fix for JDK17? > > Problem Description: The test open/test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java is applet based. > > Fix: Rewritten the above applet based test as a regular java test. > > Best Regards, > K Suman Rajkumaar test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java line 50: > 48: * @bug 8032667 > 49: * @summary [macosx] Components cannot be rendered in HiDPI to BufferedImage > 50: * @run main bug8032667 I suggest do not convert manual tests from the applet to the regular tests, until we will not find a way to unify them, in a similar way as it was for "applet/manual=yesno". Otherwise, all our manual tests will look differently, since different people will make the UI differently. ------------- PR: https://git.openjdk.java.net/jdk/pull/2094 From naoto at openjdk.java.net Sat Jan 16 00:51:09 2021 From: naoto at openjdk.java.net (Naoto Sato) Date: Sat, 16 Jan 2021 00:51:09 GMT Subject: [jdk16] RFR: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 [v2] In-Reply-To: <1Bb8sf6zFDnQSM5kRQIlrgBNwEpO-IvxsPmN05F0QFs=.13eb3035-71d5-4310-98e5-d9989e51cf60@github.com> References: <1Bb8sf6zFDnQSM5kRQIlrgBNwEpO-IvxsPmN05F0QFs=.13eb3035-71d5-4310-98e5-d9989e51cf60@github.com> Message-ID: On Fri, 15 Jan 2021 01:59:07 GMT, Leo Jiang wrote: >> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/standard.properties line 518: >> >>> 516: doclet.footer_specified=\ >>> 517: The -footer option is no longer supported and will be ignored.\n\ >>> 518: It may be removed in a future release. >> >> I believe this is to fix no newline at the end (unrelated to l10n changes). Do we need to change the copyright year for this? > > Actually I was correcting the L217, changed {) -> {}. But 2 days ago, Jonathan Gibbons fixed it in another commit 6bb6093. I found his fix after running the merge. Looks both of us forgot to update the copyright year. Any suggestion? > doclet.help.systemProperties.body=\ > The {0} page lists references to system properties. I believe your PR still contains the fix to add the newline at the end of the file (at L518). So I think you can simply change the copyright year in `standard.properties` file. ------------- PR: https://git.openjdk.java.net/jdk16/pull/123 From github.com+1247730+stanio at openjdk.java.net Sat Jan 16 06:45:35 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Sat, 16 Jan 2021 06:45:35 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v5] In-Reply-To: References: Message-ID: > Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. > > _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. > > **Current behavior** > > > > > >

Foo

> >
Bar
> >
    >
  1. Baz
  2. >
> > > > > >
Qux
> > > > **Expected behavior** > > All text should be displayed with a font size of the computed `` font-size ? 1.5. > > [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: fixup! 8257664: Fix font-size inheritance with percentage values Declare fontSizeInherit() accessor private. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1759/files - new: https://git.openjdk.java.net/jdk/pull/1759/files/497b9556..10a61015 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1759&range=04 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1759&range=03-04 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/1759.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1759/head:pull/1759 PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Sat Jan 16 06:56:16 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Sat, 16 Jan 2021 06:56:16 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v4] In-Reply-To: References: Message-ID: On Fri, 15 Jan 2021 23:27:33 GMT, Alexey Ivanov wrote: >> Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: >> >> fixup! 8257664: Fix font-size inheritance with percentage values >> >> Init the "fontSizeInherit" value using the instance "StyleSheet.css". > > src/java.desktop/share/classes/javax/swing/text/html/StyleSheet.java line 1737: > >> 1735: * property values, Cascading, and Inheritance >> 1736: */ >> 1737: Object fontSizeInherit() { > > The method can be private, right? Made it private now. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From kizune at openjdk.java.net Sun Jan 17 09:32:59 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Sun, 17 Jan 2021 09:32:59 GMT Subject: RFR: 8259650: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" Message-ID: 8259650: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" ------------- Commit messages: - 8259650: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" - Merge remote-tracking branch 'upstream/master' into master - Merge remote-tracking branch 'upstream/master' into master - Merge branch 'openjdk-master' into master - Merge branch 'master' of https://github.com/openjdk/jdk into openjdk-master - Merge pull request #7 from openjdk/master - Merge pull request #6 from openjdk/master - Merge pull request #5 from openjdk/master - Merge pull request #4 from openjdk/master - Merge pull request #3 from openjdk/master - ... and 2 more: https://git.openjdk.java.net/jdk/compare/da4cf05d...c20f311a Changes: https://git.openjdk.java.net/jdk/pull/2114/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2114&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8259650 Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/2114.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2114/head:pull/2114 PR: https://git.openjdk.java.net/jdk/pull/2114 From kizune at openjdk.java.net Sun Jan 17 10:45:53 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Sun, 17 Jan 2021 10:45:53 GMT Subject: [jdk16] RFR: 8258643: [TESTBUG] javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" [v2] In-Reply-To: References: Message-ID: <8D6d0EmbNSd5W17D6rVGTZ8nNBLSsh9RoFnA1ReYa4Y=.72a667c2-41ab-4d65-97d7-44556bdfba2e@github.com> > Backport from mainline. Alexander Zuev has updated the pull request incrementally with one additional commit since the last revision: Added 1s delay before taking the first screenshot to give slow systems enough time to finish painting. ------------- Changes: - all: https://git.openjdk.java.net/jdk16/pull/119/files - new: https://git.openjdk.java.net/jdk16/pull/119/files/c3b4e3bf..24bb1c8e Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk16&pr=119&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk16&pr=119&range=00-01 Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jdk16/pull/119.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/119/head:pull/119 PR: https://git.openjdk.java.net/jdk16/pull/119 From jdv at openjdk.java.net Mon Jan 18 03:10:36 2021 From: jdv at openjdk.java.net (Jayathirth D V) Date: Mon, 18 Jan 2021 03:10:36 GMT Subject: RFR: 8259650: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" In-Reply-To: References: Message-ID: <2gSe0GmiUQzjFoO3tVCwhohxonAvopmArDMy9dYXTS4=.bf0bc534-b6ca-46b7-8ad7-b6fa0e24291f@github.com> On Sun, 17 Jan 2021 09:26:52 GMT, Alexander Zuev wrote: > 8259650: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" Marked as reviewed by jdv (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2114 From psadhukhan at openjdk.java.net Mon Jan 18 04:54:39 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 18 Jan 2021 04:54:39 GMT Subject: RFR: 8259650: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" In-Reply-To: References: Message-ID: On Sun, 17 Jan 2021 09:26:52 GMT, Alexander Zuev wrote: > 8259650: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" test/jdk/javax/swing/JComponent/7154030/bug7154030.java line 96: > 94: robot.waitForIdle(1000); > 95: > 96: Rectangle bounds = frame.getBounds(); Iguess we missed putting this getBounds and getInsets call in EDT. Maybe that thread context switch can make up for the delay that is introduced now as we already have a 1s wait before screen capture. ------------- PR: https://git.openjdk.java.net/jdk/pull/2114 From ljiang at openjdk.java.net Mon Jan 18 05:52:57 2021 From: ljiang at openjdk.java.net (Leo Jiang) Date: Mon, 18 Jan 2021 05:52:57 GMT Subject: [jdk16] RFR: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 [v3] In-Reply-To: References: Message-ID: > This is the changes for JDK 16 msg drop 10. Leo Jiang has updated the pull request incrementally with one additional commit since the last revision: fix the missing copyright year for standard.properties ------------- Changes: - all: https://git.openjdk.java.net/jdk16/pull/123/files - new: https://git.openjdk.java.net/jdk16/pull/123/files/d72f444a..9c7574e2 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk16&pr=123&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk16&pr=123&range=01-02 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk16/pull/123.diff Fetch: git fetch https://git.openjdk.java.net/jdk16 pull/123/head:pull/123 PR: https://git.openjdk.java.net/jdk16/pull/123 From ljiang at openjdk.java.net Mon Jan 18 05:52:57 2021 From: ljiang at openjdk.java.net (Leo Jiang) Date: Mon, 18 Jan 2021 05:52:57 GMT Subject: [jdk16] RFR: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 [v2] In-Reply-To: References: <1Bb8sf6zFDnQSM5kRQIlrgBNwEpO-IvxsPmN05F0QFs=.13eb3035-71d5-4310-98e5-d9989e51cf60@github.com> Message-ID: On Sat, 16 Jan 2021 00:48:43 GMT, Naoto Sato wrote: >> Actually I was correcting the L217, changed {) -> {}. But 2 days ago, Jonathan Gibbons fixed it in another commit 6bb6093. I found his fix after running the merge. Looks both of us forgot to update the copyright year. Any suggestion? >> doclet.help.systemProperties.body=\ >> The {0} page lists references to system properties. > > I believe your PR still contains the fix to add the newline at the end of the file (at L518). So I think you can simply change the copyright year in `standard.properties` file. Thx! Fixed the copyright year for this file. ------------- PR: https://git.openjdk.java.net/jdk16/pull/123 From psadhukhan at openjdk.java.net Mon Jan 18 07:23:37 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 18 Jan 2021 07:23:37 GMT Subject: Integrated: 8202880: Test javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java fails In-Reply-To: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> References: <6jy3ChTwGm7xXWBrZYtD1_KSLgUdvAy69BrKhH_q958=.1b0b51a9-3f50-4352-85bd-c3d440dff2a4@github.com> Message-ID: On Wed, 13 Jan 2021 10:41:29 GMT, Prasanta Sadhukhan wrote: > This test was unstable in linux in nightly testing. Modified to move the frame to center of screen so that the left-taskbar of linux doesn't interfere with the mouse movement along with delay after frame is visible to make it more stable. > Mach5 job running for several iterations on all platforms is ok. Link in JBS. This pull request has now been integrated. Changeset: 3f19ef63 Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/3f19ef63 Stats: 64 lines in 2 files changed: 12 ins; 30 del; 22 mod 8202880: Test javax/swing/JPopupMenu/8075063/ContextMenuScrollTest.java fails Reviewed-by: aivanov ------------- PR: https://git.openjdk.java.net/jdk/pull/2061 From kizune at openjdk.java.net Mon Jan 18 08:26:39 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Mon, 18 Jan 2021 08:26:39 GMT Subject: Integrated: 8259650: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" In-Reply-To: References: Message-ID: <6tDVD3i10MqQjfPYX3xSLyOuyfS6gZmWi8FwYTDmzyg=.c7d3c163-96e0-4281-8b87-5bd24b3db43e@github.com> On Sun, 17 Jan 2021 09:26:52 GMT, Alexander Zuev wrote: > 8259650: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" This pull request has now been integrated. Changeset: 917f7e9a Author: Alexander Zuev URL: https://git.openjdk.java.net/jdk/commit/917f7e9a Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod 8259650: javax/swing/JComponent/7154030/bug7154030.java still fails with "Exception: Failed to hide opaque button" Reviewed-by: jdv ------------- PR: https://git.openjdk.java.net/jdk/pull/2114 From aivanov at openjdk.java.net Mon Jan 18 11:33:41 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Mon, 18 Jan 2021 11:33:41 GMT Subject: RFR: JDK-8259818: [TEST_BUG] Convert applet-based test test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java to a regular java In-Reply-To: References: Message-ID: <765zFxcbZfKE4vd3QFpnl2su3ibljAuiNehF_xboaV4=.9a4fc3f7-8728-431c-9960-82ca93d645c2@github.com> On Sat, 16 Jan 2021 00:42:45 GMT, Sergey Bylokhov wrote: >> Dear All, >> >> Hi All, Could you please review this fix for JDK17? >> >> Problem Description: The test open/test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java is applet based. >> >> Fix: Rewritten the above applet based test as a regular java test. >> >> Best Regards, >> K Suman Rajkumaar > > test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java line 50: > >> 48: * @bug 8032667 >> 49: * @summary [macosx] Components cannot be rendered in HiDPI to BufferedImage >> 50: * @run main bug8032667 > > I suggest do not convert manual tests from the applet to the regular tests, until we will not find a way to unify them, in a similar way as it was for "applet/manual=yesno". Otherwise, all our manual tests will look differently, since different people will make the UI differently. Unifying the UI makes sense in the long run. Will it involve retrofitting all the existing manual test which already have its own UI? In most cases, the UI in manual tests is similar: Pass / Fail buttons at the bottom with the test instructions above. Then there are differences? Other parts of the UI depend on the nature of the test. I've always found manual tests a bit confusing: a test, especially applet-based one, opens two windows; however, one window is easier to focus on. ------------- PR: https://git.openjdk.java.net/jdk/pull/2094 From psadhukhan at openjdk.java.net Mon Jan 18 12:14:47 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Mon, 18 Jan 2021 12:14:47 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v5] In-Reply-To: References: Message-ID: On Sat, 16 Jan 2021 06:45:35 GMT, Stanimir Stamenkov wrote: >> Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. >> >> _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. >> >> **Current behavior** >> >> >> >> >> >>

Foo

>> >>
Bar
>> >>
    >>
  1. Baz
  2. >>
>> >> >> >> >> >>
Qux
>> >> >> >> **Expected behavior** >> >> All text should be displayed with a font size of the computed `` font-size ? 1.5. >> >> [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 > > Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: > > fixup! 8257664: Fix font-size inheritance with percentage values > > Declare fontSizeInherit() accessor private. Looks good to me. Internal testing is also ok. However, I have one question, html text is honouring "font-size" value(say 1.5 as per testcase) but if Display resolution in windows system setting or via -Dsun.java2d.uiScale is set to something different(say 1.25), then should it honour the font-size property (1.5) or Display resolution (1.25)? ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From aivanov at openjdk.java.net Mon Jan 18 12:14:48 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Mon, 18 Jan 2021 12:14:48 GMT Subject: RFR: JDK-8259818: [TEST_BUG] Convert applet-based test test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java to a regular java In-Reply-To: <765zFxcbZfKE4vd3QFpnl2su3ibljAuiNehF_xboaV4=.9a4fc3f7-8728-431c-9960-82ca93d645c2@github.com> References: <765zFxcbZfKE4vd3QFpnl2su3ibljAuiNehF_xboaV4=.9a4fc3f7-8728-431c-9960-82ca93d645c2@github.com> Message-ID: <0CkmITP6hhucT5SsWhZzFdsZ1AwxNlFvl8m4p5ZBqhc=.65bd44c0-2a18-4d6a-bd66-1c72f7bb5ada@github.com> On Mon, 18 Jan 2021 11:30:55 GMT, Alexey Ivanov wrote: >> test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java line 50: >> >>> 48: * @bug 8032667 >>> 49: * @summary [macosx] Components cannot be rendered in HiDPI to BufferedImage >>> 50: * @run main bug8032667 >> >> I suggest do not convert manual tests from the applet to the regular tests, until we will not find a way to unify them, in a similar way as it was for "applet/manual=yesno". Otherwise, all our manual tests will look differently, since different people will make the UI differently. > > Unifying the UI makes sense in the long run. Will it involve retrofitting all the existing manual test which already have its own UI? > > In most cases, the UI in manual tests is similar: Pass / Fail buttons at the bottom with the test instructions above. Then there are differences? Other parts of the UI depend on the nature of the test. > > I've always found manual tests a bit confusing: a test, especially applet-based one, opens two windows; however, one window is easier to focus on. To confirm: we're not modifying the current applet-based manual tests until there's a common framework to migrate them off the applet API. Do I understand it correctly? ------------- PR: https://git.openjdk.java.net/jdk/pull/2094 From aivanov at openjdk.java.net Mon Jan 18 12:26:43 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Mon, 18 Jan 2021 12:26:43 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v5] In-Reply-To: References: Message-ID: On Sat, 16 Jan 2021 06:45:35 GMT, Stanimir Stamenkov wrote: >> Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. >> >> _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. >> >> **Current behavior** >> >> >> >> >> >>

Foo

>> >>
Bar
>> >>
    >>
  1. Baz
  2. >>
>> >> >> >> >> >>
Qux
>> >> >> >> **Expected behavior** >> >> All text should be displayed with a font size of the computed `` font-size ? 1.5. >> >> [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 > > Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: > > fixup! 8257664: Fix font-size inheritance with percentage values > > Declare fontSizeInherit() accessor private. Looks good to me. ------------- Marked as reviewed by aivanov (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/1759 From aivanov at openjdk.java.net Mon Jan 18 12:26:44 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Mon, 18 Jan 2021 12:26:44 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v5] In-Reply-To: References: Message-ID: On Mon, 18 Jan 2021 12:11:33 GMT, Prasanta Sadhukhan wrote: >> Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: >> >> fixup! 8257664: Fix font-size inheritance with percentage values >> >> Declare fontSizeInherit() accessor private. > > Looks good to me. Internal testing is also ok. > However, I have one question, html text is honouring "font-size" value(say 1.5 as per testcase) but if Display resolution in windows system setting or via -Dsun.java2d.uiScale is set to something different(say 1.25), then should it honour the font-size property (1.5) or Display resolution (1.25)? Shouldn't UI scaling be applied automatically? The font size in user space should remain the same, yet the actual font size would be scaled up according to the DPI setting. That is in my understanding, CSS rule applies 150% to the base font size according to its rules; then `uiScale` is applied to whatever the computed font size is. Does it make sense to run the test with different `uiScale` explicitly? I ran the test on my laptop with 150% scaling, it passed successfully. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From serb at openjdk.java.net Mon Jan 18 12:30:40 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Mon, 18 Jan 2021 12:30:40 GMT Subject: RFR: JDK-8259818: [TEST_BUG] Convert applet-based test test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java to a regular java In-Reply-To: <0CkmITP6hhucT5SsWhZzFdsZ1AwxNlFvl8m4p5ZBqhc=.65bd44c0-2a18-4d6a-bd66-1c72f7bb5ada@github.com> References: <765zFxcbZfKE4vd3QFpnl2su3ibljAuiNehF_xboaV4=.9a4fc3f7-8728-431c-9960-82ca93d645c2@github.com> <0CkmITP6hhucT5SsWhZzFdsZ1AwxNlFvl8m4p5ZBqhc=.65bd44c0-2a18-4d6a-bd66-1c72f7bb5ada@github.com> Message-ID: On Mon, 18 Jan 2021 12:12:10 GMT, Alexey Ivanov wrote: >> Unifying the UI makes sense in the long run. Will it involve retrofitting all the existing manual test which already have its own UI? >> >> In most cases, the UI in manual tests is similar: Pass / Fail buttons at the bottom with the test instructions above. Then there are differences? Other parts of the UI depend on the nature of the test. >> >> I've always found manual tests a bit confusing: a test, especially applet-based one, opens two windows; however, one window is easier to focus on. > > To confirm: we're not modifying the current applet-based manual tests until there's a common framework to migrate them off the applet API. Do I understand it correctly? Yes, I suggest that. Maybe we can start with that replacement? In the jtreg itself or via separate lib. ------------- PR: https://git.openjdk.java.net/jdk/pull/2094 From aivanov at openjdk.java.net Mon Jan 18 12:34:48 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Mon, 18 Jan 2021 12:34:48 GMT Subject: RFR: JDK-8259818: [TEST_BUG] Convert applet-based test test/jdk/javax/swing/JCheckBox/8032667/bug8032667.java to a regular java In-Reply-To: References: <765zFxcbZfKE4vd3QFpnl2su3ibljAuiNehF_xboaV4=.9a4fc3f7-8728-431c-9960-82ca93d645c2@github.com> <0CkmITP6hhucT5SsWhZzFdsZ1AwxNlFvl8m4p5ZBqhc=.65bd44c0-2a18-4d6a-bd66-1c72f7bb5ada@github.com> Message-ID: On Mon, 18 Jan 2021 12:28:07 GMT, Sergey Bylokhov wrote: >> To confirm: we're not modifying the current applet-based manual tests until there's a common framework to migrate them off the applet API. Do I understand it correctly? > > Yes, I suggest that. Maybe we can start with that replacement? In the jtreg itself or via separate lib. A separate lib seems more convenient as it will allow running such tests without test harness which is quicker and useful for debugging. ------------- PR: https://git.openjdk.java.net/jdk/pull/2094 From github.com+1247730+stanio at openjdk.java.net Mon Jan 18 13:23:50 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Mon, 18 Jan 2021 13:23:50 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v5] In-Reply-To: References: Message-ID: On Mon, 18 Jan 2021 12:23:27 GMT, Alexey Ivanov wrote: >> Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: >> >> fixup! 8257664: Fix font-size inheritance with percentage values >> >> Declare fontSizeInherit() accessor private. > > Looks good to me. The display dpi shouldn't matter. I've tried it in a couple of different monitor configurations: 1.25, 1.5, and 2.0 scaling. I've also tried it with non-dpi-aware Java 8. I've used `body { font-size: 14 }` which should always evaluate to `Font.size=14` (no matter the `w3cLengthUnits` value, for example) ? the other elements use relative to that base font-size. Before I type `/integrate`, is the rebase smart enough to squash the _fixup!_ commits, or should I rebase manually once for that purpose? Maybe it would squash the commits in any case? @aivanov-jdk or @prsadhuk, would you sponsor this PR? ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From aivanov at openjdk.java.net Mon Jan 18 14:25:56 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Mon, 18 Jan 2021 14:25:56 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v5] In-Reply-To: References: Message-ID: On Mon, 18 Jan 2021 12:23:27 GMT, Alexey Ivanov wrote: >> Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: >> >> fixup! 8257664: Fix font-size inheritance with percentage values >> >> Declare fontSizeInherit() accessor private. > > Looks good to me. > Before I type `/integrate`, is the rebase smart enough to squash the _fixup!_ commits, or should I rebase manually once for that purpose? Maybe it would squash the commits in any case? @aivanov-jdk or @prsadhuk, would you sponsor this PR? Yes, your commits in the branch will be squashed into one commit in the upstream. I am ready to sponsor, I think @prsadhuk wouldn't mind to sponsor too. However, @prsadhuk hasn't approved it yet; I'd like to wait until he does so. Thank you @stanio for contributing the fix! ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From kizune at openjdk.java.net Mon Jan 18 22:14:45 2021 From: kizune at openjdk.java.net (Alexander Zuev) Date: Mon, 18 Jan 2021 22:14:45 GMT Subject: [jdk16] Integrated: 8258643: [TESTBUG] javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" In-Reply-To: References: Message-ID: On Wed, 13 Jan 2021 22:26:47 GMT, Alexander Zuev wrote: > Backport from mainline. This pull request has now been integrated. Changeset: bb0821eb Author: Alexander Zuev URL: https://git.openjdk.java.net/jdk16/commit/bb0821eb Stats: 13 lines in 1 file changed: 2 ins; 5 del; 6 mod 8258643: [TESTBUG] javax/swing/JComponent/7154030/bug7154030.java failed with "Exception: Failed to hide opaque button" Reviewed-by: trebari, psadhukhan ------------- PR: https://git.openjdk.java.net/jdk16/pull/119 From trebari at openjdk.java.net Tue Jan 19 14:33:52 2021 From: trebari at openjdk.java.net (Tejpal Rebari) Date: Tue, 19 Jan 2021 14:33:52 GMT Subject: RFR: 8049700: Deprecate obsolete classes and methods in javax/swing/plaf/basic In-Reply-To: References: Message-ID: On Wed, 13 Jan 2021 05:52:49 GMT, Prasanta Sadhukhan wrote: >> Please review the following fix for jdk17. >> In this fix i have deprecated and marked for removal following classes and methods >> public void intervalAdded(ListDataEvent e) >> public void intervalRemoved(ListDataEvent e) >> protected boolean lt(File a, File b) in BasicDirectoryModel.java >> >> inner class PropertyChangeHandler, VSBChangeListener, HSBChangeListener, >> ViewportChangeHandler in BasicScrollPaneUI.java >> inner class MouseInputHandler in BasicMenuItemUI.java >> method BasicToolBarUI.java#createFloatingFrame >> >> From 8049700 not deprecated the paintText(Graphics g, JComponent c, Rectangle textRect, String text) method in BasicButtonUI as AquaButtonUI, MetalButtonUI and MetalToggleButtonUI overrides it. >> Similarly not deprecated ChangeHandler of BasicMenuUI as AquaMenuUI and MotifMenuUI uses this class. > > Please elaborate as to why this methods are to be deprecated. > It will be useful if you give alternate methods to be used in the javadoc in @deprecated tag > which are supposed to be called by user once these are removed. The methods intervalAdded(ListDataEvent e) ,intervalRemoved(ListDataEvent e) and lt(File a, File b) of javax/swing/plaf/basic/BasicDirectoryModel.java states that "Obsolete - not used" ( in the doc). The BasicDirectoryModel uses similar methods like fireIntervalAdded, fireIntervalRemoved which calls AbstractListModel#fireIntervalAdded. But not sure that these are the alternate methods.Also dont see anything similar to lt(File a, File b). The method createFloatingFrame in the BasicToolBarUI.java states that it is "No longer used" and also specifies to use BasicToolBarUI.createFloatingWindow(JToolBar). The class MouseInputHandler in BasicMenuUI.java and classes PropertyChangeHandler, VSBChangeListener, HSBChangeListener, ViewportChangeHandler in BasicScrollPaneUI.java states that(as comments inside the class) "This class exists only for backward compatibility. All its functionality has been moved into Handler." we can add the above in the doc for these classes. ------------- PR: https://git.openjdk.java.net/jdk/pull/1958 From trebari at openjdk.java.net Tue Jan 19 14:40:07 2021 From: trebari at openjdk.java.net (Tejpal Rebari) Date: Tue, 19 Jan 2021 14:40:07 GMT Subject: RFR: 8049700: Deprecate obsolete classes and methods in javax/swing/plaf/basic [v2] In-Reply-To: References: Message-ID: > Please review the following fix for jdk17. > In this fix i have deprecated and marked for removal following classes and methods > public void intervalAdded(ListDataEvent e) > public void intervalRemoved(ListDataEvent e) > protected boolean lt(File a, File b) in BasicDirectoryModel.java > > inner class PropertyChangeHandler, VSBChangeListener, HSBChangeListener, > ViewportChangeHandler in BasicScrollPaneUI.java > inner class MouseInputHandler in BasicMenuItemUI.java > method BasicToolBarUI.java#createFloatingFrame > > From 8049700 not deprecated the paintText(Graphics g, JComponent c, Rectangle textRect, String text) method in BasicButtonUI as AquaButtonUI, MetalButtonUI and MetalToggleButtonUI overrides it. > Similarly not deprecated ChangeHandler of BasicMenuUI as AquaMenuUI and MotifMenuUI uses this class. Tejpal Rebari has updated the pull request incrementally with one additional commit since the last revision: minor correction ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1958/files - new: https://git.openjdk.java.net/jdk/pull/1958/files/8504fc7c..9bdb67a3 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1958&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1958&range=00-01 Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod Patch: https://git.openjdk.java.net/jdk/pull/1958.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1958/head:pull/1958 PR: https://git.openjdk.java.net/jdk/pull/1958 From naoto at openjdk.java.net Tue Jan 19 16:55:46 2021 From: naoto at openjdk.java.net (Naoto Sato) Date: Tue, 19 Jan 2021 16:55:46 GMT Subject: [jdk16] RFR: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 [v3] In-Reply-To: References: Message-ID: On Mon, 18 Jan 2021 05:52:57 GMT, Leo Jiang wrote: >> This is the changes for JDK 16 msg drop 10. > > Leo Jiang has updated the pull request incrementally with one additional commit since the last revision: > > fix the missing copyright year for standard.properties Marked as reviewed by naoto (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk16/pull/123 From psadhukhan at openjdk.java.net Wed Jan 20 04:49:52 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 20 Jan 2021 04:49:52 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v5] In-Reply-To: References: Message-ID: On Sat, 16 Jan 2021 06:45:35 GMT, Stanimir Stamenkov wrote: >> Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. >> >> _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. >> >> **Current behavior** >> >> >> >> >> >>

Foo

>> >>
Bar
>> >>
    >>
  1. Baz
  2. >>
>> >> >> >> >> >>
Qux
>> >> >> >> **Expected behavior** >> >> All text should be displayed with a font size of the computed `` font-size ? 1.5. >> >> [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 > > Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: > > fixup! 8257664: Fix font-size inheritance with percentage values > > Declare fontSizeInherit() accessor private. Marked as reviewed by psadhukhan (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From psadhukhan at openjdk.java.net Wed Jan 20 04:49:53 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 20 Jan 2021 04:49:53 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v5] In-Reply-To: References: Message-ID: On Mon, 18 Jan 2021 13:21:18 GMT, Stanimir Stamenkov wrote: > > > The display dpi shouldn't matter. I've tried it in a couple of different monitor configurations: 1.25, 1.5, and 2.0 scaling. I've also tried it with non-dpi-aware Java 8. I've used `body { font-size: 14 }` which should always evaluate to `Font.size=14` (no matter the `w3cLengthUnits` value, for example) ? the other elements use relative to that base font-size. OK. Good to know dpi factor is not affecting this usecase, but speaking of `w3cLengthUnits` value, it seems there is another JBS issue https://bugs.openjdk.java.net/browse/JDK-8231286 where "font-size" value with this w3cLengthUnits attribute set is affected by the hidpi factor, so was wondering if this fix will have any impact on that issue as well, but it seems it's not and need something more to be done. Anyway, this is good to go. ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From psadhukhan at openjdk.java.net Wed Jan 20 11:58:00 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 20 Jan 2021 11:58:00 GMT Subject: RFR: 8260035: Deproblemlist few problemlisted test Message-ID: <4rZCe2nLn47Y02BI7JNSuQSFsfj7ZK7ytsymrihoY48=.bc3879ee-bda4-4b4b-b577-3c3560d51832@github.com> Few problemlisted test javax/swing/plaf/basic/Test6984643.java javax/swing/JMenuItem/6249972/bug6249972.java javax/swing/JTree/6263446/bug6263446.java were unstable in mach5 nightly testing. These can be deproblemlisted now with appropriate stability fixes. Mach5 jobs running for several iterations for all these tests in all platforms is ok. Link in JBS. ------------- Commit messages: - 8260035: Deproblemlist few problemlisted test Changes: https://git.openjdk.java.net/jdk/pull/2161/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2161&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8260035 Stats: 114 lines in 4 files changed: 29 ins; 9 del; 76 mod Patch: https://git.openjdk.java.net/jdk/pull/2161.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2161/head:pull/2161 PR: https://git.openjdk.java.net/jdk/pull/2161 From github.com+1247730+stanio at openjdk.java.net Wed Jan 20 13:27:57 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Wed, 20 Jan 2021 13:27:57 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v5] In-Reply-To: References: Message-ID: On Wed, 20 Jan 2021 04:47:09 GMT, Prasanta Sadhukhan wrote: >> Stanimir Stamenkov has updated the pull request incrementally with one additional commit since the last revision: >> >> fixup! 8257664: Fix font-size inheritance with percentage values >> >> Declare fontSizeInherit() accessor private. > > Marked as reviewed by psadhukhan (Reviewer). Thank you @prsadhuk for reviewing this. Should/could I go with `/integrate` now? ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From psadhukhan at openjdk.java.net Wed Jan 20 13:27:58 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 20 Jan 2021 13:27:58 GMT Subject: RFR: 8257664: HTMLEditorKit: Wrong CSS relative font sizes [v5] In-Reply-To: References: Message-ID: On Wed, 20 Jan 2021 13:24:36 GMT, Stanimir Stamenkov wrote: > > > Thank you @prsadhuk for reviewing this. Should/could I go with `/integrate` now? yes sure ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From github.com+1247730+stanio at openjdk.java.net Wed Jan 20 13:39:00 2021 From: github.com+1247730+stanio at openjdk.java.net (Stanimir Stamenkov) Date: Wed, 20 Jan 2021 13:39:00 GMT Subject: Integrated: 8257664: HTMLEditorKit: Wrong CSS relative font sizes In-Reply-To: References: Message-ID: On Sun, 13 Dec 2020 14:51:47 GMT, Stanimir Stamenkov wrote: > Fix for [JDK-8257664][] ? HTMLEditorKit: Wrong CSS relative font sizes. > > _Disclaimer:_ I'm the reporter of the issue and I've been advised the best chance to get it addressed is to submit a pull request against this repository. I haven't built the JDK myself, I'll need guidance if required. I have a proof-of-concept example ? demonstrating the bug and a workaround available as a [public gist](https://gist.github.com/stanio/b79ce9348946aa6b3395328ec4c59d14). I have included a sample test though I don't know if it is annotated properly. > > **Current behavior** > > > > > >

Foo

> >
Bar
> >
    >
  1. Baz
  2. >
> > > > > >
Qux
> > > > **Expected behavior** > > All text should be displayed with a font size of the computed `` font-size ? 1.5. > > [JDK-8257664]: https://bugs.openjdk.java.net/browse/JDK-8257664 This pull request has now been integrated. Changeset: 70b5b311 Author: Stanimir Stamenkov Committer: Alexey Ivanov URL: https://git.openjdk.java.net/jdk/commit/70b5b311 Stats: 178 lines in 2 files changed: 178 ins; 0 del; 0 mod 8257664: HTMLEditorKit: Wrong CSS relative font sizes Reviewed-by: aivanov, psadhukhan ------------- PR: https://git.openjdk.java.net/jdk/pull/1759 From ljiang at openjdk.java.net Wed Jan 20 14:01:48 2021 From: ljiang at openjdk.java.net (Leo Jiang) Date: Wed, 20 Jan 2021 14:01:48 GMT Subject: [jdk16] Integrated: JDK-8259732: JDK 16 L10n resource file update - msg drop 10 In-Reply-To: References: Message-ID: On Thu, 14 Jan 2021 14:00:00 GMT, Leo Jiang wrote: > This is the changes for JDK 16 msg drop 10. This pull request has now been integrated. Changeset: 01205109 Author: Leo Jiang URL: https://git.openjdk.java.net/jdk16/commit/01205109 Stats: 215 lines in 30 files changed: 118 ins; 16 del; 81 mod 8259732: JDK 16 L10n resource file update - msg drop 10 Reviewed-by: naoto ------------- PR: https://git.openjdk.java.net/jdk16/pull/123 From jwilhelm at openjdk.java.net Thu Jan 21 04:41:15 2021 From: jwilhelm at openjdk.java.net (Jesper Wilhelmsson) Date: Thu, 21 Jan 2021 04:41:15 GMT Subject: RFR: Merge jdk16 Message-ID: Forwardport JDK 16 -> JDK 17 ------------- Commit messages: - Merge - 8259757: add a regression test for 8259353 and 8259601 - 8259732: JDK 16 L10n resource file update - msg drop 10 The webrevs contain the adjustments done while merging with regards to each parent branch: - master: https://webrevs.openjdk.java.net/?repo=jdk&pr=2176&range=00.0 - jdk16: https://webrevs.openjdk.java.net/?repo=jdk&pr=2176&range=00.1 Changes: https://git.openjdk.java.net/jdk/pull/2176/files Stats: 296 lines in 31 files changed: 200 ins; 16 del; 80 mod Patch: https://git.openjdk.java.net/jdk/pull/2176.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2176/head:pull/2176 PR: https://git.openjdk.java.net/jdk/pull/2176 From jwilhelm at openjdk.java.net Thu Jan 21 05:26:59 2021 From: jwilhelm at openjdk.java.net (Jesper Wilhelmsson) Date: Thu, 21 Jan 2021 05:26:59 GMT Subject: Integrated: Merge jdk16 In-Reply-To: References: Message-ID: <9sHQ5RFKu7eMA2dnKWQ9ZWXPzT_HnSJdXDZ2j4kX8to=.69bf5258-7371-4404-a1af-b54febf7cad7@github.com> On Thu, 21 Jan 2021 04:33:47 GMT, Jesper Wilhelmsson wrote: > Forwardport JDK 16 -> JDK 17 This pull request has now been integrated. Changeset: 133bcb09 Author: Jesper Wilhelmsson URL: https://git.openjdk.java.net/jdk/commit/133bcb09 Stats: 296 lines in 31 files changed: 200 ins; 16 del; 80 mod Merge ------------- PR: https://git.openjdk.java.net/jdk/pull/2176 From jdv at openjdk.java.net Thu Jan 21 07:21:57 2021 From: jdv at openjdk.java.net (Jayathirth D V) Date: Thu, 21 Jan 2021 07:21:57 GMT Subject: RFR: 8260035: Deproblemlist few problemlisted test In-Reply-To: <4rZCe2nLn47Y02BI7JNSuQSFsfj7ZK7ytsymrihoY48=.bc3879ee-bda4-4b4b-b577-3c3560d51832@github.com> References: <4rZCe2nLn47Y02BI7JNSuQSFsfj7ZK7ytsymrihoY48=.bc3879ee-bda4-4b4b-b577-3c3560d51832@github.com> Message-ID: <6oI4cnAONyOuxAmoPjp_rkZU5IsA7tQjwhElxBxrWK4=.84954094-f156-4d03-a496-9f6a5930b5d0@github.com> On Wed, 20 Jan 2021 11:49:25 GMT, Prasanta Sadhukhan wrote: > Few problemlisted test > javax/swing/plaf/basic/Test6984643.java > javax/swing/JMenuItem/6249972/bug6249972.java > javax/swing/JTree/6263446/bug6263446.java > were unstable in mach5 nightly testing. These can be deproblemlisted now with appropriate stability fixes. > Mach5 jobs running for several iterations for all these tests in all platforms is ok. Link in JBS. Marked as reviewed by jdv (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2161 From serb at openjdk.java.net Thu Jan 21 07:42:57 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Thu, 21 Jan 2021 07:42:57 GMT Subject: RFR: 8260035: Deproblemlist few problemlisted test In-Reply-To: <6oI4cnAONyOuxAmoPjp_rkZU5IsA7tQjwhElxBxrWK4=.84954094-f156-4d03-a496-9f6a5930b5d0@github.com> References: <4rZCe2nLn47Y02BI7JNSuQSFsfj7ZK7ytsymrihoY48=.bc3879ee-bda4-4b4b-b577-3c3560d51832@github.com> <6oI4cnAONyOuxAmoPjp_rkZU5IsA7tQjwhElxBxrWK4=.84954094-f156-4d03-a496-9f6a5930b5d0@github.com> Message-ID: <3DjZv6sTdGVPfpA_0Pg0Or8gwarXxkHHbwrSdAfHpwk=.63294614-29f0-4a0f-94fb-ab970f57fb6a@github.com> On Thu, 21 Jan 2021 07:19:10 GMT, Jayathirth D V wrote: >> Few problemlisted test >> javax/swing/plaf/basic/Test6984643.java >> javax/swing/JMenuItem/6249972/bug6249972.java >> javax/swing/JTree/6263446/bug6263446.java >> were unstable in mach5 nightly testing. These can be deproblemlisted now with appropriate stability fixes. >> Mach5 jobs running for several iterations for all these tests in all platforms is ok. Link in JBS. > > Marked as reviewed by jdv (Reviewer). Does the JDK-8198340 bug is a test issue? ------------- PR: https://git.openjdk.java.net/jdk/pull/2161 From psadhukhan at openjdk.java.net Thu Jan 21 07:45:55 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Thu, 21 Jan 2021 07:45:55 GMT Subject: RFR: 8260035: Deproblemlist few problemlisted test In-Reply-To: <3DjZv6sTdGVPfpA_0Pg0Or8gwarXxkHHbwrSdAfHpwk=.63294614-29f0-4a0f-94fb-ab970f57fb6a@github.com> References: <4rZCe2nLn47Y02BI7JNSuQSFsfj7ZK7ytsymrihoY48=.bc3879ee-bda4-4b4b-b577-3c3560d51832@github.com> <6oI4cnAONyOuxAmoPjp_rkZU5IsA7tQjwhElxBxrWK4=.84954094-f156-4d03-a496-9f6a5930b5d0@github.com> <3DjZv6sTdGVPfpA_0Pg0Or8gwarXxkHHbwrSdAfHpwk=.63294614-29f0-4a0f-94fb-ab970f57fb6a@github.com> Message-ID: On Thu, 21 Jan 2021 07:39:31 GMT, Sergey Bylokhov wrote: > > > Does the JDK-8198340 bug is a test issue? Could be...It shows Caused by: java.lang.IllegalStateException: Shutdown in progress which I think could be caused by invokeLater so that test tried to do the execution when the main is actually finishing. ------------- PR: https://git.openjdk.java.net/jdk/pull/2161 From psadhukhan at openjdk.java.net Thu Jan 21 08:30:57 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Thu, 21 Jan 2021 08:30:57 GMT Subject: Integrated: 8260035: Deproblemlist few problemlisted test In-Reply-To: <4rZCe2nLn47Y02BI7JNSuQSFsfj7ZK7ytsymrihoY48=.bc3879ee-bda4-4b4b-b577-3c3560d51832@github.com> References: <4rZCe2nLn47Y02BI7JNSuQSFsfj7ZK7ytsymrihoY48=.bc3879ee-bda4-4b4b-b577-3c3560d51832@github.com> Message-ID: On Wed, 20 Jan 2021 11:49:25 GMT, Prasanta Sadhukhan wrote: > Few problemlisted test > javax/swing/plaf/basic/Test6984643.java > javax/swing/JMenuItem/6249972/bug6249972.java > javax/swing/JTree/6263446/bug6263446.java > were unstable in mach5 nightly testing. These can be deproblemlisted now with appropriate stability fixes. > Mach5 jobs running for several iterations for all these tests in all platforms is ok. Link in JBS. This pull request has now been integrated. Changeset: 7f7166db Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/7f7166db Stats: 114 lines in 4 files changed: 29 ins; 9 del; 76 mod 8260035: Deproblemlist few problemlisted test Reviewed-by: jdv ------------- PR: https://git.openjdk.java.net/jdk/pull/2161 From prr at openjdk.java.net Thu Jan 21 15:58:35 2021 From: prr at openjdk.java.net (Phil Race) Date: Thu, 21 Jan 2021 15:58:35 GMT Subject: RFR: 8259869: [macOS] Remove desktop module dependencies on JNF Reference APIs In-Reply-To: References: Message-ID: <2LpmfCGqLhdLNlvFIrgycqOOgQi2cJx0OZk6E15zgBE=.26f9fe00-bf88-42b1-b61e-5efc693d9bc7@github.com> On Wed, 20 Jan 2021 06:02:55 GMT, Sergey Bylokhov wrote: >> This removes desktop module usage of the JNF JNI reference convenience APIs >> These are simply a direct conversion >> JNFNewGlobalRef >> JNFDeleteGlobalRef >> JNFNewWeakGlobalRef >> JNFDeleteWeakGlobalRef >> >> These two >> JNFJObjectWrapper >> JNFWeakJObjectWrapper >> exist to allow clean up of the refs when a Cocoa wrapper object is released. >> However in all cases there are more direct ways to clean it up and in at least one usage >> the existing code directly releases it with the comment that this is more efficient. >> >> All our automated regression and JCK tests pass with this change. > > src/java.desktop/macosx/native/libawt_lwawt/awt/AWTWindow.m line 1547: > >> 1545: AWTWindow *awtWindow = [AWTWindow getTopmostWindowUnderMouse]; >> 1546: if (awtWindow != nil) { >> 1547: topmostWindowUnderMouse = awtWindow.javaPlatformWindow; > > I wonder what type of "reference" we should return here, I do not remember how the "NewWeakGlobalRef" behave when returned to java. It is no different than other cases. Java will get a new strong ref to the object and expect it to be of the return type of this native method. > src/java.desktop/macosx/native/libawt_lwawt/awt/CDragSourceContextPeer.m line 58: > >> 56: jobject gTriggerEvent = (*env)->NewGlobalRef(env, jtrigger); >> 57: jlongArray gFormats = (*env)->NewGlobalRef(env, jformats); >> 58: jobject gFormatMap = (*env)->NewGlobalRef(env, jformatmap); > > All above should be checked for NULL since OOM may occur, but it looks like it does not throw OOM? > https://docs.oracle.com/javase/7/docs/technotes/guides/jni/spec/functions.html#NewGlobalRef It is the case that per even the latest spec. none of these throw any exception. https://docs.oracle.com/en/java/javase/15/docs/specs/jni/functions.html#newglobalref So I think the existing code doesn't do anything in the event of a NULL return. And if you want to check for NULL here and return NULL from the native method there's semantic implications that require the caller never pass NULL for any of these. It is not illegal to pass NULL to NewGlobalRef. Investigating and confirming that is beyond the scope of this change. Or we make the code a bit more complex here and check that we get back non-null for a non-null arg. But once again nothing new here. > src/java.desktop/macosx/native/libawt_lwawt/awt/AWTWindow.h line 55: > >> 53: @property (nonatomic, retain) NSWindow *nsWindow; >> 54: >> 55: @property (nonatomic) jobject javaPlatformWindow; > > I think it will be useful to have a comment here that this value is a weak reference and should be copied to the local ref before usage. I suppose I can add a comment ... but it is already done in the couple of dozen usages so any one adding a new usage who misses that will likely miss the comment too. ------------- PR: https://git.openjdk.java.net/jdk/pull/2133 From prr at openjdk.java.net Thu Jan 21 16:28:19 2021 From: prr at openjdk.java.net (Phil Race) Date: Thu, 21 Jan 2021 16:28:19 GMT Subject: RFR: 8259869: [macOS] Remove desktop module dependencies on JNF Reference APIs [v2] In-Reply-To: References: Message-ID: > This removes desktop module usage of the JNF JNI reference convenience APIs > These are simply a direct conversion > JNFNewGlobalRef > JNFDeleteGlobalRef > JNFNewWeakGlobalRef > JNFDeleteWeakGlobalRef > > These two > JNFJObjectWrapper > JNFWeakJObjectWrapper > exist to allow clean up of the refs when a Cocoa wrapper object is released. > However in all cases there are more direct ways to clean it up and in at least one usage > the existing code directly releases it with the comment that this is more efficient. > > All our automated regression and JCK tests pass with this change. Phil Race has updated the pull request incrementally with one additional commit since the last revision: 8259869: [macOS] Remove desktop module dependencies on JNF Reference APIs ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/2133/files - new: https://git.openjdk.java.net/jdk/pull/2133/files/61d6476b..6a9e7a0d Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2133&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2133&range=00-01 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/2133.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2133/head:pull/2133 PR: https://git.openjdk.java.net/jdk/pull/2133 From serb at openjdk.java.net Fri Jan 22 01:22:10 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Fri, 22 Jan 2021 01:22:10 GMT Subject: RFR: 8259869: [macOS] Remove desktop module dependencies on JNF Reference APIs [v2] In-Reply-To: References: Message-ID: On Thu, 21 Jan 2021 16:28:19 GMT, Phil Race wrote: >> This removes desktop module usage of the JNF JNI reference convenience APIs >> These are simply a direct conversion >> JNFNewGlobalRef >> JNFDeleteGlobalRef >> JNFNewWeakGlobalRef >> JNFDeleteWeakGlobalRef >> >> These two >> JNFJObjectWrapper >> JNFWeakJObjectWrapper >> exist to allow clean up of the refs when a Cocoa wrapper object is released. >> However in all cases there are more direct ways to clean it up and in at least one usage >> the existing code directly releases it with the comment that this is more efficient. >> >> All our automated regression and JCK tests pass with this change. > > Phil Race has updated the pull request incrementally with one additional commit since the last revision: > > 8259869: [macOS] Remove desktop module dependencies on JNF Reference APIs Marked as reviewed by serb (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2133 From prr at openjdk.java.net Fri Jan 22 01:55:58 2021 From: prr at openjdk.java.net (Phil Race) Date: Fri, 22 Jan 2021 01:55:58 GMT Subject: Integrated: 8259869: [macOS] Remove desktop module dependencies on JNF Reference APIs In-Reply-To: References: Message-ID: On Mon, 18 Jan 2021 16:08:37 GMT, Phil Race wrote: > This removes desktop module usage of the JNF JNI reference convenience APIs > These are simply a direct conversion > JNFNewGlobalRef > JNFDeleteGlobalRef > JNFNewWeakGlobalRef > JNFDeleteWeakGlobalRef > > These two > JNFJObjectWrapper > JNFWeakJObjectWrapper > exist to allow clean up of the refs when a Cocoa wrapper object is released. > However in all cases there are more direct ways to clean it up and in at least one usage > the existing code directly releases it with the comment that this is more efficient. > > All our automated regression and JCK tests pass with this change. This pull request has now been integrated. Changeset: 92c2f084 Author: Phil Race URL: https://git.openjdk.java.net/jdk/commit/92c2f084 Stats: 153 lines in 21 files changed: 19 ins; 7 del; 127 mod 8259869: [macOS] Remove desktop module dependencies on JNF Reference APIs Reviewed-by: serb ------------- PR: https://git.openjdk.java.net/jdk/pull/2133 From psadhukhan at openjdk.java.net Fri Jan 22 07:27:55 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 22 Jan 2021 07:27:55 GMT Subject: RFR: 8164484: Unity, JTable cell editor, javax/swing/JComboBox/6559152/bug6559152.java Message-ID: <3TceOEHg5ssPawNPnKBN1ih8hEiepbvhOJhKJJyjgLI=.34e7bf6f-7f66-4533-b686-b8de42cd0aa8@github.com> This test was problemlisted for linux as it fails only on ubuntu18.04 in mach5 nightly testing. It was infact passing on ubuntu19.10, 20.04 and 20.10 so it was test specific issue. Made appropriate stability fix by adding waitForIdle() at appropriate location. Mach5 job running specifically on ubuntu18.04 and all other platforms is ok. Link in JBS. ------------- Commit messages: - 8164484: Unity, JTable cell editor, javax/swing/JComboBox/6559152/bug6559152.java Changes: https://git.openjdk.java.net/jdk/pull/2192/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2192&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8164484 Stats: 7 lines in 2 files changed: 4 ins; 3 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/2192.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2192/head:pull/2192 PR: https://git.openjdk.java.net/jdk/pull/2192 From serb at openjdk.java.net Fri Jan 22 07:45:50 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Fri, 22 Jan 2021 07:45:50 GMT Subject: RFR: 8164484: Unity, JTable cell editor, javax/swing/JComboBox/6559152/bug6559152.java In-Reply-To: <3TceOEHg5ssPawNPnKBN1ih8hEiepbvhOJhKJJyjgLI=.34e7bf6f-7f66-4533-b686-b8de42cd0aa8@github.com> References: <3TceOEHg5ssPawNPnKBN1ih8hEiepbvhOJhKJJyjgLI=.34e7bf6f-7f66-4533-b686-b8de42cd0aa8@github.com> Message-ID: On Fri, 22 Jan 2021 07:23:00 GMT, Prasanta Sadhukhan wrote: > This test was problemlisted for linux as it fails only on ubuntu18.04 in mach5 nightly testing. It was infact passing on ubuntu19.10, 20.04 and 20.10 so it was test specific issue. Made appropriate stability fix by adding waitForIdle() at appropriate location. > Mach5 job running specifically on ubuntu18.04 and all other platforms is ok. Link in JBS. Marked as reviewed by serb (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2192 From jdv at openjdk.java.net Fri Jan 22 08:02:53 2021 From: jdv at openjdk.java.net (Jayathirth D V) Date: Fri, 22 Jan 2021 08:02:53 GMT Subject: RFR: 8164484: Unity, JTable cell editor, javax/swing/JComboBox/6559152/bug6559152.java In-Reply-To: <3TceOEHg5ssPawNPnKBN1ih8hEiepbvhOJhKJJyjgLI=.34e7bf6f-7f66-4533-b686-b8de42cd0aa8@github.com> References: <3TceOEHg5ssPawNPnKBN1ih8hEiepbvhOJhKJJyjgLI=.34e7bf6f-7f66-4533-b686-b8de42cd0aa8@github.com> Message-ID: <2AvQHUZOvTSMqUo8teW22Ckq_3cHQgYBldyHI8lO14U=.92b3e18c-7e48-49b0-ba5f-7bb8458a33c2@github.com> On Fri, 22 Jan 2021 07:23:00 GMT, Prasanta Sadhukhan wrote: > This test was problemlisted for linux as it fails only on ubuntu18.04 in mach5 nightly testing. It was infact passing on ubuntu19.10, 20.04 and 20.10 so it was test specific issue. Made appropriate stability fix by adding waitForIdle() at appropriate location. > Mach5 job running specifically on ubuntu18.04 and all other platforms is ok. Link in JBS. Marked as reviewed by jdv (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2192 From psadhukhan at openjdk.java.net Fri Jan 22 08:06:05 2021 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 22 Jan 2021 08:06:05 GMT Subject: Integrated: 8164484: Unity, JTable cell editor, javax/swing/JComboBox/6559152/bug6559152.java In-Reply-To: <3TceOEHg5ssPawNPnKBN1ih8hEiepbvhOJhKJJyjgLI=.34e7bf6f-7f66-4533-b686-b8de42cd0aa8@github.com> References: <3TceOEHg5ssPawNPnKBN1ih8hEiepbvhOJhKJJyjgLI=.34e7bf6f-7f66-4533-b686-b8de42cd0aa8@github.com> Message-ID: On Fri, 22 Jan 2021 07:23:00 GMT, Prasanta Sadhukhan wrote: > This test was problemlisted for linux as it fails only on ubuntu18.04 in mach5 nightly testing. It was infact passing on ubuntu19.10, 20.04 and 20.10 so it was test specific issue. Made appropriate stability fix by adding waitForIdle() at appropriate location. > Mach5 job running specifically on ubuntu18.04 and all other platforms is ok. Link in JBS. This pull request has now been integrated. Changeset: 14522800 Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/14522800 Stats: 7 lines in 2 files changed: 4 ins; 3 del; 0 mod 8164484: Unity, JTable cell editor, javax/swing/JComboBox/6559152/bug6559152.java Reviewed-by: serb, jdv ------------- PR: https://git.openjdk.java.net/jdk/pull/2192 From aivanov at openjdk.java.net Fri Jan 22 20:04:02 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Fri, 22 Jan 2021 20:04:02 GMT Subject: RFR: 8240247: No longer need to wrap files with contentContainer Message-ID: [JDK-8231144](https://bugs.openjdk.java.net/browse/JDK-8231144) wrapped the contents of plain HTML documents into `
` to apply the styles and add the margins around the content for a consistent look. This is no longer necessary after [JDK-8239817](https://bugs.openjdk.java.net/browse/JDK-8239817) was fixed. These documents looks fine without being wrapped. So this fix basically undoes changes under JDK-8231144 and removes the redundant `
` wrapper element. ------------- Commit messages: - Remove contentContainer wrapper Changes: https://git.openjdk.java.net/jdk/pull/2203/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2203&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8240247 Stats: 45 lines in 15 files changed: 0 ins; 30 del; 15 mod Patch: https://git.openjdk.java.net/jdk/pull/2203.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2203/head:pull/2203 PR: https://git.openjdk.java.net/jdk/pull/2203 From serb at openjdk.java.net Fri Jan 22 20:14:41 2021 From: serb at openjdk.java.net (Sergey Bylokhov) Date: Fri, 22 Jan 2021 20:14:41 GMT Subject: RFR: 8240247: No longer need to wrap files with contentContainer In-Reply-To: References: Message-ID: On Fri, 22 Jan 2021 19:50:17 GMT, Alexey Ivanov wrote: > [JDK-8231144](https://bugs.openjdk.java.net/browse/JDK-8231144) wrapped the contents of plain HTML documents into `
` to apply the styles and add the margins around the content for a consistent look. > > This is no longer necessary after [JDK-8239817](https://bugs.openjdk.java.net/browse/JDK-8239817) was fixed. > > These documents looks fine without being wrapped. So this fix basically undoes changes under JDK-8231144 and removes the redundant `
` wrapper element. Marked as reviewed by serb (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/2203 From aivanov at openjdk.java.net Sat Jan 23 11:40:39 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Sat, 23 Jan 2021 11:40:39 GMT Subject: Integrated: 8240247: No longer need to wrap files with contentContainer In-Reply-To: References: Message-ID: On Fri, 22 Jan 2021 19:50:17 GMT, Alexey Ivanov wrote: > [JDK-8231144](https://bugs.openjdk.java.net/browse/JDK-8231144) wrapped the contents of plain HTML documents into `
` to apply the styles and add the margins around the content for a consistent look. > > This is no longer necessary after [JDK-8239817](https://bugs.openjdk.java.net/browse/JDK-8239817) was fixed. > > These documents looks fine without being wrapped. So this fix basically undoes changes under JDK-8231144 and removes the redundant `
` wrapper element. This pull request has now been integrated. Changeset: f624dba6 Author: Alexey Ivanov URL: https://git.openjdk.java.net/jdk/commit/f624dba6 Stats: 45 lines in 15 files changed: 0 ins; 30 del; 15 mod 8240247: No longer need to wrap files with contentContainer Reviewed-by: serb ------------- PR: https://git.openjdk.java.net/jdk/pull/2203 From aivanov at openjdk.java.net Sat Jan 23 19:10:46 2021 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Sat, 23 Jan 2021 19:10:46 GMT Subject: RFR: 8260314: Replace border="1" on tables with CSS Message-ID: <6Eozf9W0lwh5FChjqvq21kTq4ewQPRn24Ad0gLTbyyM=.c6f845d9-d3a3-4266-b907-029aa8eee61a@github.com> Replace presentational attribute `border="1"` on `` element with CSS styles: table {border: outset 1px} th, td {border: inset 1px} These declarations produce the same rendering as the default one in Firefox and Edge. Another option is to use `solid` in both cases. ------------- Commit messages: - Move table styles to common