From duke at openjdk.java.net Tue Mar 1 01:26:26 2022 From: duke at openjdk.java.net (DamonGuy) Date: Tue, 1 Mar 2022 01:26:26 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v6] In-Reply-To: <17xp212dQCFG9Yc7ML9pjPURjwxuFMOIZ75eGX3A-58=.71b5c61c-9fee-493a-a0a6-2345112cd8e7@github.com> References: <17xp212dQCFG9Yc7ML9pjPURjwxuFMOIZ75eGX3A-58=.71b5c61c-9fee-493a-a0a6-2345112cd8e7@github.com> Message-ID: On Mon, 28 Feb 2022 21:56:44 GMT, Alexey Ivanov wrote: >> DamonGuy has updated the pull request incrementally with one additional commit since the last revision: >> >> Updated exception propagation. Added button location to EDT. Updated testImageCentering for variable arguments. Moved object instantiation to the same line. > > test/jdk/javax/swing/JButton/HtmlButtonImageTest/HtmlButtonImageTest.java line 92: > >> 90: BufferedImage failImg = robot.createScreenCapture(new Rectangle(point.x - BUTTON_WIDTH / 2, point.y - BUTTON_HEIGHT / 2, BUTTON_WIDTH, BUTTON_HEIGHT)); >> 91: ImageIO.write(failImg, "png", new File(testDir + "/fail_square.png")); >> 92: throw new RuntimeException("Failed image generation: " + e); > > This doesn't make sense. I mean `IOException` can be thrown from try-block only from `generateImage` method. In this case, there's nothing to display on screen and therefore nothing to capture. You should not catch `IOException` at all. > > You should create the screenshot if the conditions you verify aren't met. The screenshot should be taken inside `if (!checkRedness(c))` block before you throw the `RuntimeException` to indicate failure. I believe I fixed the additional changes requested in the next commit. I understand what you meant with main handling exceptions now. ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From duke at openjdk.java.net Tue Mar 1 02:19:07 2022 From: duke at openjdk.java.net (duke) Date: Tue, 1 Mar 2022 02:19:07 GMT Subject: Withdrawn: JDK-8276809: java/awt/font/JNICheck/FreeTypeScalerJNICheck.java shows JNI warning on Windows In-Reply-To: <88pzPWHZU5dZbSXBuIwFqKeQPEiYxOrlZAU-xqGGVMg=.126a0ae6-4382-4986-89d0-250cc2c71081@github.com> References: <88pzPWHZU5dZbSXBuIwFqKeQPEiYxOrlZAU-xqGGVMg=.126a0ae6-4382-4986-89d0-250cc2c71081@github.com> Message-ID: On Tue, 9 Nov 2021 07:57:55 GMT, Matthias Baesken wrote: > The new test java/awt/font/JNICheck/FreeTypeScalerJNICheck.java introduced with https://bugs.openjdk.java.net/browse/JDK-8269223 adds -Xcheck:jni to an awt related test, and shows on Windows server 2019 the following JNI warning , so the test JNICheck/FreeTypeScalerJNICheck.java fails on this Windows version. > > :stdErr: > Thu Oct 28 01:27:10 CEST 2021 > stdout: [WARNING in native method: JNI call made without checking exceptions when required to from CallStaticVoidMethodV > at sun.awt.Win32GraphicsEnvironment.initDisplay(java.desktop at 18.0.0.1-internal/Native Method) > at sun.awt.Win32GraphicsEnvironment.initDisplayWrapper(java.desktop at 18.0.0.1-internal/Win32GraphicsEnvironment.java:95) > at sun.awt.Win32GraphicsEnvironment.(java.desktop at 18.0.0.1-internal/Win32GraphicsEnvironment.java:63) > at sun.awt.PlatformGraphicsInfo.createGE(java.desktop at 18.0.0.1-internal/PlatformGraphicsInfo.java:34) > at java.awt.GraphicsEnvironment$LocalGE.createGE(java.desktop at 18.0.0.1-internal/GraphicsEnvironment.java:93) > at java.awt.GraphicsEnvironment$LocalGE.(java.desktop at 18.0.0.1-internal/GraphicsEnvironment.java:84) > at java.awt.GraphicsEnvironment.getLocalGraphicsEnvironment(java.desktop at 18.0.0.1-internal/GraphicsEnvironment.java:106) > at FreeTypeScalerJNICheck.runTest(FreeTypeScalerJNICheck.java:53) > at FreeTypeScalerJNICheck.main(FreeTypeScalerJNICheck.java:44) > > We can get rid of the warning by adjusting a JNU_CallStaticMethodByName call in awt_Win32GraphicsEnv.cpp . This pull request has been closed without being integrated. ------------- PR: https://git.openjdk.java.net/jdk/pull/6306 From psadhukhan at openjdk.java.net Tue Mar 1 08:38:35 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Tue, 1 Mar 2022 08:38:35 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack Message-ID: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts ------------- Commit messages: - 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack Changes: https://git.openjdk.java.net/jdk/pull/7643/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7643&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8190907 Stats: 14 lines in 1 file changed: 2 ins; 0 del; 12 mod Patch: https://git.openjdk.java.net/jdk/pull/7643.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7643/head:pull/7643 PR: https://git.openjdk.java.net/jdk/pull/7643 From psadhukhan at openjdk.java.net Tue Mar 1 10:22:29 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Tue, 1 Mar 2022 10:22:29 GMT Subject: RFR: 8282473: Refactor swing classes javadoc to use @throws instead of @exception Message-ID: Prevailing JDK coding practices use "@throws" rather than "@exception". Refactor existing swing classes javadoc to use @throws ------------- Commit messages: - Fix - Fix - Fix Changes: https://git.openjdk.java.net/jdk/pull/7644/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7644&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8282473 Stats: 533 lines in 114 files changed: 0 ins; 0 del; 533 mod Patch: https://git.openjdk.java.net/jdk/pull/7644.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7644/head:pull/7644 PR: https://git.openjdk.java.net/jdk/pull/7644 From aivanov at openjdk.java.net Tue Mar 1 11:44:12 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 11:44:12 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v7] In-Reply-To: References: Message-ID: <-K6rtQyT-7qLsORwESekvXuVNRhPii3sMGAlrS6IwAY=.f46b22a7-8c6a-467f-bd58-54e798500bef@github.com> On Mon, 28 Feb 2022 23:27:50 GMT, DamonGuy wrote: >> Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. >> >> Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. >> >> Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. > > DamonGuy has updated the pull request incrementally with one additional commit since the last revision: > > Updated code organization. Removed unnecessary catch blocks and comments. Added null check for frame disposal. test/jdk/javax/swing/JButton/HtmlButtonImageTest/HtmlButtonImageTest.java line 53: > 51: public final class HtmlButtonImageTest { > 52: private static JFrame frame; > 53: private static Point point; `point` must be volatile as it's accessed from two threads. ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From aivanov at openjdk.java.net Tue Mar 1 12:06:15 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 12:06:15 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v2] In-Reply-To: References: Message-ID: On Sat, 5 Feb 2022 18:16:34 GMT, Phil Race wrote: >> DamonGuy has updated the pull request incrementally with two additional commits since the last revision: >> >> - Add cycling of all installed LAFs. Add image generation to avoid saving an image to repo. >> - Changed frame to dispose regardless of passing or failing. > > src/java.desktop/share/classes/javax/swing/SwingUtilities.java line 1129: > >> 1127: v = (c != null) ? (View) c.getClientProperty("html") : null; >> 1128: if (v != null) { >> 1129: if(availTextWidth < (int) v.getPreferredSpan(View.X_AXIS)){ > > Can you attach "before" and "after" images into the bug report as part of the evaluation ? > Do this for Aqua and Metal too. > > I presume this code only gets entered for labels with images ? > If this is all L&Fs then please manually test on mac. Windows and GTK L&Fs using SwingSet2 looking for problems as well as running all automated client tests using mach5. > > We do not use syntax like "if(" and "else{". > It should always be "if (" and "else {" > Also no space after (int) for the cast. lines 1129 and 1134 .. > And make sure lines are <=80 characters long which that one 1134 doesn't look to be. > @prrace Also no space after (int) for the cast. lines 1129 and 1134 .. I'd like to clarify what style is followed by the client code. [Code Conventions for the Java](https://www.oracle.com/java/technologies/javase/codeconventions-contents.html) published in 1999 in its section for white-space [state](https://www.oracle.com/java/technologies/javase/codeconventions-whitespace.html#498): _?Casts should be followed by a blank space.?_ A more recent [draft for Java Style](http://cr.openjdk.java.net/~alundblad/styleguide/index-v6.html) that is commonly referred to during code reviews also says about using the space after the cast in its [Horizontal Whitespace section](http://cr.openjdk.java.net/~alundblad/styleguide/index-v6.html#toc-whitespace): _?A single space should be used? After the closing parenthesis of a cast.?_ As far as I can see, neither style ? *no space* after the cast vs *a space* after the cast ? is followed consistently. Let's agree on one option and follow it then. Shall I bring it up for a discussion on the client-libs mailing list? ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From aivanov at openjdk.java.net Tue Mar 1 12:25:09 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 12:25:09 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v4] In-Reply-To: References: Message-ID: <9Odpq0YTy18AQHa6Nt3n1KoyW19iYs7l2GgqiEdQZd0=.9caae1b3-5818-4038-9416-3a87859705a0@github.com> On Thu, 24 Feb 2022 16:07:41 GMT, DamonGuy wrote: >> src/java.desktop/macosx/classes/com/apple/laf/AquaButtonUI.java line 313: >> >>> 311: // use zero insets for view since layout only handles text calculations >>> 312: text = layoutAndGetText(g, b, aquaBorder, new >>> 313: Insets(0,0,0,0), viewRect, iconRect, textRect); >> >> If there is only html text instead of image in JButton, will it not be a problem using 0 insets or is there a problem with text too being shifted like image? > > The text was also unexpectedly shifted, so this zero inset fixes that issue as well. Setting the insets to zero for all HTML fixes the unwanted padding. Should we create a separate test for HTML text? Probably manual only but still. If it is a known issue and it's also resolved by this fix, I find such a test useful. ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From aivanov at openjdk.java.net Tue Mar 1 12:25:10 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 12:25:10 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v4] In-Reply-To: References: Message-ID: <1VyZwzdKS6ZiYdakmm7j7Elb1ECbIH72466KjoAD4IE=.388b8d7e-554f-4e3a-be5f-00502672565e@github.com> On Thu, 24 Feb 2022 09:26:48 GMT, Prasanta Sadhukhan wrote: >> DamonGuy has updated the pull request incrementally with one additional commit since the last revision: >> >> Fixed incomplete comment. > > test/jdk/javax/swing/JButton/HtmlButtonImageTest/HtmlButtonImageTest.java line 85: > >> 83: robot.mouseMove(frame.getLocationOnScreen().x, frame.getLocationOnScreen().y); >> 84: robot.mouseMove(button.getLocationOnScreen().x, button.getLocationOnScreen().y); >> 85: > > I guess there is no need of this staggard mouseMove. One can move straight to "point" as done below, right? There's no need for `mouseMove` at all. However, it helps to analyse the image visually whether the red square is centered in the button or not. ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From aivanov at openjdk.java.net Tue Mar 1 12:35:06 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 12:35:06 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v2] In-Reply-To: References: Message-ID: <_xis0t6zeLWPEutXb6ffOJD_w0cBAgOvfoQYifD4g2Y=.1790d64d-0c1e-44a4-b4c8-bd2261d758b9@github.com> On Fri, 4 Feb 2022 02:55:02 GMT, DamonGuy wrote: >> DamonGuy has updated the pull request incrementally with two additional commits since the last revision: >> >> - Add cycling of all installed LAFs. Add image generation to avoid saving an image to repo. >> - Changed frame to dispose regardless of passing or failing. > > I noticed I mistakenly left the thrown exception for failures commented out for the most recent push while testing the new changes. The new test with the run-time generated image and cycled LAFs seems to fail for Nimbus LAF, so I will fix this and push to the branch again. A general comment: You, @DamonGuy, have quite a few comments in the test. Most of these comments are redundant because they repeat what the code does. The code should be self-documenting by using meaningful method and variable names, and I think you follow this rule. The problem with comments is that they may be left unmodified when the code changes. Thus, the comments become misleading, confusing and therefore don't help a reader understand the code but make the reader guess whether they should believe the comment or the code. A couple of points from [Programming Practices](http://cr.openjdk.java.net/~alundblad/styleguide/index-v6.html#toc-commenting-code) section from the Java Style Guide: ? _?First and foremost, try to make the code simple enough that it?s self explanatory. While comments explaining the code are good, not having to explain the code is better.?_ ? _?Avoid comments that run the risk of getting out of sync as the code evolves. (If a comment runs the risk of getting out of sync, it?s often a sign that it comments how the code works rather than what the code achieves.)?_ ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From aivanov at openjdk.java.net Tue Mar 1 14:07:07 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 14:07:07 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v7] In-Reply-To: References: Message-ID: On Mon, 28 Feb 2022 23:27:50 GMT, DamonGuy wrote: >> Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. >> >> Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. >> >> Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. > > DamonGuy has updated the pull request incrementally with one additional commit since the last revision: > > Updated code organization. Removed unnecessary catch blocks and comments. Added null check for frame disposal. Hi Jeremy, > _Mailing list message from [Jeremy Wood](mailto:mickleness at gmail.com) on [client-libs-dev](mailto:client-libs-dev at mail.openjdk.java.net):_ > > Damon: > > Sorry for joining this chat late. Is the core problem here that Swing is positioning graphic elements incorrectly -- and those bad positions are platform-independent? Yes, it looks that this issue is platform-independent and can be reproduced with other Look-and-Feels on other platforms. > > If so: this might be a good candidate for me to try to rewrite a test to avoid the Robot class. I have a Graphics2D implementation that catalogs rendering operations. If just the act of reviewing those catalogued instructions can detect the underlying problem: this might be a simpler/faster alternative. (I?m assuming tests written using the Robot class are more expensive to run and more brittle.) > > Does this sound like something that might benefit this test case? (I can have a look at it in a few days.) > > Regards, - Jeremy The code you reference cannot be used in OpenJDK until _the author of the code_ contributes it to OpenJDK. You're right headful tests are more expensive to run. @prsadhuk mentioned that `Robot.getPixelColor` may produce an incorrect result which @honkar-jdk came across. In fact, @DamonGuy sees the artefacts too: on a High DPI display (and built-in Retina display of MacBook is a High DPI display), one pixel in user's space is represented with several physical pixels. In such a case, `getPixelColor` still returns only one pixel. In [Harshitha's](https://github.com/honkar-jdk) #7219, the table header border had 1px width. The robot correctly captured the specified part of the screen but the line was drawn on the second row of the physical pixels. Damon needs to account for antialiasing. The image needs to be scaled up for Retina display, its borders aren't sharp anymore. Painting the `JButton` to `BufferedImage` could resolve these problems, the test could check for the red color without adding a tolerance. If there's no frame and button is painted to `BufferedImage`, the test could be made _headless_. Perhaps, we should advise this approach when the test doesn't need to process any input. ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From aivanov at openjdk.java.net Tue Mar 1 14:30:06 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 14:30:06 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding In-Reply-To: References: Message-ID: On Thu, 3 Feb 2022 01:43:07 GMT, DamonGuy wrote: > > The JBS says "This is reproducible with the Mac default (Aqua) look and feel; using another LAF (Metal for instance) seems to position the ImageView correctly." so it seems to be a mac issue not generic, so fixing in shared code will have other repurcussions..You probably need to fix in Aqua specific class.. > > @prsadhuk Sorry, I forgot to mention in the JBS issue that while testing if the issue was reproducible locally, I discovered that the issue exists for Aqua LAF as described, but also other LAFs (such as Metal and Basic). > > This discovery shifted my attention away from AquaButtonUI alone, and AquaButtonUI's layoutAndGetText method using alternative insets turned out to be where the issue was coming from. This method relies on "SwingUtilities.layoutCompoundLabel" to generate alternative insets, so that's where I made my changes. Is this logic valid or should I contain my changes in AquaButtonUI? Since this fix addresses `AquaButtonUI` only and we know for sure that other Look-and-Feels are also affected (Metal at least according to the comments in JBS and here), has anyone submitted a new bug for other LaFs? ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From aivanov at openjdk.java.net Tue Mar 1 16:15:08 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 16:15:08 GMT Subject: RFR: 8281569: Create tests for Frame.setMinimumSize() method [v6] In-Reply-To: References: Message-ID: On Mon, 28 Feb 2022 11:45:32 GMT, Srinivas Mandalika wrote: >> Create tests for setMinimumSize functionality >> Create a test which verifies that on increasing the Frame's minimumSize, it gets reflected in the subsequent getSize call >> Create a test which verifies that on frame resizes back to minimumSize on calling pack. > > Srinivas Mandalika has updated the pull request incrementally with one additional commit since the last revision: > > Review comments fixed, removed redundant waitForIdle Changes requested by aivanov (Reviewer). test/jdk/java/awt/Frame/SetMinimumSizeTest/SetMinimumSizeTest1.java line 40: > 38: > 39: private static Frame frame; > 40: private static volatile Dimension dimension, actualDimension; In Java, each field should be declared on its own line. test/jdk/java/awt/Frame/SetMinimumSizeTest/SetMinimumSizeTest1.java line 42: > 40: private static volatile Dimension dimension, actualDimension; > 41: > 42: public static void createGUI(Frame _frame) { The `_frame` parameter is never used, please remove it. test/jdk/java/awt/Frame/SetMinimumSizeTest/SetMinimumSizeTest1.java line 61: > 59: dimension = frame.getSize(); > 60: dimension.width = dimension.width + 20; > 61: dimension.height = dimension.height + 20; Suggestion: dimension.width += 20; dimension.height += 20; test/jdk/java/awt/Frame/SetMinimumSizeTest/SetMinimumSizeTest1.java line 73: > 71: }); > 72: > 73: robot.waitForIdle(); At line 73, `waitForIdle` doesn't affect anything, it should be removed. test/jdk/java/awt/Frame/SetMinimumSizeTest/SetMinimumSizeTest1.java line 81: > 79: } > 80: } > 81: finally { Suggestion: } finally { The finally-block as well as catch- and else-blocks should be at the same line as the closing brace from the previous block. test/jdk/java/awt/Frame/SetMinimumSizeTest/SetMinimumSizeTest2.java line 42: > 40: private static volatile Dimension dimension, actualDimension; > 41: > 42: public static void createGUI(Frame _frame) { The same as for test1: field declarations are on its own lines, `_frame` parameter removed. test/jdk/java/awt/Frame/SetMinimumSizeTest/SetMinimumSizeTest2.java line 63: > 61: > 62: EventQueue.invokeAndWait(() -> { > 63: frame.setSize(new Dimension(dimension.width + 20, dimension.height + 20)); You can call `setSize(int, int)` method without creating a new `Dimension` object. This way is also fine. test/jdk/java/awt/Frame/SetMinimumSizeTest/SetMinimumSizeTest2.java line 82: > 80: }); > 81: > 82: robot.waitForIdle(); At line 82, `waitForIdle` is not needed. ------------- PR: https://git.openjdk.java.net/jdk/pull/7444 From aivanov at openjdk.java.net Tue Mar 1 16:44:11 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 16:44:11 GMT Subject: RFR: 6911375: mouseWheel has no effect without vertical scrollbar [v2] In-Reply-To: References: Message-ID: On Mon, 28 Feb 2022 07:59:24 GMT, Prasanta Sadhukhan wrote: >> It is observed that mouseWheel doesn't scroll through a list if there is no scrollbar even though pressing PageUp/Down using keyboard moves up/down the list. >> Issue stems from the fact that BasicScrollPaneUI.Handler.mouseWheelMoved() skips the mouse wheel events when the scrollbar is not visible. >> Fixed to handle mouseWheelEvent even though scrollbar is not visible with the assumption that if both vertical and horizontal scrollbar is not visible, then assume default orientation to be vertical. >> >> Existing jtreg, jck tests are ok. All L&F works well with the fix and testcase works in all platforms except on mac which will be handled separately. > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Fix test comments test/jdk/javax/swing/JScrollPane/TestMouseWheelScroll.java line 48: > 46: static Point p; > 47: static int width; > 48: static int height; These three should also be volatile, shouldn't they? test/jdk/javax/swing/JScrollPane/TestMouseWheelScroll.java line 101: > 99: robot.delay(1000); > 100: robot.mouseWheel(1); > 101: robot.delay(500); You don't call `waitForIdle` or `delay` after the mouse click because it just selects an item but doesn't change the position, right? ------------- PR: https://git.openjdk.java.net/jdk/pull/7585 From aivanov at openjdk.java.net Tue Mar 1 16:44:12 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 16:44:12 GMT Subject: RFR: 6911375: mouseWheel has no effect without vertical scrollbar [v2] In-Reply-To: References: Message-ID: On Mon, 28 Feb 2022 08:02:39 GMT, Prasanta Sadhukhan wrote: >> test/jdk/javax/swing/JScrollPane/TestMouseWheelScroll.java line 80: >> >>> 78: scrollPane.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_NEVER); >>> 79: frame.add(scrollPane); >>> 80: frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); >> >> Suggestion: >> >> frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE); >> >> It's safer for tests. > > This testcase was failing on macos which needs to be debugged, so it is excluded in mac for now. But as I told in description, the fix is not affecting other regression test in macos. > It's working on other platforms and I have some issue in my local mac system so I will take this debugging of macos once I sort my system. > > Regarding your other question, the orientation will be horizontal. Tested against javax/swing/JScrollPane/HorizontalMouseWheelOnShiftPressed/HorizontalMouseWheelOnShiftPressed.java I see, thank you for explanation. Does it make sense to put the test in ProblemList.txt for macOS, so that it's not forgotten the test is to be run on macOS too? ------------- PR: https://git.openjdk.java.net/jdk/pull/7585 From aivanov at openjdk.java.net Tue Mar 1 16:54:02 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 16:54:02 GMT Subject: RFR: 6218162: DefaultTableColumnModel.getColumn() method should mention ArrayIndexOutOfBoundsException [v3] In-Reply-To: References: Message-ID: On Mon, 28 Feb 2022 07:24:27 GMT, Prasanta Sadhukhan wrote: >> If invalid ie 0 or > getColumnCount() index is passed to DefaultTableColumnModel.getColumn() then it returns AIOBE in current implementation which should be documented in the spec. Fixed the spec to mention the exception. > > Prasanta Sadhukhan has updated the pull request incrementally with three additional commits since the last revision: > > - javadoc fix > - javadoc fix > - javadoc fix Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7587 From aivanov at openjdk.java.net Tue Mar 1 16:54:03 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 16:54:03 GMT Subject: RFR: 6218162: DefaultTableColumnModel.getColumn() method should mention ArrayIndexOutOfBoundsException [v2] In-Reply-To: References: <4iVrHkUpcWdueUHLYWWF39Do-DXVv8w6C9_78rpC69Y=.4e74c3ec-a8e0-46d1-ac0e-ced56d2e1a37@github.com> Message-ID: On Mon, 28 Feb 2022 07:16:52 GMT, Prasanta Sadhukhan wrote: >> src/java.desktop/share/classes/javax/swing/table/DefaultTableColumnModel.java line 294: >> >>> 292: * at columnIndex >>> 293: * @exception ArrayIndexOutOfBoundsException if columnIndex >>> 294: * is out of range: >> >> As Joe Darcy pointed out, we use @throws these days. > > I was aware of that but this file uses @exception throughout, so to be consistent, I used that..But if it is recommended to use @throws even at the cost of being inconsistent within the file, I would use that. It is the reason why I didn't ask to use `@throws` which is the preferred way. Anyway, #7644 with [JDK-8282473](https://bugs.openjdk.java.net/browse/JDK-8282473) will bring consistency and all `@exception` tags will be replaced with `@throws`. ------------- PR: https://git.openjdk.java.net/jdk/pull/7587 From aivanov at openjdk.java.net Tue Mar 1 16:59:12 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 16:59:12 GMT Subject: RFR: 8281738: Create a regression test for checking the 'Space' key activation of focused Button [v7] In-Reply-To: References: Message-ID: On Wed, 23 Feb 2022 21:15:02 GMT, Alexey Ivanov wrote: > Does anyone have any comments or objections? Having heard no comments, I'm integrating. ------------- PR: https://git.openjdk.java.net/jdk/pull/7460 From mvs at openjdk.java.net Tue Mar 1 16:59:14 2022 From: mvs at openjdk.java.net (Manukumar V S) Date: Tue, 1 Mar 2022 16:59:14 GMT Subject: Integrated: 8281738: Create a regression test for checking the 'Space' key activation of focused Button In-Reply-To: References: Message-ID: On Mon, 14 Feb 2022 14:04:33 GMT, Manukumar V S wrote: > This test verifies in all platforms and in all 'Look and Feels' that pressing the 'Space' key generates ActionEvent on focused Button or not. This pull request has now been integrated. Changeset: 941e97c4 Author: Manukumar V S Committer: Alexey Ivanov URL: https://git.openjdk.java.net/jdk/commit/941e97c4eb186bc63386d17f458ba69f4bddd828 Stats: 152 lines in 1 file changed: 152 ins; 0 del; 0 mod 8281738: Create a regression test for checking the 'Space' key activation of focused Button Reviewed-by: aivanov ------------- PR: https://git.openjdk.java.net/jdk/pull/7460 From aivanov at openjdk.java.net Tue Mar 1 17:01:10 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 17:01:10 GMT Subject: RFR: 8276849: Refresh the window icon on graphics configuration changes [v5] In-Reply-To: <8ubPWqU0vTLZW-eX7r2v9GAvrlp_4-mRKwuz8jwPHlM=.ee58d4d6-1ad0-409a-98cb-2d839e8187d6@github.com> References: <8ubPWqU0vTLZW-eX7r2v9GAvrlp_4-mRKwuz8jwPHlM=.ee58d4d6-1ad0-409a-98cb-2d839e8187d6@github.com> Message-ID: On Fri, 25 Feb 2022 12:39:50 GMT, Emmanuel Bourg wrote: >> When a list of icons is set on a window, the most appropiate icon is selected depending on the graphics configuration. But if the graphics configuration changes (because the window is moved to a different screen, or because the DPI settings of the screen is changed), the frame icon isn't updated. >> >> Here is an example illustrating the issue: >> >> public static void main(String[] args) throws Exception { >> SwingUtilities.invokeLater(() -> { >> JFrame frame = new JFrame("Window Icon Test"); >> frame.setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE); >> frame.setSize(400, 300); >> frame.setVisible(true); >> >> List images = new ArrayList<>(); >> for (int size = 16; size <= 32; size++) { >> // create an image displaying the size used >> BufferedImage image = new BufferedImage(size, size, BufferedImage.TYPE_INT_ARGB); >> Graphics2D g = image.createGraphics(); >> g.setFont(new Font("dialog", Font.BOLD, 12)); >> g.setColor(Color.BLACK); >> g.drawString(String.valueOf(size), 0, size - (size - g.getFont().getSize()) / 2); >> images.add(image); >> } >> >> frame.setIconImages(images); >> }); >> } >> >> On Windows if the screen scaling is set to 100% the 16x16 icon is picked from the list. If the scaling of the screen is set to 150% while the application is running, the 16x16 icon is upscaled and looks blurry. >> >> A way to work around this issue is to listen for graphics configuration changes with: >> >> frame.addPropertyChangeListener("graphicsConfiguration", event -> frame.setIconImages(frame.getIconImages())); >> >> >> Ideally this should be done automatically by the JDK. Maybe the `WindowPeer` could call `updateIconImages()` when `updateGraphicsData()` or `displayChanged()` is invoked? > > Emmanuel Bourg has updated the pull request incrementally with one additional commit since the last revision: > > Don't refresh the frame icon on graphics configuration changes on macOS (not applicable) Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/6180 From aivanov at openjdk.java.net Tue Mar 1 17:05:06 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 17:05:06 GMT Subject: Integrated: 8282150: Drop redundant
elements from tables in java.desktop HTML files In-Reply-To: References: Message-ID: On Sun, 20 Feb 2022 18:07:54 GMT, Alexey Ivanov wrote: > Drops the redundant `
` elements from tables and lists in java.desktop HTML files. This pull request has now been integrated. Changeset: b86a8c00 Author: Alexey Ivanov URL: https://git.openjdk.java.net/jdk/commit/b86a8c004f4e67e7b6bf45ce7cf96bcd8520a881 Stats: 168 lines in 2 files changed: 0 ins; 0 del; 168 mod 8282150: Drop redundant
elements from tables in java.desktop HTML files Reviewed-by: jdv, dmarkov ------------- PR: https://git.openjdk.java.net/jdk/pull/7547 From naoto at openjdk.java.net Tue Mar 1 18:34:00 2022 From: naoto at openjdk.java.net (Naoto Sato) Date: Tue, 1 Mar 2022 18:34:00 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack In-Reply-To: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Tue, 1 Mar 2022 08:31:47 GMT, Prasanta Sadhukhan wrote: > Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts Is it OK to replace all of them? According to this document: https://docs.microsoft.com/en-us/typography/fonts/windows_10_font_list There still are Korean supplementary fonts available (Batang/Gulim/etc) if `Feature On Demand` is installed. Should they be honored? Also, for bold type, should we use `malgunbd.ttf`? ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From aivanov at openjdk.java.net Tue Mar 1 22:01:59 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Tue, 1 Mar 2022 22:01:59 GMT Subject: RFR: 8282473: Refactor swing classes javadoc to use @throws instead of @exception In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 10:16:16 GMT, Prasanta Sadhukhan wrote: > Prevailing JDK coding practices use "@throws" rather than "@exception". > Refactor existing swing classes javadoc to use @throws Marked as reviewed by aivanov (Reviewer). src/java.desktop/share/classes/javax/swing/JApplet.java line 139: > 137: * > 138: * @throws HeadlessException if GraphicsEnvironment.isHeadless() > 139: * returns true. Suggestion: * returns true In other cases, there's no ending punctuation for `@throws`. src/java.desktop/share/classes/javax/swing/JColorChooser.java line 154: > 152: * @return the selected color or null if the user opted out > 153: * @throws HeadlessException if GraphicsEnvironment.isHeadless() > 154: * returns true. It looks `HeadlessException` has the period everywhere where it's documented. In this case, I suggest addressing this under another issue. src/java.desktop/share/classes/javax/swing/JLabel.java line 625: > 623: * @return the key value if {@code key} is a legal value for the > 624: * verticalAlignment or verticalTextPosition properties > 625: * @throws IllegalArgumentException if key isn't TOP, CENTER, or BOTTOM. Another case with ending punctuation. src/java.desktop/share/classes/javax/swing/JRootPane.java line 582: > 580: * @param content the Container to use for component-contents > 581: * @throws java.awt.IllegalComponentStateException (a runtime > 582: * exception) if the content pane parameter is null Shall we remove _?(a runtime exception)?_ note? ------------- PR: https://git.openjdk.java.net/jdk/pull/7644 From duke at openjdk.java.net Tue Mar 1 23:25:04 2022 From: duke at openjdk.java.net (DamonGuy) Date: Tue, 1 Mar 2022 23:25:04 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v4] In-Reply-To: <9Odpq0YTy18AQHa6Nt3n1KoyW19iYs7l2GgqiEdQZd0=.9caae1b3-5818-4038-9416-3a87859705a0@github.com> References: <9Odpq0YTy18AQHa6Nt3n1KoyW19iYs7l2GgqiEdQZd0=.9caae1b3-5818-4038-9416-3a87859705a0@github.com> Message-ID: On Tue, 1 Mar 2022 12:20:13 GMT, Alexey Ivanov wrote: >> The text was also unexpectedly shifted, so this zero inset fixes that issue as well. Setting the insets to zero for all HTML fixes the unwanted padding. > > Should we create a separate test for HTML text? Probably manual only but still. If it is a known issue and it's also resolved by this fix, I find such a test useful. I can definitely try to do so. The newest version of the test is now headless using BufferedImage. By manual, do you mean making a headful test where the tester clicks pass or fail according to the position of HTML text on the button? ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From duke at openjdk.java.net Tue Mar 1 23:27:57 2022 From: duke at openjdk.java.net (DamonGuy) Date: Tue, 1 Mar 2022 23:27:57 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v8] In-Reply-To: References: Message-ID: > Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. > > Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. > > Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. DamonGuy has updated the pull request incrementally with one additional commit since the last revision: Updated test to be headless using BufferedImage. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7310/files - new: https://git.openjdk.java.net/jdk/pull/7310/files/d9db4c2b..dd8599ce Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7310&range=07 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7310&range=06-07 Stats: 113 lines in 1 file changed: 13 ins; 57 del; 43 mod Patch: https://git.openjdk.java.net/jdk/pull/7310.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7310/head:pull/7310 PR: https://git.openjdk.java.net/jdk/pull/7310 From duke at openjdk.java.net Tue Mar 1 23:37:12 2022 From: duke at openjdk.java.net (DamonGuy) Date: Tue, 1 Mar 2022 23:37:12 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v8] In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 23:27:57 GMT, DamonGuy wrote: >> Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. >> >> Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. >> >> Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. > > DamonGuy has updated the pull request incrementally with one additional commit since the last revision: > > Updated test to be headless using BufferedImage. I have updated the test to be headless by using BufferedImage. There is no new bug submitted yet for other LAFs that I know of. I tested other LAFs and the issue does persist, so I can submit the bug for them if necessary. The other LAFs I tested were Metal, Nimbus, and Motif. ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From smandalika at openjdk.java.net Wed Mar 2 06:10:42 2022 From: smandalika at openjdk.java.net (Srinivas Mandalika) Date: Wed, 2 Mar 2022 06:10:42 GMT Subject: RFR: 8281569: Create tests for Frame.setMinimumSize() method [v7] In-Reply-To: References: Message-ID: > Create tests for setMinimumSize functionality > Create a test which verifies that on increasing the Frame's minimumSize, it gets reflected in the subsequent getSize call > Create a test which verifies that on frame resizes back to minimumSize on calling pack. Srinivas Mandalika has updated the pull request incrementally with one additional commit since the last revision: Review comments fixed: field declared on different line, finally block on same line, changed to +=,.. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7444/files - new: https://git.openjdk.java.net/jdk/pull/7444/files/8d9141fb..e244dd3a Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7444&range=06 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7444&range=05-06 Stats: 19 lines in 2 files changed: 2 ins; 6 del; 11 mod Patch: https://git.openjdk.java.net/jdk/pull/7444.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7444/head:pull/7444 PR: https://git.openjdk.java.net/jdk/pull/7444 From myano at openjdk.java.net Wed Mar 2 06:24:05 2022 From: myano at openjdk.java.net (Masanori Yano) Date: Wed, 2 Mar 2022 06:24:05 GMT Subject: RFR: 8275715: D3D pipeline processes multiple PaintEvent at initial drawing [v2] In-Reply-To: <1xgNYge02zcX-kfQ343p2YT4v2MIbym5SHL_fDnqMPI=.ab03d946-c8ec-4a2f-871a-8d58e6a30f43@github.com> References: <1xgNYge02zcX-kfQ343p2YT4v2MIbym5SHL_fDnqMPI=.ab03d946-c8ec-4a2f-871a-8d58e6a30f43@github.com> Message-ID: On Mon, 14 Feb 2022 20:12:39 GMT, Phil Race wrote: >> Masanori Yano has updated the pull request incrementally with one additional commit since the last revision: >> >> 8275715: D3D pipeline processes multiple PaintEvent at initial drawing > > Ok. Please confirm you have run all automated headful jtreg tests on a system with D3D enabled > and that you have also done manual testing (using SwingSet2 and J2Ddemo) and no problems are observed ? @prrace How long will it take for you to reply to the comment? ------------- PR: https://git.openjdk.java.net/jdk/pull/6064 From psadhukhan at openjdk.java.net Wed Mar 2 09:23:02 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 2 Mar 2022 09:23:02 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack In-Reply-To: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Tue, 1 Mar 2022 08:31:47 GMT, Prasanta Sadhukhan wrote: > Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts Malgun Gothic is supposed to be the default korean font replacing Gulim as per the wiki mentioned in the description, so I have only replaced Gulim,not all.. Others like Batang are not replaced. Also, as I pointed out by screenshot in JBS in my Windows10 I dont have malgunbd.ttf so I was doubting it may not be available in all distribution by default, so I have used only malgun.ttf OTOH, if Gulim is available as supplementary fonts, we probably can close this issue as "Wont fix" mentioning the same. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From psadhukhan at openjdk.java.net Wed Mar 2 09:29:01 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 2 Mar 2022 09:29:01 GMT Subject: RFR: 8282473: Refactor swing classes javadoc to use @throws instead of @exception In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 10:16:16 GMT, Prasanta Sadhukhan wrote: > Prevailing JDK coding practices use "@throws" rather than "@exception". > Refactor existing swing classes javadoc to use @throws I would rather keep this PR simple just to replace @exception with @throws. Other anomalies/idiosyncracies can be tackled in separate issue, if you dont mind. ------------- PR: https://git.openjdk.java.net/jdk/pull/7644 From aivanov at openjdk.java.net Wed Mar 2 09:38:03 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 2 Mar 2022 09:38:03 GMT Subject: RFR: 8282473: Refactor swing classes javadoc to use @throws instead of @exception In-Reply-To: References: Message-ID: <1m78aouqMVQM_wYuTVgGkMxqp_T9aYV_r2Mwm5LtukU=.0d622828-b834-4dd9-8626-b91dba4a7a6d@github.com> On Wed, 2 Mar 2022 09:25:35 GMT, Prasanta Sadhukhan wrote: > I would rather keep this PR simple just to replace @exception with @throws. Other anomalies/idiosyncracies can be tackled in separate issue, if you dont mind. I didn't make myself clear: I absolutely agree with your approach. If it were only one case to address, but there are too many to include under this issue. Simple changes that address one issue at a time are easier to review. In fact, I found quite a few inaccuracies in javadoc while reviewing this change. Have you submitted a bug to address the inconsistency with usage of periods? If not, I'll submit one. ------------- PR: https://git.openjdk.java.net/jdk/pull/7644 From psadhukhan at openjdk.java.net Wed Mar 2 09:44:02 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 2 Mar 2022 09:44:02 GMT Subject: RFR: 8282473: Refactor swing classes javadoc to use @throws instead of @exception In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 10:16:16 GMT, Prasanta Sadhukhan wrote: > Prevailing JDK coding practices use "@throws" rather than "@exception". > Refactor existing swing classes javadoc to use @throws No, you can submit.. ------------- PR: https://git.openjdk.java.net/jdk/pull/7644 From aivanov at openjdk.java.net Wed Mar 2 09:56:06 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 2 Mar 2022 09:56:06 GMT Subject: RFR: 8281569: Create tests for Frame.setMinimumSize() method [v7] In-Reply-To: References: Message-ID: <4sbdmYr4Zf2u1bWeNJkT8nsvedKeirT-2PnrTEJ_rmI=.f3d4376e-7ce4-4d33-a67d-faa1d599036c@github.com> On Wed, 2 Mar 2022 06:10:42 GMT, Srinivas Mandalika wrote: >> Create tests for setMinimumSize functionality >> Create a test which verifies that on increasing the Frame's minimumSize, it gets reflected in the subsequent getSize call >> Create a test which verifies that on frame resizes back to minimumSize on calling pack. > > Srinivas Mandalika has updated the pull request incrementally with one additional commit since the last revision: > > Review comments fixed: field declared on different line, finally block on same line, changed to +=,.. Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7444 From dmarkov at openjdk.java.net Wed Mar 2 12:22:09 2022 From: dmarkov at openjdk.java.net (Dmitry Markov) Date: Wed, 2 Mar 2022 12:22:09 GMT Subject: RFR: 8282473: Refactor swing classes javadoc to use @throws instead of @exception In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 10:16:16 GMT, Prasanta Sadhukhan wrote: > Prevailing JDK coding practices use "@throws" rather than "@exception". > Refactor existing swing classes javadoc to use @throws Marked as reviewed by dmarkov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7644 From smandalika at openjdk.java.net Wed Mar 2 13:12:04 2022 From: smandalika at openjdk.java.net (Srinivas Mandalika) Date: Wed, 2 Mar 2022 13:12:04 GMT Subject: RFR: 8281569: Create tests for Frame.setMinimumSize() method [v7] In-Reply-To: <4sbdmYr4Zf2u1bWeNJkT8nsvedKeirT-2PnrTEJ_rmI=.f3d4376e-7ce4-4d33-a67d-faa1d599036c@github.com> References: <4sbdmYr4Zf2u1bWeNJkT8nsvedKeirT-2PnrTEJ_rmI=.f3d4376e-7ce4-4d33-a67d-faa1d599036c@github.com> Message-ID: On Wed, 2 Mar 2022 09:52:45 GMT, Alexey Ivanov wrote: >> Srinivas Mandalika has updated the pull request incrementally with one additional commit since the last revision: >> >> Review comments fixed: field declared on different line, finally block on same line, changed to +=,.. > > Marked as reviewed by aivanov (Reviewer). @aivanov-jdk Can you please sponsor my changes?. ------------- PR: https://git.openjdk.java.net/jdk/pull/7444 From psadhukhan at openjdk.java.net Wed Mar 2 13:21:54 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 2 Mar 2022 13:21:54 GMT Subject: RFR: 6911375: mouseWheel has no effect without vertical scrollbar [v3] In-Reply-To: References: Message-ID: > It is observed that mouseWheel doesn't scroll through a list if there is no scrollbar even though pressing PageUp/Down using keyboard moves up/down the list. > Issue stems from the fact that BasicScrollPaneUI.Handler.mouseWheelMoved() skips the mouse wheel events when the scrollbar is not visible. > Fixed to handle mouseWheelEvent even though scrollbar is not visible with the assumption that if both vertical and horizontal scrollbar is not visible, then assume default orientation to be vertical. > > Existing jtreg, jck tests are ok. All L&F works well with the fix and testcase works in all platforms except on mac which will be handled separately. Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: Add volatile keyword in test ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7585/files - new: https://git.openjdk.java.net/jdk/pull/7585/files/700a98e6..3514278e Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7585&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7585&range=01-02 Stats: 4 lines in 1 file changed: 1 ins; 0 del; 3 mod Patch: https://git.openjdk.java.net/jdk/pull/7585.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7585/head:pull/7585 PR: https://git.openjdk.java.net/jdk/pull/7585 From aivanov at openjdk.java.net Wed Mar 2 13:21:54 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 2 Mar 2022 13:21:54 GMT Subject: RFR: 6911375: mouseWheel has no effect without vertical scrollbar [v3] In-Reply-To: References: Message-ID: On Wed, 2 Mar 2022 13:17:28 GMT, Prasanta Sadhukhan wrote: >> It is observed that mouseWheel doesn't scroll through a list if there is no scrollbar even though pressing PageUp/Down using keyboard moves up/down the list. >> Issue stems from the fact that BasicScrollPaneUI.Handler.mouseWheelMoved() skips the mouse wheel events when the scrollbar is not visible. >> Fixed to handle mouseWheelEvent even though scrollbar is not visible with the assumption that if both vertical and horizontal scrollbar is not visible, then assume default orientation to be vertical. >> >> Existing jtreg, jck tests are ok. All L&F works well with the fix and testcase works in all platforms except on mac which will be handled separately. > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Add volatile keyword in test Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7585 From psadhukhan at openjdk.java.net Wed Mar 2 13:21:55 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 2 Mar 2022 13:21:55 GMT Subject: RFR: 6911375: mouseWheel has no effect without vertical scrollbar [v2] In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 16:38:30 GMT, Alexey Ivanov wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: >> >> Fix test comments > > test/jdk/javax/swing/JScrollPane/TestMouseWheelScroll.java line 48: > >> 46: static Point p; >> 47: static int width; >> 48: static int height; > > These three should also be volatile, shouldn't they? yes > test/jdk/javax/swing/JScrollPane/TestMouseWheelScroll.java line 101: > >> 99: robot.delay(1000); >> 100: robot.mouseWheel(1); >> 101: robot.delay(500); > > You don't call `waitForIdle` or `delay` after the mouse click because it just selects an item but doesn't change the position, right? added ------------- PR: https://git.openjdk.java.net/jdk/pull/7585 From psadhukhan at openjdk.java.net Wed Mar 2 13:21:55 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Wed, 2 Mar 2022 13:21:55 GMT Subject: RFR: 6911375: mouseWheel has no effect without vertical scrollbar [v3] In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 16:41:18 GMT, Alexey Ivanov wrote: >> This testcase was failing on macos which needs to be debugged, so it is excluded in mac for now. But as I told in description, the fix is not affecting other regression test in macos. >> It's working on other platforms and I have some issue in my local mac system so I will take this debugging of macos once I sort my system. >> >> Regarding your other question, the orientation will be horizontal. Tested against javax/swing/JScrollPane/HorizontalMouseWheelOnShiftPressed/HorizontalMouseWheelOnShiftPressed.java > > I see, thank you for explanation. Does it make sense to put the test in ProblemList.txt for macOS, so that it's not forgotten the test is to be run on macOS too? We problemlist a test if it fails in a particular platform whereas in this case it is taken care of by @requires so it will not fail. But no worries, it's in my radar to take it up. I will raise a JBS as soon as this is committed as I need to mention about this test in JBS description. ------------- PR: https://git.openjdk.java.net/jdk/pull/7585 From aivanov at openjdk.java.net Wed Mar 2 13:21:55 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 2 Mar 2022 13:21:55 GMT Subject: RFR: 6911375: mouseWheel has no effect without vertical scrollbar [v3] In-Reply-To: References: Message-ID: On Wed, 2 Mar 2022 13:14:08 GMT, Prasanta Sadhukhan wrote: >> I see, thank you for explanation. Does it make sense to put the test in ProblemList.txt for macOS, so that it's not forgotten the test is to be run on macOS too? > > We problemlist a test if it fails in a particular platform whereas in this case it is taken care of by @requires so it will not fail. > But no worries, it's in my radar to take it up. I will raise a JBS as soon as this is committed as I need to mention about this test in JBS description. Thanks! ------------- PR: https://git.openjdk.java.net/jdk/pull/7585 From aivanov at openjdk.java.net Wed Mar 2 14:54:55 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 2 Mar 2022 14:54:55 GMT Subject: RFR: 8282473: Refactor swing classes javadoc to use @throws instead of @exception In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 10:16:16 GMT, Prasanta Sadhukhan wrote: > Prevailing JDK coding practices use "@throws" rather than "@exception". > Refactor existing swing classes javadoc to use @throws Submitted: - [JDK-8282539](https://bugs.openjdk.java.net/browse/JDK-8282539): Remove ending punctuation from @throws HeadlessException - [JDK-8282546](https://bugs.openjdk.java.net/browse/JDK-8282546): Remove ending punctuation from @throws in java.desktop I decided to separate these two cases. ------------- PR: https://git.openjdk.java.net/jdk/pull/7644 From aivanov at openjdk.java.net Wed Mar 2 16:22:01 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Wed, 2 Mar 2022 16:22:01 GMT Subject: RFR: 8282473: Refactor swing classes javadoc to use @throws instead of @exception In-Reply-To: References: Message-ID: <9zvPmVRaNuXTPvDdAKFQoc0UrUrbFsw7608FUjNLmGU=.1937c3e9-1faa-463c-81e5-c80972f61ba1@github.com> On Tue, 1 Mar 2022 20:13:30 GMT, Alexey Ivanov wrote: >> Prevailing JDK coding practices use "@throws" rather than "@exception". >> Refactor existing swing classes javadoc to use @throws > > src/java.desktop/share/classes/javax/swing/JRootPane.java line 582: > >> 580: * @param content the Container to use for component-contents >> 581: * @throws java.awt.IllegalComponentStateException (a runtime >> 582: * exception) if the content pane parameter is null > > Shall we remove _?(a runtime exception)?_ note? Submitted [JDK-8282562](https://bugs.openjdk.java.net/browse/JDK-8282562): Remove "(a runtime exception)" from javadoc for unchecked exceptions in java.desktop ------------- PR: https://git.openjdk.java.net/jdk/pull/7644 From achung at openjdk.java.net Wed Mar 2 18:13:07 2022 From: achung at openjdk.java.net (Alisen Chung) Date: Wed, 2 Mar 2022 18:13:07 GMT Subject: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor In-Reply-To: References: Message-ID: <032ug3NHtrq71T-LPnGGJVT-OIqPe2V8QjOJXPfyOdY=.3ec1e9ee-e0ed-4437-b918-87b5ece945b1@github.com> On Mon, 14 Feb 2022 23:26:25 GMT, Sergey Bylokhov wrote: >> I think there's a rounding error when doing x.5 scalings causing the drawing area to be one pixel too small on the left side > > Can you please check how it will work for the undecorated frame, if it will be fine then the problem is somehow related to the size of the insets. @mrserb I've been trying to manually adjust the insets/variables to see what could be causing the issue, but only decreasing borderX by 1 seems to fix the issue. Could you clarify what you can gather from the undecorated frame working? ------------- PR: https://git.openjdk.java.net/jdk/pull/7449 From naoto at openjdk.java.net Wed Mar 2 18:29:03 2022 From: naoto at openjdk.java.net (Naoto Sato) Date: Wed, 2 Mar 2022 18:29:03 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack In-Reply-To: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Tue, 1 Mar 2022 08:31:47 GMT, Prasanta Sadhukhan wrote: > Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts I thought there is some mechanism to chain physical fonts, such as if `glim.ttc` is not available for `korean` then use `malgun.ttf` instead, but I may be wrong here. @prrace, can you comment on it? If there is no such fallback mechanism, `Batang` should also be replaced with `Malgun Gothic`, otherwise it would show blobs. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From cushon at openjdk.java.net Wed Mar 2 18:47:09 2022 From: cushon at openjdk.java.net (Liam Miller-Cushon) Date: Wed, 2 Mar 2022 18:47:09 GMT Subject: RFR: 8277817: java/awt/dnd/BadSerializationTest/BadSerializationTest.java failed: ClassNotFoundException: com.apple.laf.AquaImageFactory$SystemColorProxy [v10] In-Reply-To: <8bgd8AjF-ev0GEN5E0CUBZIU61-bFTqqyn26dsTrFBU=.d6a81200-5160-46b0-ae4c-e28a67649a9d@github.com> References: <8bgd8AjF-ev0GEN5E0CUBZIU61-bFTqqyn26dsTrFBU=.d6a81200-5160-46b0-ae4c-e28a67649a9d@github.com> Message-ID: On Tue, 1 Feb 2022 16:55:16 GMT, Liam Miller-Cushon wrote: > Hi, is there any feedback on this change, or the alternate fix in #7100 that updates the binary files, or the related cleanup in #7099? Friendly ping. My opinion is that https://github.com/openjdk/jdk/pull/7099 is a clear improvement to the current test setup, because it preserves the existing behaviour without relying on patching the JDK to disable constructor validation in `DragGestureEvent` and `DragGestureRecognizer` to generate the serialized objects. Additionally, I think it's worth considering the fix in https://github.com/openjdk/jdk/pull/7100. It includes a minimal fix to the `uninstallDefaults` behaviour of `BasicPanelUI`, which is the only broken uninstall behaviour exercised by the current test, and allows moving the test off the allowlist. However if the decision is that those approaches aren't worth pursuing, and it would be better to wait to set up comprehensive test coverage for the uninstall bugs in this package ([JDK-8278620](https://bugs.openjdk.java.net/browse/JDK-8278620)), I'm happy to drop these PRs. Thanks! ------------- PR: https://git.openjdk.java.net/jdk/pull/6603 From duke at openjdk.java.net Wed Mar 2 20:33:53 2022 From: duke at openjdk.java.net (lawrence.andrews) Date: Wed, 2 Mar 2022 20:33:53 GMT Subject: RFR: 8281284 : Write JSlider accessibility test [v3] In-Reply-To: References: Message-ID: > This testcase covers both old and new api ( api added in jdk17 getAccessibleAction(), getAccessibleActionCount() , getAccessibleActionDescription ) > > @shurymury > @savoptik lawrence.andrews has updated the pull request incrementally with one additional commit since the last revision: Add JSlider accessibility test ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7355/files - new: https://git.openjdk.java.net/jdk/pull/7355/files/0e1f269f..cc0f6e41 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7355&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7355&range=01-02 Stats: 344 lines in 2 files changed: 96 ins; 248 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/7355.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7355/head:pull/7355 PR: https://git.openjdk.java.net/jdk/pull/7355 From duke at openjdk.java.net Wed Mar 2 20:33:54 2022 From: duke at openjdk.java.net (lawrence.andrews) Date: Wed, 2 Mar 2022 20:33:54 GMT Subject: RFR: 8281284 : Write JSlider accessibility test [v2] In-Reply-To: <7VvKxnFl7P5VP0YzN9JRMhwNZTPShZHU8TKYQgxkW_k=.1a1d6447-19a3-4480-8ea6-1acd9632f8fc@github.com> References: <7VvKxnFl7P5VP0YzN9JRMhwNZTPShZHU8TKYQgxkW_k=.1a1d6447-19a3-4480-8ea6-1acd9632f8fc@github.com> Message-ID: On Sun, 6 Feb 2022 08:22:04 GMT, Sergey Bylokhov wrote: >> lawrence.andrews has updated the pull request incrementally with one additional commit since the last revision: >> >> Added @key headful > > test/jdk/javax/accessibility/JSlider/JSliderAccessibilityTest.java line 2: > >> 1: /* >> 2: * Copyright (c) 2019, 2022 Oracle and/or its affiliates. All rights reserved. > > Is it a new test or it is based on some old code? fixed > test/jdk/javax/accessibility/JSlider/JSliderAccessibilityTest.java line 125: > >> 123: throw new RuntimeException("AccessibleStateSet shouldn't contain both HORIZONTAL and VERTICAL"); >> 124: } >> 125: if (jSlider.getOrientation() != jSlider.HORIZONTAL) { > > Shouldn't the test creates a sliders using different orientations? converted the testcase to manual as suggested by savoptik ------------- PR: https://git.openjdk.java.net/jdk/pull/7355 From duke at openjdk.java.net Wed Mar 2 20:33:55 2022 From: duke at openjdk.java.net (lawrence.andrews) Date: Wed, 2 Mar 2022 20:33:55 GMT Subject: RFR: 8281284 : Write JSlider accessibility test [v2] In-Reply-To: References: <4jjNGD2edw9UK5P1W1WMgthuJxr1y1bP1hkswXlt3Uo=.c477cd79-81ed-4af7-a1d8-2e3451b8284f@github.com> Message-ID: On Wed, 23 Feb 2022 02:03:24 GMT, lawrence.andrews wrote: >> test/jdk/javax/accessibility/JSlider/JSliderAccessibilityTest.java line 68: >> >>> 66: frame.setLocationRelativeTo(null); >>> 67: frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE); >>> 68: frame.setVisible(true); >> >> Please add instructions to the test on how to take it on all intended platforms. And also when to consider that the test is passed correctly. > > @savoptik this is automated test. Are you suggesting me to change the test to manual ? converted the test to manual ------------- PR: https://git.openjdk.java.net/jdk/pull/7355 From duke at openjdk.java.net Wed Mar 2 20:41:59 2022 From: duke at openjdk.java.net (lawrence.andrews) Date: Wed, 2 Mar 2022 20:41:59 GMT Subject: RFR: 8281284 : Write JSlider accessibility test [v4] In-Reply-To: References: Message-ID: > This testcase covers both old and new api ( api added in jdk17 getAccessibleAction(), getAccessibleActionCount() , getAccessibleActionDescription ) > > @shurymury > @savoptik lawrence.andrews has updated the pull request incrementally with one additional commit since the last revision: Fixed the white character ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7355/files - new: https://git.openjdk.java.net/jdk/pull/7355/files/cc0f6e41..563e41e9 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7355&range=03 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7355&range=02-03 Stats: 3 lines in 1 file changed: 0 ins; 2 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/7355.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7355/head:pull/7355 PR: https://git.openjdk.java.net/jdk/pull/7355 From prr at openjdk.java.net Wed Mar 2 20:59:45 2022 From: prr at openjdk.java.net (Phil Race) Date: Wed, 2 Mar 2022 20:59:45 GMT Subject: RFR: 8282577: ICC_Profile.setTagData() invalidates the profile Message-ID: As per the bug eval, we are returning a pointer to a native raw LittleCMS profile and it should be a cooked one. The simple test fails before the fix with ~/jdk17/Contents/Home/bin/java SetTagDataValidation Exception in thread "main" java.awt.color.CMMException: LCMS error 13: Couldn't link the profiles at java.desktop/sun.java2d.cmm.lcms.LCMS.createNativeTransform(Native Method) at java.desktop/sun.java2d.cmm.lcms.LCMS.createTransform(LCMS.java:104) at java.desktop/sun.java2d.cmm.lcms.LCMSTransform.doTransform(LCMSTransform.java:143) at java.desktop/sun.java2d.cmm.lcms.LCMSTransform.colorConvert(LCMSTransform.java:633) at java.desktop/java.awt.color.ICC_ColorSpace.toRGB(ICC_ColorSpace.java:224) at SetTagDataValidation.main(SetTagDataValidation.java:61) and with the fix just prints java SetTagDataValidation 0.4 0.5000076 0.6 ------------- Commit messages: - 8282577: ICC_Profile.setTagData() invalidates the profile Changes: https://git.openjdk.java.net/jdk/pull/7668/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7668&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8282577 Stats: 79 lines in 2 files changed: 77 ins; 0 del; 2 mod Patch: https://git.openjdk.java.net/jdk/pull/7668.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7668/head:pull/7668 PR: https://git.openjdk.java.net/jdk/pull/7668 From prr at openjdk.java.net Wed Mar 2 23:17:05 2022 From: prr at openjdk.java.net (Phil Race) Date: Wed, 2 Mar 2022 23:17:05 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack In-Reply-To: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Tue, 1 Mar 2022 08:31:47 GMT, Prasanta Sadhukhan wrote: > Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts There's no chaining mechanism in the way you describe it .. it would have to be in our fallback list. Batang and Gulim are the fonts that have been available since .. well at least XP. https://docs.microsoft.com/en-us/typography/font-list/batang https://docs.microsoft.com/en-us/typography/font-list/gulim Malun Gothic only came in with Windows 7 :- https://docs.microsoft.com/en-us/typography/font-list/malgun-gothic I think it reasonable to put malun.ttf in the "allfonts" as a fallback if it really is pre-installed on ALL systems. But actual Korean users really should be installing the Korean font pack regardless and then we should pick the best available font. Maybe its the new ones now. Replacing them would not have been possible in the past but now Vista isn't supported Note that Gulim is a Sans Serif font, Batang a Serif font. Malun Gothic is described as suitable for on screen display - it sounds like a replacement for Gulim but not for Batang so it is correct you aren't replacing that. But is there a new Serif font available too ? Please investigate / check. And it seems (ah I see Naoto spotted this too) there is a Bold version of Malgun Gothic in its own ttf file : Malgunbd.ttf So we should start using that. To your point about it not being in the pre-installed fonts it doesn't matter for the reason I gave. One other thing you are going to have to check is the rendering of other scripts when running in a Korean locale. Batang lists support for Arabic .. Malun does not .. but then neither does Gulim .. so it may be no worse. But sequence.fallback doesn't have any obvious Arabic support. We used to be able to count on Lucida Sans for that .. but may be there's another gap there. Mainly make sure you don't increase that gap. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From smandalika at openjdk.java.net Thu Mar 3 03:30:10 2022 From: smandalika at openjdk.java.net (Srinivas Mandalika) Date: Thu, 3 Mar 2022 03:30:10 GMT Subject: Integrated: 8281569: Create tests for Frame.setMinimumSize() method In-Reply-To: References: Message-ID: <2dhv8XQthaOzrB8Oqz7BAMTDDEwuMBSvLf7NM3yBD1g=.7ee7952e-032e-47fb-8cf1-d7637ecf9d06@github.com> On Fri, 11 Feb 2022 17:28:09 GMT, Srinivas Mandalika wrote: > Create tests for setMinimumSize functionality > Create a test which verifies that on increasing the Frame's minimumSize, it gets reflected in the subsequent getSize call > Create a test which verifies that on frame resizes back to minimumSize on calling pack. This pull request has now been integrated. Changeset: d0eb6fa2 Author: Srinivas Mandalika Committer: Abdul Kolarkunnu URL: https://git.openjdk.java.net/jdk/commit/d0eb6fa2204c4112ef57ffa7579e209ebee62551 Stats: 185 lines in 2 files changed: 185 ins; 0 del; 0 mod 8281569: Create tests for Frame.setMinimumSize() method Reviewed-by: aivanov ------------- PR: https://git.openjdk.java.net/jdk/pull/7444 From mvs at openjdk.java.net Thu Mar 3 06:19:36 2022 From: mvs at openjdk.java.net (Manukumar V S) Date: Thu, 3 Mar 2022 06:19:36 GMT Subject: RFR: 8282402: Create a regression test for JDK-4666101 [v2] In-Reply-To: References: Message-ID: > Create a regression test for [JDK-4666101](https://bugs.openjdk.java.net/browse/JDK-4666101). > > DESCRIPTION OF THE ACTUAL PROBLEM : > In a JEditorPane, the down arrow has no effect after you add text on a line preceding a blank line. > > Testing: > As the original issue was reported in JDK 1.4.0 and it was fixed in JDK 1.4.2, I have verified the working of this test in those Java versions and the results are as follows: > > Java 1.4.0 -> Test Failed. > $ ./j2sdk1.4.0/bin/java JEditorBugWin > Caret Updated : 3 > Test Failed, Down Arrow Key Not Functional, expected caret position is 5, but actual is 4 > > > Java 1.4.2 -> Test Passed. > $ ./j2sdk1.4.2/bin/java JEditorBugWin > Caret Updated : 5 > Test Passed, Down Arrow Key Functional Manukumar V S has updated the pull request incrementally with one additional commit since the last revision: Take a screenshot when the test fails, helpful for debugging failures ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7623/files - new: https://git.openjdk.java.net/jdk/pull/7623/files/95148f72..5dec2a8a Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7623&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7623&range=00-01 Stats: 18 lines in 1 file changed: 18 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/7623.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7623/head:pull/7623 PR: https://git.openjdk.java.net/jdk/pull/7623 From psadhukhan at openjdk.java.net Thu Mar 3 07:45:46 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Thu, 3 Mar 2022 07:45:46 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v2] In-Reply-To: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: <3F6rwtKnmNxHmNk_JwNo-ovGQCtROCOTqq9JYEkxCnU=.715f8bd7-4df9-47dc-85f3-a598d01e8e78@github.com> > Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: Add bold version ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7643/files - new: https://git.openjdk.java.net/jdk/pull/7643/files/0066bfd5..9c098f99 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7643&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7643&range=00-01 Stats: 4 lines in 1 file changed: 1 ins; 0 del; 3 mod Patch: https://git.openjdk.java.net/jdk/pull/7643.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7643/head:pull/7643 PR: https://git.openjdk.java.net/jdk/pull/7643 From psadhukhan at openjdk.java.net Thu Mar 3 07:50:05 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Thu, 3 Mar 2022 07:50:05 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Wed, 2 Mar 2022 23:13:48 GMT, Phil Race wrote: > But is there a new Serif font available too ? I could not find any replacement for Batang serif font so have kept it same. > Bold version of Malgun Gothic in its own ttf file : Malgunbd.ttf So we should start using that Added > Arabic support. Tested with java/awt/font/NumericShaper/EasternArabicTest and MTTest..Rendered same before and after this fix. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From mvs at openjdk.java.net Thu Mar 3 09:06:05 2022 From: mvs at openjdk.java.net (Manukumar V S) Date: Thu, 3 Mar 2022 09:06:05 GMT Subject: RFR: 8282343: Create a regression test for JDK-4518432 In-Reply-To: References: Message-ID: On Thu, 24 Feb 2022 11:46:58 GMT, Manukumar V S wrote: > Create a regression test for [JDK-4518432](https://bugs.openjdk.java.net/browse/JDK-4518432). > > Issue: Copying from JTree and then changing the data in the component that was copied from, can cause pastes to use > the new edited data. > > This test verifies that copying from JTree node and then changing the data in the component that was copied from, > is not causing the pastes to use the new edited data instead of the original copied data. > > Testing: > Java 1.4.0 -> Test Failed. > $ ./j2sdk1.4.0/bin/java JTreeWin > Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel > Test Failed in javax.swing.plaf.metal.MetalLookAndFeel, Expected : "colorsblue", but actual : "ctextolorsblue" > Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel > Test Failed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel, Expected : "colorsblue", but actual : "ctextolorsblue" > > Java 1.4.1 -> Test Passed. > $ ./j2sdk1.4.1/bin/java JTreeWin > Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel > Test Passed in javax.swing.plaf.metal.MetalLookAndFeel > Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel > Test Passed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel @aivanov-jdk Can you please review this. ------------- PR: https://git.openjdk.java.net/jdk/pull/7605 From mvs at openjdk.java.net Thu Mar 3 09:06:06 2022 From: mvs at openjdk.java.net (Manukumar V S) Date: Thu, 3 Mar 2022 09:06:06 GMT Subject: RFR: 8282402: Create a regression test for JDK-4666101 [v2] In-Reply-To: References: Message-ID: <1rNUgM39ccKhEH1T_7Y8kHqEX4y85NyJS2PoMr3djHE=.a21d4f7a-e011-4fdd-bc5e-f5617838eeb6@github.com> On Thu, 3 Mar 2022 06:19:36 GMT, Manukumar V S wrote: >> Create a regression test for [JDK-4666101](https://bugs.openjdk.java.net/browse/JDK-4666101). >> >> DESCRIPTION OF THE ACTUAL PROBLEM : >> In a JEditorPane, the down arrow has no effect after you add text on a line preceding a blank line. >> >> Testing: >> As the original issue was reported in JDK 1.4.0 and it was fixed in JDK 1.4.2, I have verified the working of this test in those Java versions and the results are as follows: >> >> Java 1.4.0 -> Test Failed. >> $ ./j2sdk1.4.0/bin/java JEditorBugWin >> Caret Updated : 3 >> Test Failed, Down Arrow Key Not Functional, expected caret position is 5, but actual is 4 >> >> >> Java 1.4.2 -> Test Passed. >> $ ./j2sdk1.4.2/bin/java JEditorBugWin >> Caret Updated : 5 >> Test Passed, Down Arrow Key Functional > > Manukumar V S has updated the pull request incrementally with one additional commit since the last revision: > > Take a screenshot when the test fails, helpful for debugging failures @aivanov-jdk Can you please review this. ------------- PR: https://git.openjdk.java.net/jdk/pull/7623 From psadhukhan at openjdk.java.net Thu Mar 3 09:54:13 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Thu, 3 Mar 2022 09:54:13 GMT Subject: Integrated: 8037965: NullPointerException in TextLayout.getBaselineFromGraphic() for JTextComponents In-Reply-To: References: Message-ID: <9q7yeKpWiKVcnO4mRDBldtuVpwftDCKD0_-oXMtqbmI=.0cc408cf-4923-4fc3-b05a-211eb8bc53b3@github.com> On Wed, 16 Feb 2022 07:44:27 GMT, Prasanta Sadhukhan wrote: > When invalid unicode codepoints 0xFFFF and 0x10000 are added to an empty javax.swing.JTextComponent (JTextArea or JTextField), the view cannot be updated due to a NullPointerException in TextLayout.getBaselineFromGraphic() > owing to the fact GraphicAttribute is null for these invalid codepoints. > > TextLayout.getBaselineFromGraphic() is called from [TextLayout.standardInit](https://github.com/openjdk/jdk/blame/master/src/java.desktop/share/classes/java/awt/font/TextLayout.java#L647) and [TextMeasurer.initAll](https://github.com/openjdk/jdk/blame/master/src/java.desktop/share/classes/java/awt/font/TextMeasurer.java#L249) where it relies on CHAR_REPLACEMENT TextAttribute > and its spec says "The default value is null, indicating that the standard glyphs provided by the font should be used. " so if invalid codepoints is used, the default value is not changed so we need to be ready for null value too. > > Since TextAttribute_CHAR_REPLACEMENT says "It is required for correct positioning of 'inline' components within a line when bidirectional reordering is performed. Each character (Unicode code point) will be rendered using the provided GraphicAttribute." so > Fixed by checking if GraphicAttribute is null, then keep the baseline to default value (which is 0) by returning default baseline (which is ROMAN_BASELINE) > > Existing jtreg,jck tests are green. This pull request has now been integrated. Changeset: 0402a288 Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/0402a288f309d589d1dc8860e95e4ed42a1ef9f4 Stats: 41 lines in 2 files changed: 41 ins; 0 del; 0 mod 8037965: NullPointerException in TextLayout.getBaselineFromGraphic() for JTextComponents Reviewed-by: prr, aivanov ------------- PR: https://git.openjdk.java.net/jdk/pull/7490 From psadhukhan at openjdk.java.net Thu Mar 3 09:57:10 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Thu, 3 Mar 2022 09:57:10 GMT Subject: Integrated: 6911375: mouseWheel has no effect without vertical scrollbar In-Reply-To: References: Message-ID: On Wed, 23 Feb 2022 04:24:41 GMT, Prasanta Sadhukhan wrote: > It is observed that mouseWheel doesn't scroll through a list if there is no scrollbar even though pressing PageUp/Down using keyboard moves up/down the list. > Issue stems from the fact that BasicScrollPaneUI.Handler.mouseWheelMoved() skips the mouse wheel events when the scrollbar is not visible. > Fixed to handle mouseWheelEvent even though scrollbar is not visible with the assumption that if both vertical and horizontal scrollbar is not visible, then assume default orientation to be vertical. > > Existing jtreg, jck tests are ok. All L&F works well with the fix and testcase works in all platforms except on mac which will be handled separately. This pull request has now been integrated. Changeset: 832729b4 Author: Prasanta Sadhukhan URL: https://git.openjdk.java.net/jdk/commit/832729b41e690d6f05da71997bbe2a74b2d6dada Stats: 138 lines in 2 files changed: 135 ins; 1 del; 2 mod 6911375: mouseWheel has no effect without vertical scrollbar Reviewed-by: aivanov ------------- PR: https://git.openjdk.java.net/jdk/pull/7585 From psadhukhan at openjdk.java.net Thu Mar 3 12:33:23 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Thu, 3 Mar 2022 12:33:23 GMT Subject: RFR: 8282602: Refactor java/awt classes javadoc to use @throws instead of @exception Message-ID: Prevailing JDK coding practices use "@throws" rather than "@exception". Refactor existing java/awt classes javadoc to use @throws ------------- Commit messages: - Fix - Fix Changes: https://git.openjdk.java.net/jdk/pull/7675/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7675&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8282602 Stats: 350 lines in 73 files changed: 0 ins; 0 del; 350 mod Patch: https://git.openjdk.java.net/jdk/pull/7675.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7675/head:pull/7675 PR: https://git.openjdk.java.net/jdk/pull/7675 From aivanov at openjdk.java.net Thu Mar 3 15:39:06 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Thu, 3 Mar 2022 15:39:06 GMT Subject: RFR: 8282402: Create a regression test for JDK-4666101 [v2] In-Reply-To: References: Message-ID: <31bO4g20K1sGmCiJoO3yJoDGLxunJ0C7lgiahDHU518=.e3b0566d-413a-43ce-b066-3e8744cb8b39@github.com> On Thu, 3 Mar 2022 06:19:36 GMT, Manukumar V S wrote: >> Create a regression test for [JDK-4666101](https://bugs.openjdk.java.net/browse/JDK-4666101). >> >> DESCRIPTION OF THE ACTUAL PROBLEM : >> In a JEditorPane, the down arrow has no effect after you add text on a line preceding a blank line. >> >> Testing: >> As the original issue was reported in JDK 1.4.0 and it was fixed in JDK 1.4.2, I have verified the working of this test in those Java versions and the results are as follows: >> >> Java 1.4.0 -> Test Failed. >> $ ./j2sdk1.4.0/bin/java JEditorBugWin >> Caret Updated : 3 >> Test Failed, Down Arrow Key Not Functional, expected caret position is 5, but actual is 4 >> >> >> Java 1.4.2 -> Test Passed. >> $ ./j2sdk1.4.2/bin/java JEditorBugWin >> Caret Updated : 5 >> Test Passed, Down Arrow Key Functional > > Manukumar V S has updated the pull request incrementally with one additional commit since the last revision: > > Take a screenshot when the test fails, helpful for debugging failures Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7623 From prr at openjdk.java.net Thu Mar 3 16:10:08 2022 From: prr at openjdk.java.net (Phil Race) Date: Thu, 3 Mar 2022 16:10:08 GMT Subject: RFR: 8236907: JTable added to nested panels does not paint last visible row [v6] In-Reply-To: References: Message-ID: On Tue, 22 Feb 2022 04:38:21 GMT, Tejesh R wrote: >> Issue in painting last row of JTable due to decrement in `MaxRow `during computation. Differences in `MaxRow `computation between `TablePrintable` and `BasicTableUI `class. The Bug is fixed in both the classes. Test Case added to validate the same. > > Tejesh R has updated the pull request incrementally with one additional commit since the last revision: > > 8236907: JTable added to nested panels does not paint last Marked as reviewed by prr (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7422 From mvs at openjdk.java.net Thu Mar 3 16:16:50 2022 From: mvs at openjdk.java.net (Manukumar V S) Date: Thu, 3 Mar 2022 16:16:50 GMT Subject: RFR: 8282402: Create a regression test for JDK-4666101 [v3] In-Reply-To: References: Message-ID: > Create a regression test for [JDK-4666101](https://bugs.openjdk.java.net/browse/JDK-4666101). > > DESCRIPTION OF THE ACTUAL PROBLEM : > In a JEditorPane, the down arrow has no effect after you add text on a line preceding a blank line. > > Testing: > As the original issue was reported in JDK 1.4.0 and it was fixed in JDK 1.4.2, I have verified the working of this test in those Java versions and the results are as follows: > > Java 1.4.0 -> Test Failed. > $ ./j2sdk1.4.0/bin/java JEditorBugWin > Caret Updated : 3 > Test Failed, Down Arrow Key Not Functional, expected caret position is 5, but actual is 4 > > > Java 1.4.2 -> Test Passed. > $ ./j2sdk1.4.2/bin/java JEditorBugWin > Caret Updated : 5 > Test Passed, Down Arrow Key Functional Manukumar V S has updated the pull request incrementally with two additional commits since the last revision: - Removed one extra blank line before import static - Review comments fixed: Formatting changes, removed extra blank lines, removed casting for jEditorLoc.x ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7623/files - new: https://git.openjdk.java.net/jdk/pull/7623/files/5dec2a8a..dc26ec28 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7623&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7623&range=01-02 Stats: 12 lines in 1 file changed: 1 ins; 8 del; 3 mod Patch: https://git.openjdk.java.net/jdk/pull/7623.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7623/head:pull/7623 PR: https://git.openjdk.java.net/jdk/pull/7623 From naoto at openjdk.java.net Thu Mar 3 17:09:01 2022 From: naoto at openjdk.java.net (Naoto Sato) Date: Thu, 3 Mar 2022 17:09:01 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v2] In-Reply-To: <3F6rwtKnmNxHmNk_JwNo-ovGQCtROCOTqq9JYEkxCnU=.715f8bd7-4df9-47dc-85f3-a598d01e8e78@github.com> References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> <3F6rwtKnmNxHmNk_JwNo-ovGQCtROCOTqq9JYEkxCnU=.715f8bd7-4df9-47dc-85f3-a598d01e8e78@github.com> Message-ID: On Thu, 3 Mar 2022 07:45:46 GMT, Prasanta Sadhukhan wrote: >> Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Add bold version OK, so there is no way to chain the physical fonts. One other thing I noticed was that `GulimChe` is a `monospaced` font, so I would also leave it for `monospaced` properties, as `Malgun Gothic` is a proportional font. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From duke at openjdk.java.net Thu Mar 3 17:11:39 2022 From: duke at openjdk.java.net (lawrence.andrews) Date: Thu, 3 Mar 2022 17:11:39 GMT Subject: RFR: 8281284 : Write JSlider accessibility test [v5] In-Reply-To: References: Message-ID: > This testcase covers both old and new api ( api added in jdk17 getAccessibleAction(), getAccessibleActionCount() , getAccessibleActionDescription ) > > @shurymury > @savoptik lawrence.andrews has updated the pull request incrementally with one additional commit since the last revision: run the on windows and mac ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7355/files - new: https://git.openjdk.java.net/jdk/pull/7355/files/563e41e9..b815681d Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7355&range=04 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7355&range=03-04 Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/7355.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7355/head:pull/7355 PR: https://git.openjdk.java.net/jdk/pull/7355 From prr at openjdk.java.net Thu Mar 3 18:11:05 2022 From: prr at openjdk.java.net (Phil Race) Date: Thu, 3 Mar 2022 18:11:05 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v2] In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> <3F6rwtKnmNxHmNk_JwNo-ovGQCtROCOTqq9JYEkxCnU=.715f8bd7-4df9-47dc-85f3-a598d01e8e78@github.com> Message-ID: On Thu, 3 Mar 2022 17:05:52 GMT, Naoto Sato wrote: > OK, so there is no way to chain the physical fonts. > One other thing I noticed was that `GulimChe` is a `monospaced` font, so I would also leave it for `monospaced` properties, as `Malgun Gothic` is a proportional font. Good point. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From prr at openjdk.java.net Thu Mar 3 18:11:05 2022 From: prr at openjdk.java.net (Phil Race) Date: Thu, 3 Mar 2022 18:11:05 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v2] In-Reply-To: <3F6rwtKnmNxHmNk_JwNo-ovGQCtROCOTqq9JYEkxCnU=.715f8bd7-4df9-47dc-85f3-a598d01e8e78@github.com> References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> <3F6rwtKnmNxHmNk_JwNo-ovGQCtROCOTqq9JYEkxCnU=.715f8bd7-4df9-47dc-85f3-a598d01e8e78@github.com> Message-ID: On Thu, 3 Mar 2022 07:45:46 GMT, Prasanta Sadhukhan wrote: >> Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Add bold version make/data/fontconfig/windows.fontconfig.properties line 169: > 167: dialog.bolditalic.hebrew=David Bold > 168: dialog.bolditalic.japanese=MS Gothic > 169: dialog.bolditalic.korean=Malgun Gothic You should use the Bold version of the font here too - ie for bold italic - not just for bold - since it is clearly closer .. make/data/fontconfig/windows.fontconfig.properties line 184: > 182: dialoginput.bold.japanese=MS Gothic > 183: dialoginput.bold.korean=Malgun Gothic Bold > 184: Since dialoginput is usually the same as monospaced, per Naoto's comment, this should be reverted. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From aivanov at openjdk.java.net Thu Mar 3 18:12:02 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Thu, 3 Mar 2022 18:12:02 GMT Subject: RFR: 8282343: Create a regression test for JDK-4518432 In-Reply-To: References: Message-ID: On Thu, 24 Feb 2022 11:46:58 GMT, Manukumar V S wrote: > Create a regression test for [JDK-4518432](https://bugs.openjdk.java.net/browse/JDK-4518432). > > Issue: Copying from JTree and then changing the data in the component that was copied from, can cause pastes to use > the new edited data. > > This test verifies that copying from JTree node and then changing the data in the component that was copied from, > is not causing the pastes to use the new edited data instead of the original copied data. > > Testing: > Java 1.4.0 -> Test Failed. > $ ./j2sdk1.4.0/bin/java JTreeWin > Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel > Test Failed in javax.swing.plaf.metal.MetalLookAndFeel, Expected : "colorsblue", but actual : "ctextolorsblue" > Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel > Test Failed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel, Expected : "colorsblue", but actual : "ctextolorsblue" > > Java 1.4.1 -> Test Passed. > $ ./j2sdk1.4.1/bin/java JTreeWin > Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel > Test Passed in javax.swing.plaf.metal.MetalLookAndFeel > Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel > Test Passed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel Marked as reviewed by aivanov (Reviewer). test/jdk/javax/swing/JTree/4518432/JTreeNodeCopyPasteTest.java line 100: > 98: throw new RuntimeException("Test Failed, waited too long, " + > 99: "but the JTree can't gain focus for L&F: " + laf); > 100: } The tree is the only component on the frame. By default, it will get focus. The call to `waitForIdle` guarantees all the events are processed. Even if tree doesn't get focus, the next thing you do is click a node in the tree, which brings the focus. ------------- PR: https://git.openjdk.java.net/jdk/pull/7605 From aivanov at openjdk.java.net Thu Mar 3 19:07:59 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Thu, 3 Mar 2022 19:07:59 GMT Subject: RFR: 8282402: Create a regression test for JDK-4666101 [v3] In-Reply-To: References: Message-ID: On Thu, 3 Mar 2022 16:16:50 GMT, Manukumar V S wrote: >> Create a regression test for [JDK-4666101](https://bugs.openjdk.java.net/browse/JDK-4666101). >> >> DESCRIPTION OF THE ACTUAL PROBLEM : >> In a JEditorPane, the down arrow has no effect after you add text on a line preceding a blank line. >> >> Testing: >> As the original issue was reported in JDK 1.4.0 and it was fixed in JDK 1.4.2, I have verified the working of this test in those Java versions and the results are as follows: >> >> Java 1.4.0 -> Test Failed. >> $ ./j2sdk1.4.0/bin/java JEditorBugWin >> Caret Updated : 3 >> Test Failed, Down Arrow Key Not Functional, expected caret position is 5, but actual is 4 >> >> >> Java 1.4.2 -> Test Passed. >> $ ./j2sdk1.4.2/bin/java JEditorBugWin >> Caret Updated : 5 >> Test Passed, Down Arrow Key Functional > > Manukumar V S has updated the pull request incrementally with two additional commits since the last revision: > > - Removed one extra blank line before import static > - Review comments fixed: Formatting changes, removed extra blank lines, removed casting for jEditorLoc.x Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7623 From mvs at openjdk.java.net Thu Mar 3 19:33:07 2022 From: mvs at openjdk.java.net (Manukumar V S) Date: Thu, 3 Mar 2022 19:33:07 GMT Subject: Integrated: 8282402: Create a regression test for JDK-4666101 In-Reply-To: References: Message-ID: On Fri, 25 Feb 2022 15:37:10 GMT, Manukumar V S wrote: > Create a regression test for [JDK-4666101](https://bugs.openjdk.java.net/browse/JDK-4666101). > > DESCRIPTION OF THE ACTUAL PROBLEM : > In a JEditorPane, the down arrow has no effect after you add text on a line preceding a blank line. > > Testing: > As the original issue was reported in JDK 1.4.0 and it was fixed in JDK 1.4.2, I have verified the working of this test in those Java versions and the results are as follows: > > Java 1.4.0 -> Test Failed. > $ ./j2sdk1.4.0/bin/java JEditorBugWin > Caret Updated : 3 > Test Failed, Down Arrow Key Not Functional, expected caret position is 5, but actual is 4 > > > Java 1.4.2 -> Test Passed. > $ ./j2sdk1.4.2/bin/java JEditorBugWin > Caret Updated : 5 > Test Passed, Down Arrow Key Functional This pull request has now been integrated. Changeset: 1581e3fa Author: Manukumar V S Committer: Alexey Ivanov URL: https://git.openjdk.java.net/jdk/commit/1581e3faa06358f192799b3a89718028c7f6a24b Stats: 197 lines in 1 file changed: 197 ins; 0 del; 0 mod 8282402: Create a regression test for JDK-4666101 Reviewed-by: aivanov ------------- PR: https://git.openjdk.java.net/jdk/pull/7623 From aivanov at openjdk.java.net Thu Mar 3 20:49:05 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Thu, 3 Mar 2022 20:49:05 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v8] In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 23:27:57 GMT, DamonGuy wrote: >> Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. >> >> Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. >> >> Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. > > DamonGuy has updated the pull request incrementally with one additional commit since the last revision: > > Updated test to be headless using BufferedImage. Changes requested by aivanov (Reviewer). test/jdk/javax/swing/JButton/HtmlButtonImageTest/HtmlButtonImageTest.java line 42: > 40: import java.io.IOException; > 41: import java.io.File; > 42: import javax.imageio.ImageIO; Could you sort the imports, please? *Optimize Imports* or a similar command in your IDE should do the magic. test/jdk/javax/swing/JButton/HtmlButtonImageTest/HtmlButtonImageTest.java line 68: > 66: > 67: SwingUtilities.invokeAndWait(HtmlButtonImageTest::createButton); > 68: SwingUtilities.invokeAndWait(HtmlButtonImageTest::createImage); `paintButton` would be clearer. test/jdk/javax/swing/JButton/HtmlButtonImageTest/HtmlButtonImageTest.java line 100: > 98: } > 99: > 100: private static boolean checkRedness(int rgb) { `checkRed` or `checkRedColor`? test/jdk/javax/swing/JButton/HtmlButtonImageTest/HtmlButtonImageTest.java line 114: > 112: System.out.println("Passed"); > 113: } > 114: } It's usually recommended to end the file with a new line. ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From prr at openjdk.java.net Thu Mar 3 20:59:45 2022 From: prr at openjdk.java.net (Phil Race) Date: Thu, 3 Mar 2022 20:59:45 GMT Subject: RFR: 8282577: ICC_Profile.setTagData() invalidates the profile [v2] In-Reply-To: References: Message-ID: > As per the bug eval, we are returning a pointer to a native raw LittleCMS profile and it should be a cooked one. > The simple test fails before the fix with > ~/jdk17/Contents/Home/bin/java SetTagDataValidation > Exception in thread "main" java.awt.color.CMMException: LCMS error 13: Couldn't link the profiles > at java.desktop/sun.java2d.cmm.lcms.LCMS.createNativeTransform(Native Method) > at java.desktop/sun.java2d.cmm.lcms.LCMS.createTransform(LCMS.java:104) > at java.desktop/sun.java2d.cmm.lcms.LCMSTransform.doTransform(LCMSTransform.java:143) > at java.desktop/sun.java2d.cmm.lcms.LCMSTransform.colorConvert(LCMSTransform.java:633) > at java.desktop/java.awt.color.ICC_ColorSpace.toRGB(ICC_ColorSpace.java:224) > at SetTagDataValidation.main(SetTagDataValidation.java:61) > > and with the fix just prints > java SetTagDataValidation > 0.4 > 0.5000076 > 0.6 Phil Race has updated the pull request incrementally with one additional commit since the last revision: 8282577: ICC_Profile.setTagData() invalidates the profile ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7668/files - new: https://git.openjdk.java.net/jdk/pull/7668/files/e46c65c1..2ecf8211 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7668&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7668&range=00-01 Stats: 7 lines in 2 files changed: 6 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/7668.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7668/head:pull/7668 PR: https://git.openjdk.java.net/jdk/pull/7668 From prr at openjdk.java.net Thu Mar 3 21:27:36 2022 From: prr at openjdk.java.net (Phil Race) Date: Thu, 3 Mar 2022 21:27:36 GMT Subject: RFR: 8282577: ICC_Profile.setData(int, byte[]) invalidates the profile [v3] In-Reply-To: References: Message-ID: > As per the bug eval, we are returning a pointer to a native raw LittleCMS profile and it should be a cooked one. > The simple test fails before the fix with > ~/jdk17/Contents/Home/bin/java SetTagDataValidation > Exception in thread "main" java.awt.color.CMMException: LCMS error 13: Couldn't link the profiles > at java.desktop/sun.java2d.cmm.lcms.LCMS.createNativeTransform(Native Method) > at java.desktop/sun.java2d.cmm.lcms.LCMS.createTransform(LCMS.java:104) > at java.desktop/sun.java2d.cmm.lcms.LCMSTransform.doTransform(LCMSTransform.java:143) > at java.desktop/sun.java2d.cmm.lcms.LCMSTransform.colorConvert(LCMSTransform.java:633) > at java.desktop/java.awt.color.ICC_ColorSpace.toRGB(ICC_ColorSpace.java:224) > at SetTagDataValidation.main(SetTagDataValidation.java:61) > > and with the fix just prints > java SetTagDataValidation > 0.4 > 0.5000076 > 0.6 Phil Race has updated the pull request incrementally with one additional commit since the last revision: 8282577: ICC_Profile.setData(int, byte[]) invalidates the profile ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7668/files - new: https://git.openjdk.java.net/jdk/pull/7668/files/2ecf8211..5bf31b11 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7668&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7668&range=01-02 Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/7668.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7668/head:pull/7668 PR: https://git.openjdk.java.net/jdk/pull/7668 From duke at openjdk.java.net Thu Mar 3 21:27:45 2022 From: duke at openjdk.java.net (DamonGuy) Date: Thu, 3 Mar 2022 21:27:45 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v9] In-Reply-To: References: Message-ID: > Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. > > Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. > > Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. DamonGuy has updated the pull request incrementally with one additional commit since the last revision: Organized imports. Renamed functions for clarity. Added newline at end. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7310/files - new: https://git.openjdk.java.net/jdk/pull/7310/files/dd8599ce..1a885b7b Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7310&range=08 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7310&range=07-08 Stats: 16 lines in 1 file changed: 5 ins; 5 del; 6 mod Patch: https://git.openjdk.java.net/jdk/pull/7310.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7310/head:pull/7310 PR: https://git.openjdk.java.net/jdk/pull/7310 From prr at openjdk.java.net Thu Mar 3 21:44:00 2022 From: prr at openjdk.java.net (Phil Race) Date: Thu, 3 Mar 2022 21:44:00 GMT Subject: RFR: 8282577: ICC_Profile.setData(int, byte[]) invalidates the profile [v3] In-Reply-To: References: Message-ID: On Thu, 3 Mar 2022 21:27:36 GMT, Phil Race wrote: >> As per the bug eval, we are returning a pointer to a native raw LittleCMS profile and it should be a cooked one. >> The simple test fails before the fix with >> ~/jdk17/Contents/Home/bin/java SetTagDataValidation >> Exception in thread "main" java.awt.color.CMMException: LCMS error 13: Couldn't link the profiles >> at java.desktop/sun.java2d.cmm.lcms.LCMS.createNativeTransform(Native Method) >> at java.desktop/sun.java2d.cmm.lcms.LCMS.createTransform(LCMS.java:104) >> at java.desktop/sun.java2d.cmm.lcms.LCMSTransform.doTransform(LCMSTransform.java:143) >> at java.desktop/sun.java2d.cmm.lcms.LCMSTransform.colorConvert(LCMSTransform.java:633) >> at java.desktop/java.awt.color.ICC_ColorSpace.toRGB(ICC_ColorSpace.java:224) >> at SetTagDataValidation.main(SetTagDataValidation.java:61) >> >> and with the fix just prints >> java SetTagDataValidation >> 0.4 >> 0.5000076 >> 0.6 > > Phil Race has updated the pull request incrementally with one additional commit since the last revision: > > 8282577: ICC_Profile.setData(int, byte[]) invalidates the profile I have added the problem listing of a test sun/java2d/cmm/ReadWriteProfileTest.java https://bugs.openjdk.java.net/browse/JDK-8282635 The test would not have been passing if the profile were actually validated and it looks like there's various issues to be solved - if possible. ------------- PR: https://git.openjdk.java.net/jdk/pull/7668 From aivanov at openjdk.java.net Thu Mar 3 21:57:08 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Thu, 3 Mar 2022 21:57:08 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v9] In-Reply-To: References: Message-ID: On Thu, 3 Mar 2022 21:27:45 GMT, DamonGuy wrote: >> Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. >> >> Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. >> >> Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. > > DamonGuy has updated the pull request incrementally with one additional commit since the last revision: > > Organized imports. Renamed functions for clarity. Added newline at end. Looks good other than reversed method names to create a button and then paint the button into an image. ------------- Marked as reviewed by aivanov (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/7310 From aivanov at openjdk.java.net Thu Mar 3 21:57:09 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Thu, 3 Mar 2022 21:57:09 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v8] In-Reply-To: References: Message-ID: <0gk2h41UhnL9rn15_XOGq7vtLxkjYN0Hy7IzqoYk7hc=.7d902cf9-600f-415a-94ed-5c90e5d76c2e@github.com> On Thu, 3 Mar 2022 20:42:16 GMT, Alexey Ivanov wrote: >> DamonGuy has updated the pull request incrementally with one additional commit since the last revision: >> >> Updated test to be headless using BufferedImage. > > test/jdk/javax/swing/JButton/HtmlButtonImageTest/HtmlButtonImageTest.java line 68: > >> 66: >> 67: SwingUtilities.invokeAndWait(HtmlButtonImageTest::createButton); >> 68: SwingUtilities.invokeAndWait(HtmlButtonImageTest::createImage); > > `paintButton` would be clearer. No, `createButton` then `paintButton` :) ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From duke at openjdk.java.net Thu Mar 3 22:51:48 2022 From: duke at openjdk.java.net (DamonGuy) Date: Thu, 3 Mar 2022 22:51:48 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v10] In-Reply-To: References: Message-ID: > Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. > > Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. > > Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. DamonGuy has updated the pull request incrementally with two additional commits since the last revision: - Changed method name. - Reorganized imports. Added space in conditional. ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7310/files - new: https://git.openjdk.java.net/jdk/pull/7310/files/1a885b7b..93f19ca2 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7310&range=09 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7310&range=08-09 Stats: 14 lines in 1 file changed: 6 ins; 5 del; 3 mod Patch: https://git.openjdk.java.net/jdk/pull/7310.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7310/head:pull/7310 PR: https://git.openjdk.java.net/jdk/pull/7310 From achung at openjdk.java.net Fri Mar 4 00:49:39 2022 From: achung at openjdk.java.net (Alisen Chung) Date: Fri, 4 Mar 2022 00:49:39 GMT Subject: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor [v2] In-Reply-To: References: Message-ID: > Changed the drawing area to be increased by 0.5 on the left side to prevent clipping Alisen Chung has updated the pull request incrementally with three additional commits since the last revision: - adjusted pixels to check for border - typo - reverted old change, swapped order of painting to prevent overdrawing ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7449/files - new: https://git.openjdk.java.net/jdk/pull/7449/files/c0b31892..6056faf8 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7449&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7449&range=00-01 Stats: 64 lines in 3 files changed: 48 ins; 1 del; 15 mod Patch: https://git.openjdk.java.net/jdk/pull/7449.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7449/head:pull/7449 PR: https://git.openjdk.java.net/jdk/pull/7449 From achung at openjdk.java.net Fri Mar 4 00:49:40 2022 From: achung at openjdk.java.net (Alisen Chung) Date: Fri, 4 Mar 2022 00:49:40 GMT Subject: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor In-Reply-To: References: Message-ID: On Fri, 11 Feb 2022 23:44:22 GMT, Alisen Chung wrote: > Changed the drawing area to be increased by 0.5 on the left side to prevent clipping I've updated the fix to stop overdrawing, please review @mrserb @prrace @prsadhuk ------------- PR: https://git.openjdk.java.net/jdk/pull/7449 From mvs at openjdk.java.net Fri Mar 4 01:55:45 2022 From: mvs at openjdk.java.net (Manukumar V S) Date: Fri, 4 Mar 2022 01:55:45 GMT Subject: RFR: 8282343: Create a regression test for JDK-4518432 [v2] In-Reply-To: References: Message-ID: > Create a regression test for [JDK-4518432](https://bugs.openjdk.java.net/browse/JDK-4518432). > > Issue: Copying from JTree and then changing the data in the component that was copied from, can cause pastes to use > the new edited data. > > This test verifies that copying from JTree node and then changing the data in the component that was copied from, > is not causing the pastes to use the new edited data instead of the original copied data. > > Testing: > Java 1.4.0 -> Test Failed. > $ ./j2sdk1.4.0/bin/java JTreeWin > Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel > Test Failed in javax.swing.plaf.metal.MetalLookAndFeel, Expected : "colorsblue", but actual : "ctextolorsblue" > Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel > Test Failed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel, Expected : "colorsblue", but actual : "ctextolorsblue" > > Java 1.4.1 -> Test Passed. > $ ./j2sdk1.4.1/bin/java JTreeWin > Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel > Test Passed in javax.swing.plaf.metal.MetalLookAndFeel > Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel > Test Passed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel Manukumar V S has updated the pull request incrementally with one additional commit since the last revision: Review comments fixed: Formatting changes, removed extra blank lines, removed redundant tree focus listening code ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7605/files - new: https://git.openjdk.java.net/jdk/pull/7605/files/00f02705..de0712a5 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7605&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7605&range=00-01 Stats: 20 lines in 1 file changed: 0 ins; 20 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/7605.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7605/head:pull/7605 PR: https://git.openjdk.java.net/jdk/pull/7605 From mvs at openjdk.java.net Fri Mar 4 02:05:48 2022 From: mvs at openjdk.java.net (Manukumar V S) Date: Fri, 4 Mar 2022 02:05:48 GMT Subject: RFR: 8282234: Create a regression test for JDK-4532513 [v3] In-Reply-To: References: Message-ID: <4yaJRW-hhl_zQtTvYmkYk2DFmJ5dtzF8f2P47iWxrQU=.3a367d66-de8d-4447-b1c1-daa896de7090@github.com> > Create a regression test for [JDK-4532513](https://bugs.openjdk.java.net/browse/JDK-4532513). > This test verifies that the DefaultCaret doesn't requests focus in mouseClick and mousePressed causing the associated input verifier to fire twice. > As per [JDK-4532513](https://bugs.openjdk.java.net/browse/JDK-4532513), the DefaultCaret requests focus in mouseClick and mousePressed causing the associated input verifier to fire twice. > This issue was raised in JDK 1.4.0 and fixed in 1.4.1, so the test need to be failed in 1.4.0 and must pass in 1.4.1. > > Testing: > > In JDK 1.4.0: > $ ./j2sdk1.4.0/bin/java DefaultCaretWin > Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel > shouldYieldFocus called 1 > shouldYieldFocus called 2 > shouldYieldFocus called 3 > java.lang.RuntimeException: Test failed for javax.swing.plaf.metal.MetalLookAndFeel as InputVerifier.shouldYieldFocus() getting called 3 times on jTextField2, but it is expected to call only once. > at DefaultCaretWin.runTest(DefaultCaretWin.java:114) > at DefaultCaretWin.main(DefaultCaretWin.java:58) > > In JDK 1.4.1: > $ ./j2sdk1.4.1/bin/java DefaultCaretWin > Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel > shouldYieldFocus called 1 > Test passed for laf: javax.swing.plaf.metal.MetalLookAndFeel > Testing L&F: com.sun.java.swing.plaf.motif.MotifLookAndFeel > shouldYieldFocus called 1 > Test passed for laf: com.sun.java.swing.plaf.motif.MotifLookAndFeel > Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel > shouldYieldFocus called 1 > Test passed for laf: com.sun.java.swing.plaf.windows.WindowsLookAndFeel Manukumar V S has updated the pull request incrementally with one additional commit since the last revision: Formatting changes, removed extra blank lines, rearranged methods order ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7577/files - new: https://git.openjdk.java.net/jdk/pull/7577/files/6d8a8c89..96942f37 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7577&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7577&range=01-02 Stats: 11 lines in 1 file changed: 1 ins; 6 del; 4 mod Patch: https://git.openjdk.java.net/jdk/pull/7577.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7577/head:pull/7577 PR: https://git.openjdk.java.net/jdk/pull/7577 From mvs at openjdk.java.net Fri Mar 4 02:05:48 2022 From: mvs at openjdk.java.net (Manukumar V S) Date: Fri, 4 Mar 2022 02:05:48 GMT Subject: RFR: 8282234: Create a regression test for JDK-4532513 [v2] In-Reply-To: References: Message-ID: On Wed, 23 Feb 2022 07:13:25 GMT, Manukumar V S wrote: >> Create a regression test for [JDK-4532513](https://bugs.openjdk.java.net/browse/JDK-4532513). >> This test verifies that the DefaultCaret doesn't requests focus in mouseClick and mousePressed causing the associated input verifier to fire twice. >> As per [JDK-4532513](https://bugs.openjdk.java.net/browse/JDK-4532513), the DefaultCaret requests focus in mouseClick and mousePressed causing the associated input verifier to fire twice. >> This issue was raised in JDK 1.4.0 and fixed in 1.4.1, so the test need to be failed in 1.4.0 and must pass in 1.4.1. >> >> Testing: >> >> In JDK 1.4.0: >> $ ./j2sdk1.4.0/bin/java DefaultCaretWin >> Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel >> shouldYieldFocus called 1 >> shouldYieldFocus called 2 >> shouldYieldFocus called 3 >> java.lang.RuntimeException: Test failed for javax.swing.plaf.metal.MetalLookAndFeel as InputVerifier.shouldYieldFocus() getting called 3 times on jTextField2, but it is expected to call only once. >> at DefaultCaretWin.runTest(DefaultCaretWin.java:114) >> at DefaultCaretWin.main(DefaultCaretWin.java:58) >> >> In JDK 1.4.1: >> $ ./j2sdk1.4.1/bin/java DefaultCaretWin >> Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel >> shouldYieldFocus called 1 >> Test passed for laf: javax.swing.plaf.metal.MetalLookAndFeel >> Testing L&F: com.sun.java.swing.plaf.motif.MotifLookAndFeel >> shouldYieldFocus called 1 >> Test passed for laf: com.sun.java.swing.plaf.motif.MotifLookAndFeel >> Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel >> shouldYieldFocus called 1 >> Test passed for laf: com.sun.java.swing.plaf.windows.WindowsLookAndFeel > > Manukumar V S has updated the pull request incrementally with one additional commit since the last revision: > > Set the default number of columns in JTextField constructor, otherwise in linux platforms the column width is coming default as 0 and in turns fails the test @aivanov-jdk Can you please review this. ------------- PR: https://git.openjdk.java.net/jdk/pull/7577 From psadhukhan at openjdk.java.net Fri Mar 4 05:07:44 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 4 Mar 2022 05:07:44 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v3] In-Reply-To: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: > Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: Revert dialogInput. Add Bolditalic ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/7643/files - new: https://git.openjdk.java.net/jdk/pull/7643/files/9c098f99..0f74bf39 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7643&range=02 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7643&range=01-02 Stats: 6 lines in 1 file changed: 0 ins; 0 del; 6 mod Patch: https://git.openjdk.java.net/jdk/pull/7643.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7643/head:pull/7643 PR: https://git.openjdk.java.net/jdk/pull/7643 From smandalika at openjdk.java.net Fri Mar 4 06:03:28 2022 From: smandalika at openjdk.java.net (Srinivas Mandalika) Date: Fri, 4 Mar 2022 06:03:28 GMT Subject: RFR: 8282640: Create a test for JDK-4740761 Message-ID: Create a test for [JDK-4740761](https://bugs.openjdk.java.net/browse/JDK-4740761) The issue observed is in a JFrame with a JTextField and a JScrollPane which contains focused component. When the JScrollPane was is removed from its parent, focus stays with the the scroller of the removed JScrollPane, instead of moving to the TextField. The test verifies the same - i.e. it verifies that the removal of the scrollpane moves the focus to the JTextField present in the JFrame. ------------- Commit messages: - 8282640: Create a test for JDK-4740761 - 8282640: Create a test for JDK-4740761 - 8282640: Create a test for JDK-4740761 Changes: https://git.openjdk.java.net/jdk/pull/7686/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7686&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8282640 Stats: 118 lines in 1 file changed: 118 ins; 0 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/7686.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7686/head:pull/7686 PR: https://git.openjdk.java.net/jdk/pull/7686 From asemenov at openjdk.java.net Fri Mar 4 09:13:00 2022 From: asemenov at openjdk.java.net (Artem Semenov) Date: Fri, 4 Mar 2022 09:13:00 GMT Subject: RFR: 8281284 : Write JSlider accessibility test [v5] In-Reply-To: References: Message-ID: On Thu, 3 Mar 2022 17:11:39 GMT, lawrence.andrews wrote: >> This testcase covers both old and new api ( api added in jdk17 getAccessibleAction(), getAccessibleActionCount() , getAccessibleActionDescription ) >> >> @shurymury >> @savoptik > > lawrence.andrews has updated the pull request incrementally with one additional commit since the last revision: > > run the on windows and mac Looks good to me. I agree ------------- PR: https://git.openjdk.java.net/jdk/pull/7355 From dbatrak at openjdk.java.net Fri Mar 4 09:24:59 2022 From: dbatrak at openjdk.java.net (Dmitry Batrak) Date: Fri, 4 Mar 2022 09:24:59 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v3] In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: <3ghRGxNVhvU1lK6xujw3BlBepie3GvPONnIF3cAwIlo=.51b5b48d-62fa-4573-b99d-1fba84482087@github.com> On Fri, 4 Mar 2022 05:07:44 GMT, Prasanta Sadhukhan wrote: >> Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Revert dialogInput. Add Bolditalic If I may comment, in the past we received a noticeable amount of complaints about Korean text not rendered in our IDEs on Windows. This was solved by installing Korean font pack, but relying on users always doing that by themselves doesn't seem to provide the best user experience. So, two years ago, in JetBrains Runtime we ended up replacing Gulim/GulimChe/Batang with Malgun Gothic. This works well for us so far. I understand that this approach can give worse results for users that do install Korean font pack. But, I believe, the 'chaining' proposed by Naoto can actually be implemented, if desired - we just need to introduce a new 'category' to the properties file, say `korean_supplementary`. By putting it before `korean` in sequence definitions, we'll make e.g. Batang used if it's installed, and fall back to Malgun Gothic otherwise. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From aivanov at openjdk.java.net Fri Mar 4 09:39:09 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Fri, 4 Mar 2022 09:39:09 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v10] In-Reply-To: References: Message-ID: On Thu, 3 Mar 2022 22:51:48 GMT, DamonGuy wrote: >> Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. >> >> Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. >> >> Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. > > DamonGuy has updated the pull request incrementally with two additional commits since the last revision: > > - Changed method name. > - Reorganized imports. Added space in conditional. Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From psadhukhan at openjdk.java.net Fri Mar 4 09:56:04 2022 From: psadhukhan at openjdk.java.net (Prasanta Sadhukhan) Date: Fri, 4 Mar 2022 09:56:04 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v10] In-Reply-To: References: Message-ID: On Thu, 3 Mar 2022 22:51:48 GMT, DamonGuy wrote: >> Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. >> >> Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. >> >> Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. > > DamonGuy has updated the pull request incrementally with two additional commits since the last revision: > > - Changed method name. > - Reorganized imports. Added space in conditional. Marked as reviewed by psadhukhan (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From aivanov at openjdk.java.net Fri Mar 4 10:53:02 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Fri, 4 Mar 2022 10:53:02 GMT Subject: RFR: 8282343: Create a regression test for JDK-4518432 [v2] In-Reply-To: References: Message-ID: On Fri, 4 Mar 2022 01:55:45 GMT, Manukumar V S wrote: >> Create a regression test for [JDK-4518432](https://bugs.openjdk.java.net/browse/JDK-4518432). >> >> Issue: Copying from JTree and then changing the data in the component that was copied from, can cause pastes to use >> the new edited data. >> >> This test verifies that copying from JTree node and then changing the data in the component that was copied from, >> is not causing the pastes to use the new edited data instead of the original copied data. >> >> Testing: >> Java 1.4.0 -> Test Failed. >> $ ./j2sdk1.4.0/bin/java JTreeWin >> Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel >> Test Failed in javax.swing.plaf.metal.MetalLookAndFeel, Expected : "colorsblue", but actual : "ctextolorsblue" >> Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel >> Test Failed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel, Expected : "colorsblue", but actual : "ctextolorsblue" >> >> Java 1.4.1 -> Test Passed. >> $ ./j2sdk1.4.1/bin/java JTreeWin >> Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel >> Test Passed in javax.swing.plaf.metal.MetalLookAndFeel >> Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel >> Test Passed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel > > Manukumar V S has updated the pull request incrementally with one additional commit since the last revision: > > Review comments fixed: Formatting changes, removed extra blank lines, removed redundant tree focus listening code Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7605 From mvs at openjdk.java.net Fri Mar 4 10:59:05 2022 From: mvs at openjdk.java.net (Manukumar V S) Date: Fri, 4 Mar 2022 10:59:05 GMT Subject: Integrated: 8282343: Create a regression test for JDK-4518432 In-Reply-To: References: Message-ID: On Thu, 24 Feb 2022 11:46:58 GMT, Manukumar V S wrote: > Create a regression test for [JDK-4518432](https://bugs.openjdk.java.net/browse/JDK-4518432). > > Issue: Copying from JTree and then changing the data in the component that was copied from, can cause pastes to use > the new edited data. > > This test verifies that copying from JTree node and then changing the data in the component that was copied from, > is not causing the pastes to use the new edited data instead of the original copied data. > > Testing: > Java 1.4.0 -> Test Failed. > $ ./j2sdk1.4.0/bin/java JTreeWin > Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel > Test Failed in javax.swing.plaf.metal.MetalLookAndFeel, Expected : "colorsblue", but actual : "ctextolorsblue" > Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel > Test Failed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel, Expected : "colorsblue", but actual : "ctextolorsblue" > > Java 1.4.1 -> Test Passed. > $ ./j2sdk1.4.1/bin/java JTreeWin > Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel > Test Passed in javax.swing.plaf.metal.MetalLookAndFeel > Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel > Test Passed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel This pull request has now been integrated. Changeset: b3837808 Author: Manukumar V S Committer: Alexey Ivanov URL: https://git.openjdk.java.net/jdk/commit/b3837808bf13de2aa3d8c6c21cf61baac9baf2b2 Stats: 217 lines in 1 file changed: 217 ins; 0 del; 0 mod 8282343: Create a regression test for JDK-4518432 Reviewed-by: aivanov ------------- PR: https://git.openjdk.java.net/jdk/pull/7605 From zgu at openjdk.java.net Fri Mar 4 13:32:31 2022 From: zgu at openjdk.java.net (Zhengyu Gu) Date: Fri, 4 Mar 2022 13:32:31 GMT Subject: RFR: 8282628: Potential memory leak in sun.font.FontConfigManager.getFontConfig() Message-ID: Please review this small patch that fixes a potential memory leak that exception return fails to release allocated `cacheDirs` Test: - [x] jdk_desktop on Linux x86_64 ------------- Commit messages: - 8282628: Potential memory leak in sun.font.FontConfigManager.getFontConfig() Changes: https://git.openjdk.java.net/jdk/pull/7691/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7691&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8282628 Stats: 21 lines in 2 files changed: 18 ins; 0 del; 3 mod Patch: https://git.openjdk.java.net/jdk/pull/7691.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7691/head:pull/7691 PR: https://git.openjdk.java.net/jdk/pull/7691 From stuefe at openjdk.java.net Fri Mar 4 16:51:05 2022 From: stuefe at openjdk.java.net (Thomas Stuefe) Date: Fri, 4 Mar 2022 16:51:05 GMT Subject: RFR: 8282628: Potential memory leak in sun.font.FontConfigManager.getFontConfig() In-Reply-To: References: Message-ID: On Fri, 4 Mar 2022 13:25:12 GMT, Zhengyu Gu wrote: > Please review this small patch that fixes a potential memory leak that exception return fails to release allocated `cacheDirs` > > Test: > > - [x] jdk_desktop on Linux x86_64 Looks good. ------------- Marked as reviewed by stuefe (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/7691 From duke at openjdk.java.net Fri Mar 4 16:56:25 2022 From: duke at openjdk.java.net (Matteo Baccan) Date: Fri, 4 Mar 2022 16:56:25 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo Hi I have pushed this PR about 1 month ago. Only 3 days ago OCA was accepted. Now: what is the next step? This is a cleanup PR that removes some double semicolons at the end of some lines inside the JDK code. ciao matteo ------------- PR: https://git.openjdk.java.net/jdk/pull/7268 From duke at openjdk.java.net Fri Mar 4 16:56:25 2022 From: duke at openjdk.java.net (Matteo Baccan) Date: Fri, 4 Mar 2022 16:56:25 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines Message-ID: Hi I have reviewed the code for removing double semicolons at the end of lines all the best matteo ------------- Commit messages: - Removed double semicolon at the end of lines Changes: https://git.openjdk.java.net/jdk/pull/7268/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=7268&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8282657 Stats: 93 lines in 82 files changed: 0 ins; 0 del; 93 mod Patch: https://git.openjdk.java.net/jdk/pull/7268.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/7268/head:pull/7268 PR: https://git.openjdk.java.net/jdk/pull/7268 From jwaters at openjdk.java.net Fri Mar 4 16:56:26 2022 From: jwaters at openjdk.java.net (Julian Waters) Date: Fri, 4 Mar 2022 16:56:26 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 25 Feb 2022 15:40:09 GMT, Matteo Baccan wrote: >> Hi >> >> I have reviewed the code for removing double semicolons at the end of lines >> >> all the best >> matteo > > Hi > > I have pushed this PR about 1 month ago. Only 3 days ago OCA was accepted. > Now: what is the next step? > > This is a cleanup PR that removes some double semicolons at the end of some lines inside the JDK code. > > ciao > matteo Hi @matteobaccan The next step would be to create a relevant issue on the tracker and set it to track this PR. Since you don't have the ability to create new issues on the JBS yet I'll help you create one, please rename your PR title to 8282657 and the system should take care of the rest and automatically mark your PR as ready for review after setting it to track the corresponding JBS entry. Cheers, Julian ------------- PR: https://git.openjdk.java.net/jdk/pull/7268 From lancea at openjdk.java.net Fri Mar 4 17:07:59 2022 From: lancea at openjdk.java.net (Lance Andersen) Date: Fri, 4 Mar 2022 17:07:59 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo The changes look OK. The copyright year probably should be updated as part of this PR ------------- Marked as reviewed by lancea (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/7268 From duke at openjdk.java.net Fri Mar 4 17:14:05 2022 From: duke at openjdk.java.net (Matteo Baccan) Date: Fri, 4 Mar 2022 17:14:05 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo Hi Lance I can make a second commit updating the copyright year Tell me if this is necessary ciao matteo ------------- PR: https://git.openjdk.java.net/jdk/pull/7268 From rriggs at openjdk.java.net Fri Mar 4 17:20:05 2022 From: rriggs at openjdk.java.net (Roger Riggs) Date: Fri, 4 Mar 2022 17:20:05 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo We usually request that these be be broken up by area to attract the appropriate reviewers and avoid eye-strain. The client modules are usually separated out, as are hotspot. And corresponding tests. This kind of change is pretty low value for the code base and requires the involvement of quite a few reviewers. If you had ask ahead of time, I would have suggested finding something with more value. ------------- Changes requested by rriggs (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/7268 From naoto at openjdk.java.net Fri Mar 4 17:29:01 2022 From: naoto at openjdk.java.net (Naoto Sato) Date: Fri, 4 Mar 2022 17:29:01 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v2] In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> <3F6rwtKnmNxHmNk_JwNo-ovGQCtROCOTqq9JYEkxCnU=.715f8bd7-4df9-47dc-85f3-a598d01e8e78@github.com> Message-ID: <38BaTaUcyQ6-Xdt3q-mJlHgJQ4JHR2u8Pfp52jiq5E4=.a79d91c1-6f58-4088-b0d1-4a1d43921b51@github.com> On Thu, 3 Mar 2022 18:07:25 GMT, Phil Race wrote: >> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: >> >> Add bold version > > make/data/fontconfig/windows.fontconfig.properties line 184: > >> 182: dialoginput.bold.japanese=MS Gothic >> 183: dialoginput.bold.korean=Malgun Gothic Bold >> 184: > > Since dialoginput is usually the same as monospaced, per Naoto's comment, this should be reverted. Not only reverting, but I think `GulimChe` here is the right choice, as it is `monospaced`? ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From naoto at openjdk.java.net Fri Mar 4 17:36:06 2022 From: naoto at openjdk.java.net (Naoto Sato) Date: Fri, 4 Mar 2022 17:36:06 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v3] In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Fri, 4 Mar 2022 05:07:44 GMT, Prasanta Sadhukhan wrote: >> Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Revert dialogInput. Add Bolditalic Initially I thought it was surprising this problem has not been reported quite some time since `Malgun Gothic` was introduced. Then I guessed that Korean customers possibly would all install Korean language pack because without it, they would not be able to input Korean chars (no Korean input method included by default). I think in IntelliJ's case, possibly the typical users are all developers installing English Windows, thus there have been bug reports. So, I would rather think replacing those fonts with `Malgun Gothic` could be problematic to the former Korean customers. Having said that, I would want the `chaining` functionality which could effectively satisfy both users. Other than the option you provided, there could be an extension of the fontconfig, such as, serif.plain.korean=Batang, Malgun Gothic so that if `Batang` file is not ther, we can fall back to `Malgun Gothic`. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From ihse at openjdk.java.net Fri Mar 4 17:50:03 2022 From: ihse at openjdk.java.net (Magnus Ihse Bursie) Date: Fri, 4 Mar 2022 17:50:03 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 4 Mar 2022 17:17:12 GMT, Roger Riggs wrote: >> Hi >> >> I have reviewed the code for removing double semicolons at the end of lines >> >> all the best >> matteo > > We usually request that these be be broken up by area to attract the appropriate reviewers and avoid eye-strain. The client modules are usually separated out, as are hotspot. > And corresponding tests. > This kind of change is pretty low value for the code base and requires the involvement of quite a few reviewers. > If you had ask ahead of time, I would have suggested finding something with more value. @RogerRiggs Otoh, this change is really trivial. I have verified that all changes are replacing trailing ";;" in Java code. These are all clearly typos. I think it's nice that we try to strive for a high quality, and while you are correct this is maybe not the most pressing issue, I think it's nice to get a cleanup like this done. I'd argue that this is a trivial change. If you are worried, let's get a couple of more reviewers. I can't see the need to split this up per area. ------------- PR: https://git.openjdk.java.net/jdk/pull/7268 From prr at openjdk.java.net Fri Mar 4 18:20:05 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 18:20:05 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v3] In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Fri, 4 Mar 2022 05:07:44 GMT, Prasanta Sadhukhan wrote: >> Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Revert dialogInput. Add Bolditalic But serif.plain.korean isn't used unless it is a Korean locale .. so what's the point ? And this one line already added allfonts.korean=Malgun Gothic will provide it as a fall back so I don't see the need. If I am an English language user running in a US / UK locale I'll get my Korean glyphs from there with the only difference being it isn't in the same place in the precedence order. This also ensures that any "new" code points added in that font not present in the older ones will always be available. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From naoto at openjdk.java.net Fri Mar 4 18:27:06 2022 From: naoto at openjdk.java.net (Naoto Sato) Date: Fri, 4 Mar 2022 18:27:06 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v3] In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Fri, 4 Mar 2022 18:16:22 GMT, Phil Race wrote: > allfonts.korean=Malgun Gothic will provide it as a fall back so I don't see the need. OK, that's what I needed. Thanks. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From prr at openjdk.java.net Fri Mar 4 18:57:02 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 18:57:02 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v3] In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Fri, 4 Mar 2022 05:07:44 GMT, Prasanta Sadhukhan wrote: >> Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Revert dialogInput. Add Bolditalic BTW does anyone know if this is a problem for Chinese or Japanese too ? I think Indic might have needs a font pack too .. maybe not in this bug but should we be looking wider than Korean ? ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From prr at openjdk.java.net Fri Mar 4 19:10:06 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 19:10:06 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo Marked as reviewed by prr (Reviewer). Looks like there's only one client source code file touched Most of the client changes are only in jtreg tests - and this is very trivial, so I'm OK with them being included here. ------------- PR: https://git.openjdk.java.net/jdk/pull/7268 From prr at openjdk.java.net Fri Mar 4 19:14:01 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 19:14:01 GMT Subject: RFR: 8282473: Refactor swing classes javadoc to use @throws instead of @exception In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 10:16:16 GMT, Prasanta Sadhukhan wrote: > Prevailing JDK coding practices use "@throws" rather than "@exception". > Refactor existing swing classes javadoc to use @throws This is touching a lot of files but I can't see it interfering with clean backports as we are very unlikely to touch lines in javadoc of SE API in backports ! And of course by implication, this fix is not one that would/should be backported. ------------- Marked as reviewed by prr (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/7644 From naoto at openjdk.java.net Fri Mar 4 19:20:06 2022 From: naoto at openjdk.java.net (Naoto Sato) Date: Fri, 4 Mar 2022 19:20:06 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v3] In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Fri, 4 Mar 2022 05:07:44 GMT, Prasanta Sadhukhan wrote: >> Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Revert dialogInput. Add Bolditalic Not that I know of. Simple testing with Japanese text worked OK, but extensive testing for CJK would be needed. ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From aivanov at openjdk.java.net Fri Mar 4 19:27:04 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Fri, 4 Mar 2022 19:27:04 GMT Subject: RFR: 8282602: Refactor awt classes javadoc to use @throws instead of @exception In-Reply-To: References: Message-ID: On Thu, 3 Mar 2022 12:26:19 GMT, Prasanta Sadhukhan wrote: > Prevailing JDK coding practices use "@throws" rather than "@exception". > Refactor existing java/awt classes javadoc to use @throws Marked as reviewed by aivanov (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7675 From duke at openjdk.java.net Fri Mar 4 19:30:13 2022 From: duke at openjdk.java.net (DamonGuy) Date: Fri, 4 Mar 2022 19:30:13 GMT Subject: Integrated: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding In-Reply-To: References: Message-ID: On Tue, 1 Feb 2022 16:11:52 GMT, DamonGuy wrote: > Html does not fit in JButton at certain sizes because default Insets cause html to be displayed off-center. > > Changes made to SwingUtilities.java layoutCompoundLabelImpl method to enable clipping if html does not fit, similar to regular text. AquaButtonUI.java now detects when html does not fit, and an implementation for alternate insets that are recursively tested for regular text inside layoutAndGetText() are now also being used for html content. > > Created test (Bug8015854.java) with the same format as the test described on the issue. The button is of size 37x37 with an image of a red square sized 19x19. The test checks for red pixels on the edges of where the square image should be from the center of the button. The test fails with the non-changed jdk, but passes with the changes. This pull request has now been integrated. Changeset: 9c817d38 Author: Damon Nguyen Committer: Alexey Ivanov URL: https://git.openjdk.java.net/jdk/commit/9c817d38806708f84139e3180ac86a3805dcce5f Stats: 125 lines in 2 files changed: 123 ins; 1 del; 1 mod 8015854: [macosx] JButton's HTML ImageView adding unwanted padding Reviewed-by: psadhukhan, aivanov ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From rriggs at openjdk.java.net Fri Mar 4 19:33:06 2022 From: rriggs at openjdk.java.net (Roger Riggs) Date: Fri, 4 Mar 2022 19:33:06 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo Marked as reviewed by rriggs (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7268 From prr at openjdk.java.net Fri Mar 4 19:42:08 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 19:42:08 GMT Subject: RFR: 8282628: Potential memory leak in sun.font.FontConfigManager.getFontConfig() In-Reply-To: References: Message-ID: On Fri, 4 Mar 2022 13:25:12 GMT, Zhengyu Gu wrote: > Please review this small patch that fixes a potential memory leak that exception return fails to release allocated `cacheDirs` > > Test: > > - [x] jdk_desktop on Linux x86_64 I've increased the number of reviewers because as well as the current reviewer it should have (1) a reviewer from the client team (2) a reviewer from whoever owns jni_util.h If there were no change to that file it would have been simpler. ------------- PR: https://git.openjdk.java.net/jdk/pull/7691 From iris at openjdk.java.net Fri Mar 4 19:43:03 2022 From: iris at openjdk.java.net (Iris Clark) Date: Fri, 4 Mar 2022 19:43:03 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo Nice tidy of the code. Is there anything that can be done to prevent re-introduction of this trivial problem? Perhaps a new Skara bot jcheck option similar to what is already in place for trailing whitespace? ------------- Marked as reviewed by iris (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/7268 From wetmore at openjdk.java.net Fri Mar 4 19:56:03 2022 From: wetmore at openjdk.java.net (Bradford Wetmore) Date: Fri, 4 Mar 2022 19:56:03 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: <1X39o4ON1uvbSXAp_r66zAmSy6sWZFKaP7-M54vAqX0=.d6abe0d5-9dd2-409b-91df-255d838196cb@github.com> On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo LGTM also. Similar suggestion for updating copyrights. ------------- Marked as reviewed by wetmore (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/7268 From aivanov at openjdk.java.net Fri Mar 4 20:02:11 2022 From: aivanov at openjdk.java.net (Alexey Ivanov) Date: Fri, 4 Mar 2022 20:02:11 GMT Subject: RFR: 8015854: [macosx] JButton's HTML ImageView adding unwanted padding [v8] In-Reply-To: References: Message-ID: On Tue, 1 Mar 2022 23:33:40 GMT, DamonGuy wrote: > There is no new bug submitted yet for other LAFs that I know of. I tested other LAFs and the issue does persist, so I can submit the bug for them if necessary. The other LAFs I tested were Metal, Nimbus, and Motif. Please, submit a new bug so that we don't forget about this issue in other LAFs. ------------- PR: https://git.openjdk.java.net/jdk/pull/7310 From prr at openjdk.java.net Fri Mar 4 20:19:00 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 20:19:00 GMT Subject: RFR: 8282602: Refactor awt classes javadoc to use @throws instead of @exception In-Reply-To: References: Message-ID: <2YKBIloBXEg_Jwc1l_fMElQLJOsyLbBEn_4OMuLkJi8=.8ba591c3-1c95-4e93-aa97-3a38911fd9c8@github.com> On Thu, 3 Mar 2022 12:26:19 GMT, Prasanta Sadhukhan wrote: > Prevailing JDK coding practices use "@throws" rather than "@exception". > Refactor existing java/awt classes javadoc to use @throws Marked as reviewed by prr (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7675 From prr at openjdk.java.net Fri Mar 4 20:20:09 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 20:20:09 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v3] In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: On Fri, 4 Mar 2022 05:07:44 GMT, Prasanta Sadhukhan wrote: >> Malgun_Gothic(https://en.wikipedia.org/wiki/Malgun_Gothic) had replaced Gulim korean font so it needs to be updated to show korean fonts > > Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision: > > Revert dialogInput. Add Bolditalic Marked as reviewed by prr (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7643 From prr at openjdk.java.net Fri Mar 4 20:44:06 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 20:44:06 GMT Subject: RFR: 8282422: JTable.print() failed with UnsupportedCharsetException on AIX ko_KR locale In-Reply-To: References: Message-ID: On Sat, 26 Feb 2022 16:24:07 GMT, Ichiroh Takiguchi wrote: > On AIX ko_KR locale, I could see following exception after JTable.print() was called > > java.nio.charset.UnsupportedCharsetException: sun.font.X11KSC5601 > at java.base/java.nio.charset.Charset.forName(Charset.java:527) > at java.desktop/sun.awt.FontConfiguration.getFontCharsetEncoder(FontConfiguration.java:964) > ...... > > According to my investigation, sun.font.X11KSC5601 moved from sun.awt.motif.X11KSC5601. Should be safe enough since I think AIX may be the only platform left that needs this. ------------- Marked as reviewed by prr (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/7630 From prr at openjdk.java.net Fri Mar 4 21:13:07 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 21:13:07 GMT Subject: RFR: 8282270: java/awt/Robot Screen Capture tests fail after 8280861 [v2] In-Reply-To: <6sowygRcdclz1LCYIHZSWMKV2LhuPo0D_EaYC_JIq0c=.b25c1f95-2510-4372-a8e4-99178d1762ce@github.com> References: <8HCDsHGSuDheojW6JRvidlZNmYGSftTKQ8f9mP_rI1A=.3407b062-6213-4960-b44c-1efc125ec7df@github.com> <6sowygRcdclz1LCYIHZSWMKV2LhuPo0D_EaYC_JIq0c=.b25c1f95-2510-4372-a8e4-99178d1762ce@github.com> Message-ID: <9TbYruwfo7JM4xbW6Mwj_WAAXdbrZR3O62Cc6xgt2TI=.74af7b62-adef-4b98-b195-4f9d47d5098d@github.com> On Mon, 28 Feb 2022 14:54:19 GMT, Maxim Kartashev wrote: >> The two tests `ScreenCaptureGtkTest.java` and `HiDPIRobotScreenCaptureTest.java` under `java/awt/Robot/HiDPIScreenCapture` started to intermittently fail under Windows and Linux after the [recent changes](https://github.com/openjdk/jdk/commit/cc7cf81256ed4d74493472017b1c4df20fa2208a) made exclusively for the Linux `Robot` implementation. >> >> `HiDPIRobotScreenCaptureTest.java` >> Since the Windows failures cannot possibly have their origin in the fix made - the latter being Linux-only - the test apparently either exposes an existing bug in the Windows `Robot` or is bumping against a peculiarity of that platform. The test is reverted back to its original form that didn't fail. >> >> `ScreenCaptureGtkTest.java` >> The coordinates in the log `(83, 78)` of a failure are higher up than the test suggests `(83, 97)`. I've seen similar failures on Ubuntu 20.04 when the coordinates were set to `(0, 0)`. The color then picked matched the color of the bar drawn at the top of the screen. I believe it's best to place the test pixels towards the center of the window to avoid desktop elements interference. >> >> The other possible reason for intermittent failures are window coordinates. Suppose that the test is executing with `sun.java2d.uiScale` set to 2. This effectively means that it can only pick colors of pixels at even coordinates (in the absolute desktop space). So if the window is placed at, say, `(201, 201)`, `Robot` can only pick the color at either `(200, 200)` or `(202, 202)`. Since the test only makes sense if it is pixel-accurate, I removed all `@run`s with `sun.java2d.uiScale` other than 1. This way window placement will not cause a failure. >> >> This was tested on Ubuntu 20.04 and 18.04 with desktop scaling set to 100%, 200%, and 300%. > > Maxim Kartashev has updated the pull request incrementally with one additional commit since the last revision: > > Fixed HiDPIRobotScreenCaptureTest to save image on failure test/jdk/java/awt/Robot/HiDPIScreenCapture/ScreenCaptureGtkTest.java line 44: > 42: * @run main/othervm -Djdk.gtk.version=2 -Dsun.java2d.uiScale=1 ScreenCaptureGtkTest > 43: * @run main/othervm -Djdk.gtk.version=3 -Dsun.java2d.uiScale=1 ScreenCaptureGtkTest > 44: */ So having removed all of these, it isn't testing hi-dpi any more, is it ? Even if running on a hi-dpi screen. Would it be better to let it just use whatever it wants for default ? Meaning if it happens to run on a hi-dpi screen we want to know if that works don't we ? ------------- PR: https://git.openjdk.java.net/jdk/pull/7613 From prr at openjdk.java.net Fri Mar 4 21:15:03 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 21:15:03 GMT Subject: RFR: 6218162: DefaultTableColumnModel.getColumn() method should mention ArrayIndexOutOfBoundsException [v3] In-Reply-To: References: Message-ID: <7U-SomFGRaFJPIZu_PxlYB92SMh4x0vMz5OxNLHFbLU=.2faa415f-e1f4-4bb5-a1bd-a84954c60e52@github.com> On Mon, 28 Feb 2022 07:24:27 GMT, Prasanta Sadhukhan wrote: >> If invalid ie 0 or > getColumnCount() index is passed to DefaultTableColumnModel.getColumn() then it returns AIOBE in current implementation which should be documented in the spec. Fixed the spec to mention the exception. > > Prasanta Sadhukhan has updated the pull request incrementally with three additional commits since the last revision: > > - javadoc fix > - javadoc fix > - javadoc fix Marked as reviewed by prr (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7587 From prr at openjdk.java.net Fri Mar 4 21:22:05 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 21:22:05 GMT Subject: RFR: 7017094: ParsedSynthStyle: parameter name "direction" should be changed to "tabIndex" In-Reply-To: References: Message-ID: On Mon, 21 Feb 2022 04:59:34 GMT, Prasanta Sadhukhan wrote: > [SynthPainter](https://github.com/openjdk/jdk/blob/master/src/java.desktop/share/classes/javax/swing/plaf/synth/SynthPainter.java#L1759) > and > [SynthPainterImpl](https://github.com/openjdk/jdk/blob/master/src/java.desktop/share/classes/javax/swing/plaf/nimbus/SynthPainterImpl.java#L2074) > expects a "tabIndex" although the parameter is not used currently. Marked as reviewed by prr (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7551 From darcy at openjdk.java.net Fri Mar 4 21:33:07 2022 From: darcy at openjdk.java.net (Joe Darcy) Date: Fri, 4 Mar 2022 21:33:07 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo Marked as reviewed by darcy (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7268 From dholmes at openjdk.java.net Fri Mar 4 21:34:07 2022 From: dholmes at openjdk.java.net (David Holmes) Date: Fri, 4 Mar 2022 21:34:07 GMT Subject: RFR: 8282628: Potential memory leak in sun.font.FontConfigManager.getFontConfig() In-Reply-To: References: Message-ID: On Fri, 4 Mar 2022 13:25:12 GMT, Zhengyu Gu wrote: > Please review this small patch that fixes a potential memory leak that exception return fails to release allocated `cacheDirs` > > Test: > > - [x] jdk_desktop on Linux x86_64 I would have just inlined JNU_CHECK_EXCEPTION and added the cleanup code directly. The additional macro wasn't really necessary and would not really be usable for any kind of general cleanup actions beyond a one-liner. But it is okay. Thanks. ------------- Marked as reviewed by dholmes (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/7691 From prr at openjdk.java.net Fri Mar 4 21:35:17 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 21:35:17 GMT Subject: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor [v2] In-Reply-To: References: Message-ID: On Fri, 4 Mar 2022 00:49:39 GMT, Alisen Chung wrote: >> Changed the drawing area to be increased by 0.5 on the left side to prevent clipping > > Alisen Chung has updated the pull request incrementally with three additional commits since the last revision: > > - adjusted pixels to check for border > - typo > - reverted old change, swapped order of painting to prevent overdrawing Can you explain (walk through) why the old code was wrong and the new code is correct ? There's a lot of things that aren't obvious to me. The old code would use the same coords and just toggle which parts were in highlight/shadow Now you've spilit is so that lowered does things in a different order and so forth and I find it impossible without an explanation of your reasoning to say if that reasoning is correct ... Also the bug report deserves an evaluation. ------------- PR: https://git.openjdk.java.net/jdk/pull/7449 From zgu at openjdk.java.net Fri Mar 4 21:57:58 2022 From: zgu at openjdk.java.net (Zhengyu Gu) Date: Fri, 4 Mar 2022 21:57:58 GMT Subject: RFR: 8282628: Potential memory leak in sun.font.FontConfigManager.getFontConfig() In-Reply-To: References: Message-ID: On Fri, 4 Mar 2022 21:30:53 GMT, David Holmes wrote: > I would have just inlined JNU_CHECK_EXCEPTION and added the cleanup code directly. The additional macro wasn't really necessary and would not really be usable for any kind of general cleanup actions beyond a one-liner. > > But it is okay. > > Thanks. Thanks for the review, @dholmes-ora . The macros are used to hide the different syntax of c and c++ to avoid polluting code here, it seems to be a common pattern in client code. ------------- PR: https://git.openjdk.java.net/jdk/pull/7691 From duke at openjdk.java.net Fri Mar 4 22:42:04 2022 From: duke at openjdk.java.net (Jeremy) Date: Fri, 4 Mar 2022 22:42:04 GMT Subject: RFR: 8176501: Method Shape.getBounds2D() incorrectly includes Bezier control points in bounding box [v12] In-Reply-To: References: Message-ID: On Thu, 16 Dec 2021 18:33:37 GMT, Jeremy wrote: >> This removes code that relied on consulting the Bezier control points to calculate the Rectangle2D bounding box. Instead it's pretty straight-forward to convert the Bezier control points into the x & y parametric equations. At their most complex these equations are cubic polynomials, so calculating their extrema is just a matter of applying the quadratic formula to calculate their extrema. (Or in path segments that are quadratic/linear/constant: we do even less work.) >> >> The bug writeup indicated they wanted Path2D#getBounds2D() to be more accurate/concise. They didn't explicitly say they wanted CubicCurve2D and QuadCurve2D to become more accurate too. But a preexisting unit test failed when Path2D#getBounds2D() was updated and those other classes weren't. At this point I considered either: >> A. Updating CubicCurve2D and QuadCurve2D to use the new more accurate getBounds2D() or >> B. Updating the unit test to forgive the discrepancy. >> >> I chose A. Which might technically be seen as scope creep, but it feels like a more holistic/better approach. >> >> Other shapes in java.awt.geom should not require updating, because they already identify concise bounds. >> >> This also includes a new unit test (in Path2D/UnitTest.java) that fails without the changes in this commit. > > Jeremy has updated the pull request incrementally with one additional commit since the last revision: > > 8176501: Method Shape.getBounds2D() incorrectly includes Bezier control points in bounding box > > This is a second follow-up response to prrace's code review feedback about method modifiers. > > This commit more carefully preserves the getBounds2D() method modifiers for all 3 classes: the Path2D.Double, the Path2D.Float, and the Path2D itself. > > It is possible (if unlikely) that someone previously extended the Path2D class and overrode getBounds2D(), because the Path2D#getBounds2D() method was *not* final. So with this commit: any such existing code will not break. Meanwhile the subclasses (Double and Float) preserve their previous modifiers (final, synchronized). > > This is in response to prrace's code review: > > > You are changing the signature of public API > > src/java.desktop/share/classes/java/awt/geom/Path2D.java > > public final synchronized Rectangle2D getBounds2D() => public Rectangle2D getBounds2D() { > > > > So no longer final, and no longer synchronized. > > This means a CSR is required and we need to think about it .. > > the intention was that the subclass not over-ride. > > And why remove synchronized ? I am fairly sure it was there to > > make sure no one was mutating the Path whilst > > bounds are being calculated. > > And you are using getPathIterator(AffineTransform) and the docs > > for that say it isn't thread safe. > > So I think this implementation needs to be thought about very carefully. I don't suppose any new client-lib members would be willing to set up a CSR for this PR? No worries if not. I just wanted to double-check before this PR gets auto-closed. ------------- PR: https://git.openjdk.java.net/jdk/pull/6227 From duke at openjdk.java.net Fri Mar 4 22:54:16 2022 From: duke at openjdk.java.net (Tejesh R) Date: Fri, 4 Mar 2022 22:54:16 GMT Subject: Integrated: 8236907: JTable added to nested panels does not paint last visible row In-Reply-To: References: Message-ID: On Thu, 10 Feb 2022 09:49:41 GMT, Tejesh R wrote: > Issue in painting last row of JTable due to decrement in `MaxRow `during computation. Differences in `MaxRow `computation between `TablePrintable` and `BasicTableUI `class. The Bug is fixed in both the classes. Test Case added to validate the same. This pull request has now been integrated. Changeset: bae0d5e7 Author: TejeshR13 Committer: Phil Race URL: https://git.openjdk.java.net/jdk/commit/bae0d5e7174fed8ffa8a30408c3cffa6e4dd3ddc Stats: 245 lines in 3 files changed: 224 ins; 17 del; 4 mod 8236907: JTable added to nested panels does not paint last visible row Reviewed-by: psadhukhan, prr ------------- PR: https://git.openjdk.java.net/jdk/pull/7422 From prr at openjdk.java.net Fri Mar 4 23:14:16 2022 From: prr at openjdk.java.net (Phil Race) Date: Fri, 4 Mar 2022 23:14:16 GMT Subject: RFR: 8275715: D3D pipeline processes multiple PaintEvent at initial drawing [v2] In-Reply-To: References: Message-ID: On Thu, 28 Oct 2021 08:27:44 GMT, Masanori Yano wrote: >> Could you please review the 8275715 bug fixes? >> >> I think D3DScreenUpdateManager posts unnecessary PaintEvent during processing PaintEvent. When the validate method is called from createGraphics, repaintPeerTarget should not be called. > > Masanori Yano has updated the pull request incrementally with one additional commit since the last revision: > > 8275715: D3D pipeline processes multiple PaintEvent at initial drawing Marked as reviewed by prr (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/6064 From achung at openjdk.java.net Sat Mar 5 01:59:04 2022 From: achung at openjdk.java.net (Alisen Chung) Date: Sat, 5 Mar 2022 01:59:04 GMT Subject: RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor [v2] In-Reply-To: References: Message-ID: On Fri, 4 Mar 2022 21:31:55 GMT, Phil Race wrote: > Can you explain (walk through) why the old code was wrong and the new code is correct ? There's a lot of things that aren't obvious to me. The old code would use the same coords and just toggle which parts were in highlight/shadow Now you've spilit is so that lowered does things in a different order and so forth and I find it impossible without an explanation of your reasoning to say if that reasoning is correct ... > > Also the bug report deserves an evaluation. I think currently the problem is the lighter color is overdrawing the darker color, so I swapped the drawing order so that the darker color is always drawn second. I think the reason the rect and lines are drawn at the same x value probably has to do with the affine transform in the graphics object, but I'm not sure why it happens. ------------- PR: https://git.openjdk.java.net/jdk/pull/7449 From duke at openjdk.java.net Sat Mar 5 04:03:06 2022 From: duke at openjdk.java.net (lawrence.andrews) Date: Sat, 5 Mar 2022 04:03:06 GMT Subject: RFR: 8281284 : Write JSlider accessibility test [v5] In-Reply-To: References: Message-ID: On Fri, 4 Mar 2022 09:10:02 GMT, Artem Semenov wrote: >> lawrence.andrews has updated the pull request incrementally with one additional commit since the last revision: >> >> run the on windows and mac > > Looks good to me. I agree @savoptik , can you approve this PR so that this PR is eligible for merge. ------------- PR: https://git.openjdk.java.net/jdk/pull/7355 From dholmes at openjdk.java.net Sat Mar 5 05:32:05 2022 From: dholmes at openjdk.java.net (David Holmes) Date: Sat, 5 Mar 2022 05:32:05 GMT Subject: RFR: 8282628: Potential memory leak in sun.font.FontConfigManager.getFontConfig() In-Reply-To: References: Message-ID: On Fri, 4 Mar 2022 21:54:57 GMT, Zhengyu Gu wrote: > The macros are used to hide the different syntax of c and c++ to avoid polluting code here, it seems to be a common pattern in client code. That's to allow for general use by C or C++ source files. But you are only needing this in a C source file so you would only need the C variant of the calling syntax. ------------- PR: https://git.openjdk.java.net/jdk/pull/7691 From dholmes at openjdk.java.net Sat Mar 5 05:52:04 2022 From: dholmes at openjdk.java.net (David Holmes) Date: Sat, 5 Mar 2022 05:52:04 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: <2NJw-71OOOvNs9519H__uYdXQnJm23L-Ez4jKoAuKrk=.c277d644-fd63-442e-99a1-6d3d66cb3405@github.com> On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo I eyeballed the diff file and all seems okay. Thanks, David ------------- Marked as reviewed by dholmes (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/7268 From jwaters at openjdk.java.net Sat Mar 5 06:52:13 2022 From: jwaters at openjdk.java.net (Julian Waters) Date: Sat, 5 Mar 2022 06:52:13 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo Nice, good work matteo Should I change the JBS issue title to match the PR title, or is it preferred for the PR title to change? ------------- PR: https://git.openjdk.java.net/jdk/pull/7268 From duke at openjdk.java.net Sat Mar 5 07:22:03 2022 From: duke at openjdk.java.net (Jan =?UTF-8?B?U2NobMO2w59pbg==?=) Date: Sat, 5 Mar 2022 07:22:03 GMT Subject: RFR: 8282657: Code cleanup: removing double semicolons at the end of lines In-Reply-To: References: Message-ID: On Fri, 28 Jan 2022 14:39:31 GMT, Matteo Baccan wrote: > Hi > > I have reviewed the code for removing double semicolons at the end of lines > > all the best > matteo This PR changes a comment in javax/swing/RepaintManager.java. This commented out code should be removed altogether in another PR? Because its an System.out.println and because its commented out code. ------------- PR: https://git.openjdk.java.net/jdk/pull/7268 From asemenov at openjdk.java.net Sat Mar 5 08:08:58 2022 From: asemenov at openjdk.java.net (Artem Semenov) Date: Sat, 5 Mar 2022 08:08:58 GMT Subject: RFR: 8281284 : Write JSlider accessibility test [v5] In-Reply-To: References: Message-ID: On Sat, 5 Mar 2022 03:59:21 GMT, lawrence.andrews wrote: >> Looks good to me. I agree > > @savoptik , can you approve this PR so that this PR is eligible for merge. @lawrence-andrew I am not reviewer. Contact @mrserb or @azuev-java ------------- PR: https://git.openjdk.java.net/jdk/pull/7355 From duke at openjdk.java.net Sat Mar 5 08:30:05 2022 From: duke at openjdk.java.net (Maxim Kartashev) Date: Sat, 5 Mar 2022 08:30:05 GMT Subject: RFR: 8282270: java/awt/Robot Screen Capture tests fail after 8280861 [v2] In-Reply-To: <9TbYruwfo7JM4xbW6Mwj_WAAXdbrZR3O62Cc6xgt2TI=.74af7b62-adef-4b98-b195-4f9d47d5098d@github.com> References: <8HCDsHGSuDheojW6JRvidlZNmYGSftTKQ8f9mP_rI1A=.3407b062-6213-4960-b44c-1efc125ec7df@github.com> <6sowygRcdclz1LCYIHZSWMKV2LhuPo0D_EaYC_JIq0c=.b25c1f95-2510-4372-a8e4-99178d1762ce@github.com> <9TbYruwfo7JM4xbW6Mwj_WAAXdbrZR3O62Cc6xgt2TI=.74af7b62-adef-4b98-b195-4f9d47d5098d@github.com> Message-ID: On Fri, 4 Mar 2022 21:10:03 GMT, Phil Race wrote: > So having removed all of these, it isn't testing hi-dpi any more, is it ? True. But there still is `ScreenCaptureTest.java` that will continue testing hi-dpi. Keeping `ScreenCaptureGtkTest.java` still makes sense because it verifies that `getPixelColor()` works precisely when neighboring pixels are of a different color; also, it shuffles the GTK versions. Unfortunately, I see no way to fully and reliably test the changes for 8280861 with scaling above 1. I am forced to choose between two evils: 1. either I set the scale above 1 and pick color inside a large area to be sure that I don't pick neighboring pixels because of an unfortunate window placement 1. or I set the scale to 1 and test colors of pixels sitting next to one another. (1) was tested by the existing tests, (2) is tested by `ScreenCaptureGtkTest.java`. ------------- PR: https://git.openjdk.java.net/jdk/pull/7613 From dbatrak at openjdk.java.net Sat Mar 5 10:38:00 2022 From: dbatrak at openjdk.java.net (Dmitry Batrak) Date: Sat, 5 Mar 2022 10:38:00 GMT Subject: RFR: 8190907: Windows 10 default Korean Font Malgun Gothic available not used by GUI elements even though available without Korean Language Pack [v3] In-Reply-To: References: <_cAab99k8SxU4y3G_FwRT3fg8t39fYQUDTdEwOZYZmc=.5d71beab-25a0-4c29-b194-d4680961b530@github.com> Message-ID: <7FJIWE5xd3Fh0W0iu6nmtcuZoWq2cuPDUidpy5l8jjs=.47bc0c69-86c1-43ef-b7da-29d9b6f34af5@github.com> On Fri, 4 Mar 2022 18:16:22 GMT, Phil Race wrote: > But serif.plain.korean isn't used unless it is a Korean locale That's not the case, and is easily verifiable for English locale - whatever font is specified there will become a component of Serif composite. 'korean' here is a script id, which is referenced from `sequence.fallback`, used for all locales. > And this one line already added allfonts.korean=Malgun Gothic will provide it as a fall back so I don't see the need. That's also not the case. With the patch applied, Serif font still doesn't have Malgun Gothic as a component. That line in the current patch has no effect at all - `allfonts.