Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v6]

2022-08-05 Thread Harshitha Onkar
On Fri, 5 Aug 2022 19:37:57 GMT, Harshitha Onkar wrote: >> test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 297: >> >>> 295: .getDefaultScreenDevice().getDefaultConfiguration(); >>> 296: Insets screenInsets = >>> Toolkit.getDefaultToolkit().getScreenInsets(gc);

Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v7]

2022-08-05 Thread Harshitha Onkar
> Additional position setting (TOP_LEFT_CORNER) and a method to obtain bounds > of test instruction frame are added to PassFailJFrame to handle positioning > of multiple test frames. > > In scenarios where multiple test windows might be present, the test windows > might overlap the instruction

Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v6]

2022-08-05 Thread Harshitha Onkar
On Fri, 5 Aug 2022 20:26:31 GMT, Phil Race wrote: >> Is that possible? Toolkit thread is not EDT, therefore native events are >> processed, it's probably enough to update the frame coordinates while EDT is >> blocked and doesn't handle events. > > Can you expand on what you mean ? > What's the

Re: RFR: 8291266: RenderPerfTest: missing content while rendering some primitives [v2]

2022-08-05 Thread Alexey Ushakov
On Fri, 5 Aug 2022 18:01:26 GMT, Alexey Ushakov wrote: >> Do not cause redundant endEncoder calls with batch processing of drawing >> primitives > > Alexey Ushakov has updated the pull request incrementally with one additional > commit since the last revision: > > 8291266: RenderPerfTest: mi

Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v6]

2022-08-05 Thread Phil Race
On Fri, 5 Aug 2022 20:14:35 GMT, Alexey Ivanov wrote: >> The intention is that it can be called either way - on or off. > > Is that possible? Toolkit thread is not EDT, therefore native events are > processed, it's probably enough to update the frame coordinates while EDT is > blocked and doesn

Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v6]

