On Tue, 26 Sep 2023 14:34:20 GMT, Alexey Ivanov wrote:
>> Alisen Chung has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> updated tests based on feedback
>
> test/jdk/javax/swing/ToolTipManager/bug50
On Thu, 28 Sep 2023 15:39:27 GMT, Alisen Chung wrote:
> Since this isn't a test for applet I think we should keep it for now
What do you mean?
All the test does is it verifies the `installComponents` method is accessible
in a subclass of `BasicMenuItemUI`. The
On Fri, 22 Sep 2023 12:39:04 GMT, Alexey Ivanov wrote:
>> test/jdk/javax/swing/ToolTipManager/bug5078214.java line 102:
>>
>>> 100: System.out.print("We need at least one screen with
>>> ");
>>> 101:
On Mon, 25 Sep 2023 22:00:33 GMT, Alisen Chung wrote:
>> test/jdk/javax/swing/BasicMenuUI/bug4244616.java line 67:
>>
>>> 65: action.actionPerformed(ev);
>>> 66: } catch (Exception ignored) {
>>> 67: }
>>
>> Are exceptions thrown here?
>
>
On Mon, 25 Sep 2023 22:32:05 GMT, Alisen Chung wrote:
>> Opening closed tests:
>> 12 javax/swing/ToolTipManager/5078214/bug5078214.java
>> 13 javax/swing/plaf/basic/BasicMenuItemUI/4239714/bug4239714.java
>> 14 javax/swing/plaf/basic/BasicMenuUI/4244616/bug4244616.java
>> 15
On Wed, 13 Sep 2023 09:17:16 GMT, Alexey Ivanov wrote:
> **Problem**
>
> On a standard display with the scale of 100%, the BoxLayout sample has
> blurred edges.
>
> It is the result of how SVG is rendered: the strokes fall between the pixel
> grid, therefore a 1-pixel w
On Thu, 14 Sep 2023 14:53:08 GMT, Alexey Ivanov wrote:
>> test/jdk/java/awt/Window/8294156/manyWindows/TwoWindowColumnsH.java line 96:
>>
>>> 94: private static final int COLUMNS = 3;
>>> 95:
>>> 96: public static void positionTestUI(List wind
On Wed, 13 Sep 2023 14:25:19 GMT, Alexey Ivanov wrote:
>> This enhances the `Builder` pattern added in
>> [JDK-8294535](https://bugs.openjdk.org/browse/JDK-8294535) with a new method
>> `testUI` which allows passing a lambda expression or a method reference to
>> cr
On Wed, 20 Sep 2023 18:12:25 GMT, Phil Race wrote:
>>> Shouldn't we mention here a window added by `testUI` via builder?
>>
>> I don't think it's necessary: the builder uses `addTestWindow` under the
>> hood. It's the builder that requires documentation.
>>
>> In fact, the entire class
On Wed, 20 Sep 2023 18:05:31 GMT, Phil Race wrote:
> One thing about these and the other enhancements I see is that there's an
> (increasing) need for a nice bit of documentation in PassFailJFrame itself
> about all its features.
> Easiest to do this as you are adding them, so a doc comment
On Mon, 25 Sep 2023 15:45:59 GMT, Alexey Ivanov wrote:
>> Alexander Zvegintsev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> review comments
>
> test/jdk/java/awt/Frame/DefaultSizeTest.java line 42:
&g
On Tue, 26 Sep 2023 14:43:13 GMT, Alexey Ivanov wrote:
>> I think it's fine the way it is now. Just in case the user sees some unusual
>> failure.
>
> Well, yes… I haven't thought about it.
Anyway I submitted [JDK-8317111](https://bugs.openjdk.org/browse/JDK-8317111):
Prov
On Tue, 26 Sep 2023 14:46:39 GMT, Alexey Ivanov wrote:
>> I have already tried this using keyboard shortcuts.
>>
>> Alt+F4 works fine on Window and Linux (many popular DEs close windows by
>> Alt+F4 by default, e.g. Gnome shell, Plasma).
>>
>> For macos we
On Wed, 27 Sep 2023 08:04:02 GMT, Christoph Langer wrote:
>>> Yet keeping tests in the same area closer seems like a good idea to me.
>>> Either line is fine with me.
>>
>> Let's go for the latter one, line 416.
>>
>>> I started an internal discussion in clientlibs to see whether there are
On Wed, 27 Sep 2023 08:10:16 GMT, Christoph Langer wrote:
>> Exclude java/awt/font/Rotate/RotatedTextTest.java until
>> [JDK-8219641](https://bugs.openjdk.org/browse/JDK-8219641) is fixed. The
>> error shows in several linux installations.
>
> Christoph Langer has updated the pull request with
On Mon, 25 Sep 2023 20:30:50 GMT, Alexander Zvegintsev
wrote:
>> Open sourcing several manual applet tests
>>
>>
>> test/jdk/java/awt/Frame/DefaultSizeTest.java
>> test/jdk/java/awt/LightweightComponent/LightweightCliprect.java
>> test/jdk/java/awt/event/KeyEvent/FunctionKeyTest.java
>>
On Mon, 25 Sep 2023 20:19:17 GMT, Alexander Zvegintsev
wrote:
> But this will probably be a task for one of the next test sprints.
Absolutely, it's too many changes. Yet it may make the test easier.
In fact, I thought about iterating the values automatically … and ask the
tester to click the
On Mon, 25 Sep 2023 20:23:27 GMT, Alexander Zvegintsev
wrote:
>> test/jdk/javax/swing/JFrame/DefaultCloseOperation.java line 78:
>>
>>> 76: - Select "Dispose" from the "JDialog Default Close Operation"
>>> ComboBox
>>> 77: - On the TestDialog, select "Close" from the
On Mon, 25 Sep 2023 14:15:52 GMT, Alexey Ivanov wrote:
> Yet keeping tests in the same area closer seems like a good idea to me.
> Either line is fine with me.
Let's go for the latter one, line 416.
> I started an internal discussion in clientlibs to see whether there are any
>
On Mon, 25 Sep 2023 18:31:04 GMT, Alexander Zvegintsev
wrote:
>> Open sourcing few tests:
>>
>> java/awt/Frame/FrameRepackTest.java
>> java/awt/Frame/FrameResizeTest/FrameResizeTest_1.java
>> java/awt/Frame/FrameResizeTest/FrameResizeTest_2.java
>> java/awt/Frame/WindowMoveTest.java
>
>
On Mon, 31 Jul 2023 10:26:44 GMT, Alexey Ivanov wrote:
> The private field `mask` is never used in `JFormattedTextField`.
>
> I couldn't find any usages of the field, including JNI. Therefore, it is safe
> to remove.
This pull request has now been integrated.
Changeset: b65
On Mon, 25 Sep 2023 16:46:26 GMT, Alexander Zvegintsev
wrote:
>> test/jdk/java/awt/Frame/FrameResizeTest/FrameResizeTest_1.java line 62:
>>
>>> 60: .build();
>>> 61:
>>> 62: SwingUtilities.invokeAndWait(() -> {
>>
>> Suggestion:
>>
>>
On Mon, 25 Sep 2023 17:21:13 GMT, Alexander Zvegintsev
wrote:
>> Open sourcing few tests:
>>
>> java/awt/Frame/FrameRepackTest.java
>> java/awt/Frame/FrameResizeTest/FrameResizeTest_1.java
>> java/awt/Frame/FrameResizeTest/FrameResizeTest_2.java
>> java/awt/Frame/WindowMoveTest.java
>
>
On Mon, 25 Sep 2023 16:48:51 GMT, Alexander Zvegintsev
wrote:
>> test/jdk/java/awt/Frame/FrameRepackTest.java line 134:
>>
>>> 132: north.setLayout(new BorderLayout(2, 2));
>>> 133: north.add("North", new Label(" This panel is always
>>> displayed"));
>>> 134:
On Wed, 20 Sep 2023 20:05:01 GMT, Phil Race wrote:
> Swing isn't serializable across releases, so I see no need for a CSR.
Thank you. That's what I thought.
-
PR Comment: https://git.openjdk.org/jdk/pull/15088#issuecomment-1734125157
On Fri, 22 Sep 2023 15:43:18 GMT, Alexander Zvegintsev
wrote:
>> Open sourcing several manual applet tests
>>
>>
>> test/jdk/java/awt/Frame/DefaultSizeTest.java
>> test/jdk/java/awt/LightweightComponent/LightweightCliprect.java
>> test/jdk/java/awt/event/KeyEvent/FunctionKeyTest.java
>>
On Mon, 25 Sep 2023 15:27:15 GMT, Alexander Zvegintsev
wrote:
>> Open sourcing few tests:
>>
>> java/awt/Frame/FrameRepackTest.java
>> java/awt/Frame/FrameResizeTest/FrameResizeTest_1.java
>> java/awt/Frame/FrameResizeTest/FrameResizeTest_2.java
>> java/awt/Frame/WindowMoveTest.java
>
>
On Wed, 20 Sep 2023 16:55:30 GMT, Alexander Zvegintsev
wrote:
>> Open sourcing few tests:
>>
>> java/awt/Frame/FrameRepackTest.java
>> java/awt/Frame/FrameResizeTest/FrameResizeTest_1.java
>> java/awt/Frame/FrameResizeTest/FrameResizeTest_2.java
>> java/awt/Frame/WindowMoveTest.java
>
>
On Mon, 25 Sep 2023 13:29:09 GMT, Christoph Langer wrote:
>> test/jdk/ProblemList.txt line 472:
>>
>>> 470:
>>> 471: java/awt/font/Rotate/RotatedTextTest.java 8219641 linux-all
>>> 472:
>>
>> Looks good to me, except for its placement: shouldn't it be placed close to
>> other font tests?
On Sun, 24 Sep 2023 06:12:17 GMT, Alisen Chung wrote:
>> Opening closed tests:
>> 12 javax/swing/ToolTipManager/5078214/bug5078214.java
>> 13 javax/swing/plaf/basic/BasicMenuItemUI/4239714/bug4239714.java
>> 14 javax/swing/plaf/basic/BasicMenuUI/4244616/bug4244616.java
>> 15
On Mon, 18 Sep 2023 19:08:48 GMT, Damon Nguyen wrote:
>> test/jdk/javax/swing/JTabbedPane/bug4703690.java line 54:
>>
>>> 52: static volatile boolean focusButtonTwo = false;
>>> 53: static volatile boolean switchToTabTwo = false;
>>> 54: static volatile boolean focusButtonOne =
On Sun, 24 Sep 2023 06:12:17 GMT, Alisen Chung wrote:
>> Opening closed tests:
>> 12 javax/swing/ToolTipManager/5078214/bug5078214.java
>> 13 javax/swing/plaf/basic/BasicMenuItemUI/4239714/bug4239714.java
>> 14 javax/swing/plaf/basic/BasicMenuUI/4244616/bug4244616.java
>> 15
On Mon, 25 Sep 2023 06:53:49 GMT, Prasanta Sadhukhan
wrote:
>> test/jdk/javax/swing/JTableHeader/SerializeJTableHeader.java line 55:
>>
>>> 53: for (UIManager.LookAndFeelInfo laf :
>>> UIManager.getInstalledLookAndFeels()) {
>>> 54: System.out.println("Testing L: " + laf);
On Mon, 25 Sep 2023 07:00:38 GMT, Prasanta Sadhukhan
wrote:
>> After the first time a JTableHeader is serialized, it no longer will
>> uninstall its UI upon subsequent serializations.
>> This happens for classes that use the BasicTableHeaderUI class. Any LAF that
>> extends the
On Fri, 22 Sep 2023 06:20:42 GMT, Christoph Langer wrote:
> Exclude java/awt/font/Rotate/RotatedTextTest.java until
> [JDK-8219641](https://bugs.openjdk.org/browse/JDK-8219641) is fixed. The
> error shows in several linux installations.
test/jdk/ProblemList.txt line 472:
> 470:
> 471:
On Sun, 24 Sep 2023 10:13:03 GMT, Andrey Turbanov wrote:
>> PR with non-swing changes extracted from initial #14847
>
> Andrey Turbanov has updated the pull request incrementally with one
> additional commit since the last revision:
>
> 8316556: Fix typos in java.desktop
Marked as reviewed
On Tue, 19 Sep 2023 13:41:27 GMT, Alexey Ivanov wrote:
> **Root cause**
>
> The _Baekmuk Headline_ font maps `\u6f22` (漢) to glyph id 16950 which has
> zero length.
>
> The test fails if the right half of the image with the rendered glyph
> contains only pixels
On Thu, 21 Sep 2023 22:36:16 GMT, Harshitha Onkar wrote:
>> Opening closed tests:
>> 12 javax/swing/ToolTipManager/5078214/bug5078214.java
>> 13 javax/swing/plaf/basic/BasicMenuItemUI/4239714/bug4239714.java
>> 14 javax/swing/plaf/basic/BasicMenuUI/4244616/bug4244616.java
>> 15
On Sat, 16 Sep 2023 16:22:40 GMT, Andrey Turbanov wrote:
> PR with non-swing changes extracted from initial #14847
src/java.desktop/unix/classes/sun/awt/X11/XMenuBarPeer.java line 132:
> 130: * need to have somewhat strange behaviour
> 131: * deduced from java.awt.MenuBar.
> 132:
On Fri, 22 Sep 2023 07:12:08 GMT, Prasanta Sadhukhan
wrote:
> I guess in test, resource leak probably wouldn't matter…
No, it wouldn't. Yet someone could later copy the code and use file streams
where closing the streams matters. Always following the pattern is safer.
-
PR
On Fri, 22 Sep 2023 07:18:09 GMT, Prasanta Sadhukhan
wrote:
>> After the first time a JTableHeader is serialized, it no longer will
>> uninstall its UI upon subsequent serializations.
>> This happens for classes that use the BasicTableHeaderUI class. Any LAF that
>> extends the
On Tue, 19 Sep 2023 21:51:01 GMT, Phil Race wrote:
> Are you saying this test would be better with an enhancement to
> PassFailJFrame ?
@prrace Yes. Or perhaps, with a reduced set implemented as another class.
> But it is OK right, just more clunky than it could be .. ?
Yes, it is OK. Yet
On Wed, 20 Sep 2023 14:07:17 GMT, Alexey Ivanov wrote:
>> **Root cause**
>>
>> The _Baekmuk Headline_ font maps `\u6f22` (漢) to glyph id 16950 which has
>> zero length.
>>
>> The test fails if the right half of the image with the rendered glyph
>>
o contains background-coloured pixels only.
>
> The updated test does not fail with the _Baekmuk Headline_ font. It still
> detects the original problem and fails on builds without the fix for
> [JDK-8312555](https://bugs.openjdk.org/browse/JDK-8312555).
Alexey Ivanov has updated the pull r
On Thu, 17 Aug 2023 21:37:17 GMT, Phil Race wrote:
>> The private field `mask` is never used in `JFormattedTextField`.
>>
>> I couldn't find any usages of the field, including JNI. Therefore, it is
>> safe to remove.
>
> Marked as reviewed by prr (Reviewer).
> @prrace could you confirm,
On Wed, 20 Sep 2023 07:52:33 GMT, Brice Dutheil wrote:
> I encountered an issue very similar to this problem, is there any chance to
> see progress on the fix ?
@bric3 This is not how it works… Yet feel free to propose a patch that
addresses the problem. It could be based on this PR.
I looks
On Tue, 19 Sep 2023 18:35:12 GMT, Phil Race wrote:
>> Refactored timeout handling in PassFailJFrame:
>>
>> 1. Managing the timer and formatting the time left is inside
>> `TimeoutHandler` class.
>> 2. The class handles timer events and updates the label accordingly.
>>
>> This is implemented
On Tue, 19 Sep 2023 17:04:59 GMT, Damon Nguyen wrote:
>> These are the tests being converted:
>>
>> javax/swing/JToolBar/4138694/bug4138694.java
>> javax/swing/JToolBar/4140421/bug4140421.java
>> javax/swing/JToolBar/4196662/bug4196662.java
>> javax/swing/JToolBar/4243930/bug4243930.java
>
>
On Tue, 19 Sep 2023 17:02:54 GMT, Harshitha Onkar wrote:
>> Following tests are open sourced as part of this PR.
>>
>> 1. test/jdk/javax/swing/text/html/CSS/bug4174871.java
>> 2. test/jdk/javax/swing/text/html/CSS/bug4174874.java
>> 3. test/jdk/javax/swing/text/html/CSS/bug4284162.java
>> 4.
On Tue, 19 Sep 2023 16:55:02 GMT, Harshitha Onkar wrote:
>> test/jdk/javax/swing/text/html/HTMLDocument/bug4209280.java line 36:
>>
>>> 34: public static void main(String[] args) throws Exception {
>>> 35: String html = "Foo";
>>> 36: new JLabel(html);
>>
>> Suggestion:
>>
On Tue, 19 Sep 2023 16:51:31 GMT, Harshitha Onkar wrote:
>> Following tests are open sourced as part of this PR.
>>
>> 1. java/awt/event/PaintEvent/RepaintTest/RepaintTest.java
>> 2. javax/swing/text/html/HTMLEditorKit/4214848/bug4214848.java
>> 3.
On Tue, 19 Sep 2023 09:15:41 GMT, Arno Zeller wrote:
> > The bug report has "Angle: 15, width diff: -4" Over the length of the
> > string "The quick brown fox jumps over the lazy dog" that could be the
> > occasional rounding difference and there is a comment in the bug report
> > claiming
On Tue, 19 Sep 2023 16:02:21 GMT, Harshitha Onkar wrote:
> Better to close the stream objects in finally block in case of Exception.
>
> ```
> finally {
> if (oos != null) {
> oos.close();
> }
> if (ois != null) {
> ois.close();
> }
> }
> ```
No,
**Root cause**
The _Baekmuk Headline_ font maps `\u6f22` (漢) to glyph id 16950 which has zero
length.
The test fails if the right half of the image with the rendered glyph contains
only pixels of background colour. In this case, the left half of the image is
also blank. It's somewhat false
On Tue, 19 Sep 2023 00:00:45 GMT, Harshitha Onkar wrote:
> Looks like a valid explanation as to why `x + width` is used.
This is why `cellWidth` is a bad name: it doesn't convey the meaning. Consider
*renaming the variable* or adding a comment to clarify the meaning.
-
PR Review
On Tue, 19 Sep 2023 00:22:35 GMT, Harshitha Onkar wrote:
>> Following tests are open sourced as part of this PR.
>>
>> 1. test/jdk/javax/swing/text/html/CSS/bug4174871.java
>> 2. test/jdk/javax/swing/text/html/CSS/bug4174874.java
>> 3. test/jdk/javax/swing/text/html/CSS/bug4284162.java
>> 4.
On Tue, 19 Sep 2023 00:03:34 GMT, Harshitha Onkar wrote:
>> With volatile, it makes the test slightly slower. Other than that, there's
>> not much harm.
>>
>> In most of these tests, there's no need for the `passed` field at all: you
>> can throw the exception from the code where you set the
On Mon, 18 Sep 2023 23:30:26 GMT, Damon Nguyen wrote:
>> These are the tests being converted:
>>
>> javax/swing/JToolBar/4138694/bug4138694.java
>> javax/swing/JToolBar/4140421/bug4140421.java
>> javax/swing/JToolBar/4196662/bug4196662.java
>> javax/swing/JToolBar/4243930/bug4243930.java
>
>
On Mon, 18 Sep 2023 21:53:36 GMT, Harshitha Onkar wrote:
>> Following tests are open sourced as part of this PR.
>>
>> 1. java/awt/event/PaintEvent/RepaintTest/RepaintTest.java
>> 2. javax/swing/text/html/HTMLEditorKit/4214848/bug4214848.java
>> 3.
On Mon, 18 Sep 2023 19:30:46 GMT, Damon Nguyen wrote:
>> These are the tests being converted:
>>
>> javax/swing/JToolBar/4368050/bug4368050.java
>> javax/swing/JToolBar/4465534/bug4465534.java
>> javax/swing/JToolBar/4700351/bug4700351.java
>> javax/swing/JToolTip/4107843/bug4107843.java
>>
On Mon, 18 Sep 2023 19:09:41 GMT, Damon Nguyen wrote:
>> These are the tests being converted:
>>
>> javax/swing/JTree/4210432/bug4210432.java
>> javax/swing/JTree/4213868/bug4213868.java
>> javax/swing/JTree/4224491/bug4224491.java
>> javax/swing/JTree/4237370/bug4237370.java
>>
On Mon, 18 Sep 2023 19:14:42 GMT, Damon Nguyen wrote:
>> These are the tests being converted:
>>
>> javax/swing/JTabbedPane/4703690/bug4703690.java
>> javax/swing/JTabbedPane/GetComponentAt/GetComponentAtTest.java
>> javax/swing/JTabbedPane/ReplaceCompTab/ReplaceCompTab.java
>>
On Mon, 18 Sep 2023 18:54:39 GMT, Harshitha Onkar wrote:
>> Following tests are open sourced as part of this PR.
>>
>> 1. test/jdk/javax/swing/text/html/CSS/bug4174871.java
>> 2. test/jdk/javax/swing/text/html/CSS/bug4174874.java
>> 3. test/jdk/javax/swing/text/html/CSS/bug4284162.java
>> 4.
On Mon, 18 Sep 2023 18:16:38 GMT, Harshitha Onkar wrote:
>> test/jdk/javax/swing/text/html/CSS/bug4174871.java line 41:
>>
>>> 39: private static JFrame frame;
>>> 40: private static JTextPane pane;
>>> 41: private static volatile boolean passed = false;
>>
>> Explicitly assigning
On Mon, 18 Sep 2023 19:00:48 GMT, Damon Nguyen wrote:
>> These are the tests being converted:
>>
>> javax/swing/JTabbedPane/4703690/bug4703690.java
>> javax/swing/JTabbedPane/GetComponentAt/GetComponentAtTest.java
>> javax/swing/JTabbedPane/ReplaceCompTab/ReplaceCompTab.java
>>
On Mon, 18 Sep 2023 18:55:37 GMT, Damon Nguyen wrote:
>> These are the tests being converted:
>>
>> javax/swing/JToolBar/4138694/bug4138694.java
>> javax/swing/JToolBar/4140421/bug4140421.java
>> javax/swing/JToolBar/4196662/bug4196662.java
>> javax/swing/JToolBar/4243930/bug4243930.java
>
>
On Mon, 18 Sep 2023 17:56:28 GMT, Harshitha Onkar wrote:
>> test/jdk/javax/swing/text/html/HTMLDocument/bug4209280.java line 41:
>>
>>> 39: JFrame jFrame = new JFrame("Unknown HTML tag Test");
>>> 40: String html = "Foo";
>>> 41: JLabel label = new
On Mon, 18 Sep 2023 14:34:31 GMT, Matthias Baesken wrote:
> The test RotatedTextTest shows a small width difference in the rotation (but
> the test assumes the width difference is 0 and fails). I wonder if the test
> can really assume a width difference 0 , is this really what we can expect
>
On Mon, 18 Sep 2023 06:19:05 GMT, Arno Zeller wrote:
> The test fails on newer SLES versions that have RobotoSlab-Regular.ttf as
> default font. I suggest to try getting a DejaVu font as default on Linux
> because it is known to work without issues.
What is the root cause for the test failing
On Thu, 14 Sep 2023 21:40:55 GMT, Damon Nguyen wrote:
> These are the tests being converted:
>
> javax/swing/JTree/4210432/bug4210432.java
> javax/swing/JTree/4213868/bug4213868.java
> javax/swing/JTree/4224491/bug4224491.java
> javax/swing/JTree/4237370/bug4237370.java
>
On Thu, 14 Sep 2023 21:34:56 GMT, Damon Nguyen wrote:
> These are the tests being converted:
>
> javax/swing/JToolBar/4368050/bug4368050.java
> javax/swing/JToolBar/4465534/bug4465534.java
> javax/swing/JToolBar/4700351/bug4700351.java
> javax/swing/JToolTip/4107843/bug4107843.java
>
On Thu, 14 Sep 2023 16:35:44 GMT, Damon Nguyen wrote:
> These are the tests being converted:
>
> javax/swing/JTabbedPane/4703690/bug4703690.java
> javax/swing/JTabbedPane/GetComponentAt/GetComponentAtTest.java
> javax/swing/JTabbedPane/ReplaceCompTab/ReplaceCompTab.java
>
On Thu, 14 Sep 2023 16:43:29 GMT, Damon Nguyen wrote:
> These are the tests being converted:
>
> javax/swing/JToolBar/4138694/bug4138694.java
> javax/swing/JToolBar/4140421/bug4140421.java
> javax/swing/JToolBar/4196662/bug4196662.java
> javax/swing/JToolBar/4243930/bug4243930.java
On Fri, 15 Sep 2023 22:54:32 GMT, Harshitha Onkar wrote:
>> These are the tests being converted:
>>
>> javax/swing/JTabbedPane/4703690/bug4703690.java
>> javax/swing/JTabbedPane/GetComponentAt/GetComponentAtTest.java
>> javax/swing/JTabbedPane/ReplaceCompTab/ReplaceCompTab.java
>>
On Thu, 14 Sep 2023 16:35:44 GMT, Damon Nguyen wrote:
> These are the tests being converted:
>
> javax/swing/JTabbedPane/4703690/bug4703690.java
> javax/swing/JTabbedPane/GetComponentAt/GetComponentAtTest.java
> javax/swing/JTabbedPane/ReplaceCompTab/ReplaceCompTab.java
>
On Thu, 14 Sep 2023 18:45:34 GMT, Harshitha Onkar wrote:
> Following tests are open sourced as part of this PR.
>
> 1. java/awt/event/PaintEvent/RepaintTest/RepaintTest.java
> 2. javax/swing/text/html/HTMLEditorKit/4214848/bug4214848.java
> 3.
On Fri, 15 Sep 2023 19:35:15 GMT, Harshitha Onkar wrote:
> Following tests are open sourced as part of this PR.
>
> 1. test/jdk/javax/swing/text/html/CSS/bug4174871.java
> 2. test/jdk/javax/swing/text/html/CSS/bug4174874.java
> 3. test/jdk/javax/swing/text/html/CSS/bug4284162.java
> 4.
On Tue, 1 Aug 2023 04:16:30 GMT, Renjith Kannath Pariyangad
wrote:
>> Renjith Kannath Pariyangad has updated the pull request incrementally with
>> one additional commit since the last revision:
>>
>> Added CoInitialize status check
>
> Did further investigation on JDK-7116070 (name
On Thu, 3 Aug 2023 10:35:50 GMT, Renjith Kannath Pariyangad
wrote:
>> Hi Reviewers,
>>
>> Observations :
>> 1. Without com initialize if we access Mixer for recording, library loaded
>> invalid GUID and clipped description in windows(ID not found in registry).
>> With com initialization
On Thu, 3 Aug 2023 04:02:16 GMT, Renjith Kannath Pariyangad
wrote:
>> src/java.desktop/windows/native/libjsound/PLATFORM_API_WinOS_DirectSound.cpp
>> line 183:
>>
>>> 181: INT32 cacheIndex;
>>> 182:
>>> 183: if (!DS_lockCache() || FAILED(::CoInitialize(NULL))) {
>>
>> I believe this
;
> Also, I marked up references to classes and members with `{@code}`.
>
> Look at the updated version: [`Document.remove` in JDK
> 22](https://cr.openjdk.org/~aivanov/4622866/api/java.desktop/javax/swing/text/Document.html#remove(int,int)).
Alexey Ivanov has updated the pull requ
On Wed, 13 Sep 2023 21:17:46 GMT, Alexander Zvegintsev
wrote:
>> Alexey Ivanov has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8294156: Additional layouts for vertically positioned test windows
>
> te
On Tue, 12 Sep 2023 19:59:27 GMT, Andrey Turbanov wrote:
> Do we need a second reviewer or I can integrate?
It's a good question.
Overall, the changes are trivial: only comments are updated, some javadoc for
private classes which are comments, then a couple of method names and variable
names
On Wed, 30 Aug 2023 14:08:21 GMT, Alexey Ivanov wrote:
> I reverted changes in non-Swing parts to reduce amount of files to review.
Did you submit a follow-up bug for those files?
Do you have a PR already?
-
PR Comment: https://git.openjdk.org/jdk/pull/14847#issuecomm
On Tue, 12 Sep 2023 21:33:44 GMT, Alexander Zvegintsev
wrote:
>> test/jdk/java/awt/Choice/ChoiceSelectTest.java line 160:
>>
>>> 158: public static void main(String[] args) throws Exception {
>>> 159: EventQueue.invokeAndWait(() -> new ChoiceSelectTest().test());
>>> 160: }
>>
On Tue, 12 Sep 2023 21:52:33 GMT, Alexander Zvegintsev
wrote:
>> Some closed AWT test are open sourced.
>
> Alexander Zvegintsev has updated the pull request incrementally with one
> additional commit since the last revision:
>
> fix path
Looks good.
-
Marked as reviewed by
On Wed, 13 Sep 2023 18:50:54 GMT, Alexey Ivanov wrote:
>> The manual test frame does not provide any layout managers for test windows,
>> the user has to implement one. This demonstrates how to position a several
>> test UI windows.
>>
>> We may include some i
On Tue, 12 Sep 2023 19:51:45 GMT, Alexey Ivanov wrote:
>>> However, to test this case I'll need a (simple) scenario where multiple
>>> windows are created.
>>
>> For example
>> `test/jdk/java/awt/event/MouseEvent/SpuriousExitEnter/SpuriousExitEnter
the tests: `test/jdk/java/awt/Window/8294156`.
>
> Alternatively, you can checkout directly from my fork:
>
>
> git fetch https://github.com/aivanov-jdk/jdk.git
> demo-manyTestWindows:demo-manyTestWindows
> git checkout demo-manyTestWindows
Alexey Ivanov has updated the pull request
g a manual test.
>
> This change depends on #15665 and
> [JDK-8294156](https://bugs.openjdk.org/browse/JDK-8294156) as well as #15661,
> both of which depend on #15660.
Alexey Ivanov has updated the pull request with a new target base due to a
merge or a rebase. The incremental webr
On Wed, 13 Sep 2023 14:25:19 GMT, Alexey Ivanov wrote:
>> This enhances the `Builder` pattern added in
>> [JDK-8294535](https://bugs.openjdk.org/browse/JDK-8294535) with a new method
>> `testUI` which allows passing a lambda expression or a method reference to
>> cr
The manual test frame does not provide any layout managers for test windows,
the user has to implement one. This demonstrates how to position a several test
UI windows.
We may include some implementations into the manual test framework in the
future.
**This PR is not meant to be integrated.**
stUI(() -> createTestUI())
> .build()
> .awaitAndCheck();
> }
>
> where `createTestUI` returns a test UI window.
Alexey Ivanov has updated the pull request incrementally with one additional
commit since the last revision:
8294156: Allow creating several t
**Problem**
On a standard display with the scale of 100%, the BoxLayout sample has blurred
edges.
It is the result of how SVG is rendered: the strokes fall between the pixel
grid, therefore a 1-pixel wide stroke is blurred between the previous and the
following pixels with 50% of colour on
On Wed, 13 Sep 2023 07:46:50 GMT, Alexey Ivanov wrote:
> The image in the [documentation for
> `Document.remove`](https://docs.oracle.com/en/java/javase/17/docs/api/java.desktop/javax/swing/text/Document.html#remove(int,int))
> looks as if the portion between the offsets 6 and 10 i
The image in the [documentation for
`Document.remove`](https://docs.oracle.com/en/java/javase/17/docs/api/java.desktop/javax/swing/text/Document.html#remove(int,int))
looks as if the portion between the offsets 6 and 10 is removed. However, the
entire region for _‘quick ’_ is highlighted.
I
On Tue, 12 Sep 2023 12:26:38 GMT, Alexander Zvegintsev
wrote:
>> Some closed AWT test are open sourced.
>
> Alexander Zvegintsev has updated the pull request incrementally with one
> additional commit since the last revision:
>
> spacing
Changes requested by aivanov (Reviewer).
On Tue, 12 Sep 2023 18:41:23 GMT, Alexander Zvegintsev
wrote:
>>> Sometimes we may need to create more than one window(e.g. some DnD test
>>> with two windows). Perhaps we should consider that here by adding
>>> `List createTestUI()`.
>>
>> I thought about it. Yet I couldn't find a simple
them stand out. The test instructions for single- and
> multi-selection are emphasised with bold text.
>
> This change depends on #15660 and
> [JDK-8294158](https://bugs.openjdk.org/browse/JDK-8294158).
Alexey Ivanov has updated the pull request incrementally with one additional
com
On Tue, 12 Sep 2023 13:49:41 GMT, Alexander Zvegintsev
wrote:
>>> Probably `trim()` and `toLowerCase()` could be useful here.
>>
>> Could be… On the other hand, in post XHTML era, the elements are usually in
>> lower-case. Thus we should stick with it and use lowercase HTML elements
>>
901 - 1000 of 1976 matches
Mail list logo