2022-08-05 Thread Alexey Ivanov
On Fri, 5 Aug 2022 20:10:48 GMT, Phil Race wrote: >> test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 309: >> >>> 307: } catch (InterruptedException e) { >>> 308: e.printStackTrace(); >>> 309: } >> >> Is it safe to call on EDT? >> >> If `positio

Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v6]

2022-08-05 Thread Phil Race
On Fri, 5 Aug 2022 19:18:21 GMT, Alexey Ivanov wrote: >> Harshitha Onkar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> added null check in positionWindow > > test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 309: > >> 307:

Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v6]

2022-08-05 Thread Harshitha Onkar
On Fri, 5 Aug 2022 19:13:18 GMT, Alexey Ivanov wrote: >> Harshitha Onkar has updated the pull request incrementally with one >> additional commit since the last revision: >> >> added null check in positionWindow > > test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 297: > >> 295:

Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v6]

2022-08-05 Thread Alexey Ivanov
On Fri, 5 Aug 2022 17:42:03 GMT, Harshitha Onkar wrote: >> Additional position setting (TOP_LEFT_CORNER) and a method to obtain bounds >> of test instruction frame are added to PassFailJFrame to handle positioning >> of multiple test frames. >> >> In scenarios where multiple test windows might

Re: RFR: 8288882: JFileChooser - empty (0 bytes) file is displayed as 1 KB [v14]

2022-08-05 Thread Alexey Ivanov
On Thu, 28 Jul 2022 10:08:24 GMT, Abhishek Kumar wrote: >> JFileChooser - empty file size issue fixed. >> For empty file, now the size 0 bytes. >> Manual Test Case "ZeroFileSizeCheck.java" created. > > Abhishek Kumar has updated the pull request incrementally with one additional > commit since

Re: RFR: 8288882: JFileChooser - empty (0 bytes) file is displayed as 1 KB [v14]

2022-08-05 Thread Alexey Ivanov
On Thu, 28 Jul 2022 10:08:24 GMT, Abhishek Kumar wrote: >> JFileChooser - empty file size issue fixed. >> For empty file, now the size 0 bytes. >> Manual Test Case "ZeroFileSizeCheck.java" created. > > Abhishek Kumar has updated the pull request incrementally with one additional > commit since

Re: RFR: 8291266: RenderPerfTest: missing content while rendering some primitives [v2]

2022-08-05 Thread Phil Race
On Fri, 5 Aug 2022 18:01:26 GMT, Alexey Ushakov wrote: >> Do not cause redundant endEncoder calls with batch processing of drawing >> primitives > > Alexey Ushakov has updated the pull request incrementally with one additional > commit since the last revision: > > 8291266: RenderPerfTest: mi

Re: RFR: 8291266: RenderPerfTest: missing content while rendering some primitives [v2]

2022-08-05 Thread Alexey Ushakov
> Do not cause redundant endEncoder calls with batch processing of drawing > primitives Alexey Ushakov has updated the pull request incrementally with one additional commit since the last revision: 8291266: RenderPerfTest: missing content while rendering some primitives Added regression

Re: RFR: 8291959: FileFontStrike#initNative does not properly initialize IG Table on Windows

2022-08-05 Thread Phil Race
On Fri, 5 Aug 2022 09:46:13 GMT, Julian Waters wrote: > FileFontStrike#initNative calls memset with LCDLUTCOUNT (the maximum length) > as the number of bytes to zero out, which is incorrect as the elements are > not 1 byte in size (unsigned char*), and will result in only part of the IG > Tabl

Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v3]

2022-08-05 Thread Harshitha Onkar
On Fri, 22 Jul 2022 20:31:44 GMT, Phil Race wrote: >> Following is a proposed solution to push the latest position changes to the >> window manager. >> >> The get and setLocation calls are wrapped in invokeLater(). >> Robot.waitForIdle() are added between setLocation() and the subsequent >> g

Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v5]

2022-08-05 Thread Harshitha Onkar
On Fri, 5 Aug 2022 00:45:59 GMT, Harshitha Onkar wrote: >> Additional position setting (TOP_LEFT_CORNER) and a method to obtain bounds >> of test instruction frame are added to PassFailJFrame to handle positioning >> of multiple test frames. >> >> In scenarios where multiple test windows might

Re: RFR: JDK-8290469: Add new positioning options to PassFailJFrame test framework [v6]

2022-08-05 Thread Harshitha Onkar
> Additional position setting (TOP_LEFT_CORNER) and a method to obtain bounds > of test instruction frame are added to PassFailJFrame to handle positioning > of multiple test frames. > > In scenarios where multiple test windows might be present, the test windows > might overlap the instruction

Re: RFR: 7189422: [macosx] Submenu's arrow have a wrong position

2022-08-05 Thread Phil Race
On Fri, 5 Aug 2022 08:58:51 GMT, Prasanta Sadhukhan wrote: > Issue is Arrow in submenu with empty title have a wrong position in Aqua L&F > as can be seen > > src="https://user-images.githubusercontent.com/43534309/183023538-de8e51b4-31e6-45d7-b2bd-35da5e29c1e8.png";> > > > which is because

Re: RFR: 6521141: DebugGraphics NPE @ setFont(); [v9]

2022-08-05 Thread Phil Race
On Fri, 5 Aug 2022 11:35:54 GMT, Tejesh R wrote: >> `DebugGraphics` class has a Graphics instance which is been used in slowed >> down drawing. The `graphics` object is not initialized anywhere inside the >> class, where it is expected to set explicitly by the user. When the user >> doesn't se

Re: RFR: 8291640: java/beans/XMLDecoder/8028054/Task.java should use the 3-arg Class.forName [v2]

2022-08-05 Thread Dio Brando
On Fri, 5 Aug 2022 02:25:11 GMT, Ao Qi wrote: >> The issue comes from >> https://github.com/openjdk/jdk/pull/9677#issuecomment-1200811357. >> >> If Loom is not supported, two XMLDecoder tests would fail. The issue could >> be reproduced by zero, for example: >> >> >> ---

Re: RFR: 6521141: DebugGraphics NPE @ setFont(); [v9]

2022-08-05 Thread Dio Brando
On Fri, 5 Aug 2022 11:35:54 GMT, Tejesh R wrote: >> `DebugGraphics` class has a Graphics instance which is been used in slowed >> down drawing. The `graphics` object is not initialized anywhere inside the >> class, where it is expected to set explicitly by the user. When the user >> doesn't se

Re: RFR: 7189422: [macosx] Submenu's arrow have a wrong position

2022-08-05 Thread Dio Brando
On Fri, 5 Aug 2022 08:58:51 GMT, Prasanta Sadhukhan wrote: > Issue is Arrow in submenu with empty title have a wrong position in Aqua L&F > as can be seen > > src="https://user-images.githubusercontent.com/43534309/183023538-de8e51b4-31e6-45d7-b2bd-35da5e29c1e8.png";> > > > which is because

Re: RFR: 8290973: In AffineTransform, equals(Object) is inconsistent with hashCode()

2022-08-05 Thread Dio Brando
On Fri, 10 Jun 2022 09:39:48 GMT, Martin Desruisseaux wrote: > `AffineTransform.equals(Object)` and `hashCode()` break two contracts: > > * `A.equals(A)` returns `false` if at least one affine transform coefficient > is NaN. > * `A.equals(B)` should imply `A.hashCode() == B.hashCode()`, but it

Re: RFR: 8281966: Absolute path of symlink is null in JFileChooser [v6]

2022-08-05 Thread Dio Brando
On Fri, 5 Aug 2022 05:46:20 GMT, Tejesh R wrote: >> Absolute path of Symbolic Link created in Windows is set to `null` in >> `BasicFileChooserUI` class. This happens when propertyChangeListener is >> implemented to get the Symbolic link's Absolute path on Mouse click through >> JFileChooser. T

Re: RFR: 8269806: Emoji rendering on Linux [v13]

2022-08-05 Thread Nikita Gubarkov
> It was implemented in JetBrains Runtime a year ago and was ported & > refactored for this PR > It includes: > - Bitmap glyph loading via Freetype > - Manual scaling & transformation of bitmap glyphs with nearest-neighbor or > bilinear-mipmap style algorithms depending on the text antialiasing h

Re: RFR: 8281966: Absolute path of symlink is null in JFileChooser [v6]

2022-08-05 Thread Alexey Ivanov
On Fri, 5 Aug 2022 05:46:20 GMT, Tejesh R wrote: >> Absolute path of Symbolic Link created in Windows is set to `null` in >> `BasicFileChooserUI` class. This happens when propertyChangeListener is >> implemented to get the Symbolic link's Absolute path on Mouse click through >> JFileChooser. T

Re: RFR: 6521141: DebugGraphics NPE @ setFont(); [v8]

2022-08-05 Thread Tejesh R
On Fri, 5 Aug 2022 10:52:43 GMT, Prasanta Sadhukhan wrote: >> Tejesh R has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Updated based on review comments >> - Updated based on review comments > > test/jdk/javax/swing/DebugGraphics/Debu

Re: RFR: 6521141: DebugGraphics NPE @ setFont(); [v9]

2022-08-05 Thread Tejesh R
> `DebugGraphics` class has a Graphics instance which is been used in slowed > down drawing. The `graphics` object is not initialized anywhere inside the > class, where it is expected to set explicitly by the user. When the user > doesn't set it and try to use the any mehtods like `drawing/setFo

Re: RFR: 6521141: DebugGraphics NPE @ setFont(); [v8]

2022-08-05 Thread Prasanta Sadhukhan
On Fri, 5 Aug 2022 09:32:05 GMT, Tejesh R wrote: >> `DebugGraphics` class has a Graphics instance which is been used in slowed >> down drawing. The `graphics` object is not initialized anywhere inside the >> class, where it is expected to set explicitly by the user. When the user >> doesn't se

RFR: 8291959: FileFontStrike#initNative does not properly initialize IG Table on Windows

2022-08-05 Thread Julian Waters
FileFontStrike#initNative calls memset with LCDLUTCOUNT (the maximum length) as the number of bytes to zero out, which is incorrect as the elements are not 1 byte in size (unsigned char*), and will result in only part of the IG Table being correctly zeroed. This is more correctly resolved by pas

Re: RFR: 6521141: DebugGraphics NPE @ setFont(); [v7]

2022-08-05 Thread Tejesh R
On Thu, 4 Aug 2022 20:24:38 GMT, Phil Race wrote: >> Tejesh R has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Updated based on review comments > > src/java.desktop/share/classes/javax/swing/DebugGraphics.java line 79: > >> 77: * >

Re: RFR: 6521141: DebugGraphics NPE @ setFont(); [v4]

2022-08-05 Thread Tejesh R
On Thu, 28 Jul 2022 18:11:02 GMT, Phil Race wrote: >> Tejesh R has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Removed whitespace error > > You wrote " The graphics object is not initialized anywhere inside the class, > where it is expe

Re: RFR: 6521141: DebugGraphics NPE @ setFont(); [v8]

2022-08-05 Thread Tejesh R
> `DebugGraphics` class has a Graphics instance which is been used in slowed > down drawing. The `graphics` object is not initialized anywhere inside the > class, where it is expected to set explicitly by the user. When the user > doesn't set it and try to use the any mehtods like `drawing/setFo

RFR: 7189422: [macosx] Submenu's arrow have a wrong position

2022-08-05 Thread Prasanta Sadhukhan
Issue is Arrow in submenu with empty title have a wrong position in Aqua L&F as can be seen https://user-images.githubusercontent.com/43534309/183023538-de8e51b4-31e6-45d7-b2bd-35da5e29c1e8.png";> which is because the text being null/empty, `labelR` rectangle width/height is 0 so arrowIcon y c

Re: RFR: 8290973: In AffineTransform, equals(Object) is inconsistent with hashCode()

2022-08-05 Thread Martin Desruisseaux
On Fri, 10 Jun 2022 09:39:48 GMT, Martin Desruisseaux wrote: > `AffineTransform.equals(Object)` and `hashCode()` break two contracts: > > * `A.equals(A)` returns `false` if at least one affine transform coefficient > is NaN. > * `A.equals(B)` should imply `A.hashCode() == B.hashCode()`, but it

Re: RFR: 8289208: Test DrawRotatedStringUsingRotatedFont.java occasionally crashes on MacOS

2022-08-05 Thread Maxim Kartashev
On Wed, 13 Jul 2022 08:58:38 GMT, Sergey Bylokhov wrote: >>> The code that disposes on the rendering thread is invoked from a dispose() >>> method >> that was on the Disposer thread. It then waits for that to finish. >> At that time the Disposer thread is blocked so not doing anything and the >

Re: RFR: 8291640: java/beans/XMLDecoder/8028054/Task.java should use the 3-arg Class.forName

2022-08-05 Thread Ao Qi
On Wed, 3 Aug 2022 05:59:34 GMT, Alan Bateman wrote: >>> One other comment on this is that the proposed change should mean that the >>> tests no longer need to run with --enable-preview. >> >> You've lost me all over again. >> Why did these two tests need that in the first place and why would t