Re: RFR: 8306812: Open source several AWT Miscellaneous tests [v3]

2023-05-10 Thread Tejesh R
> Open source few AWT Miscellaneous ( Panel, Popup, robot and scrollbar) tests. 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 - Changes: - all:

Re: RFR: 8306812: Open source several AWT Miscellaneous tests [v2]

2023-05-10 Thread Tejesh R
On Wed, 10 May 2023 16:41:55 GMT, Damon Nguyen wrote: >> Tejesh R has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Updated based on review comments > > test/jdk/java/awt/Robot/RobotMoveMultiscreen.java line 81: > >> 79:

Re: RFR: 8280982: [Wayland] [XWayland] java.awt.Robot taking screenshots [v2]

2023-05-10 Thread Phil Race
On Tue, 9 May 2023 22:09:26 GMT, Alexander Zvegintsev wrote: > > I want to work through the scenarios and how much of it is specific to the > > behaviours of the API you are using and so forth. Since you use the > > Preferences API for saving the token, if you keyed it off the Robot class >

Re: RFR: 8307779: Relax the java.awt.Robot specification [v3]

2023-05-10 Thread Phil Race
On Wed, 10 May 2023 19:23:39 GMT, Phil Race wrote: >> I agree, I'll try to come up with something less confusing. > > If there's no exception, how does the application know the colours aren't > valid ? > I think throwing the exception should be preferred behaviour and undefined > colours would

Re: RFR: 8307194: Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries [v12]

2023-05-10 Thread Jiangli Zhou
On Wed, 10 May 2023 17:13:13 GMT, Jiangli Zhou wrote: >> This PR is branched from the makefile changes for >> https://bugs.openjdk.org/browse/JDK-8303796 and contains the following for >> handling the JDK/hotspot static libraries: >> >> - Build hotspot libjvm.a and JDK static libraries for

Re: RFR: 8306996: Open source Swing MenuItem related tests [v2]

2023-05-10 Thread Damon Nguyen
On Wed, 10 May 2023 20:54:26 GMT, Harshitha Onkar wrote: >> Which part is a Util method? I see a JRobot method. Or is Util and JRobot >> linked somehow? My understanding is that they're separate since they're >> separate in regtesthelpers. > > @DamonGuy 'clickMouseOn()'. Earlier this test had

Re: RFR: 8307194: Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries [v12]

2023-05-10 Thread Erik Joelsson
On Wed, 10 May 2023 20:38:50 GMT, Jiangli Zhou wrote: > > This change caused all our builds but Linux to fail. Did you verify on > > other platforms than Linux at all? I see you have GHA turned off. > > Sorry about the issue. There were failed workflows after I merged with the > latest JDK

Re: RFR: 8307779: Relax the java.awt.Robot specification [v2]

2023-05-10 Thread Alexander Zvegintsev
On Wed, 10 May 2023 19:20:48 GMT, Phil Race wrote: >> Alexander Zvegintsev has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - rephrase wording about mouse pointer may not visually move >> - Remove mention of mouse cursor for screen

Re: RFR: 8307779: Relax the java.awt.Robot specification [v3]

2023-05-10 Thread Alexander Zvegintsev
> We need to relax the java.awt.Robot specification according to the latest > operating system trends. > This should at least cover the case of Wayland, which has changed many > familiar concepts in Linux. > > https://bugs.openjdk.org/browse/JDK-8280982 [Wayland] [XWayland] > java.awt.Robot

Re: RFR: 8306996: Open source Swing MenuItem related tests [v2]

2023-05-10 Thread Harshitha Onkar
On Mon, 8 May 2023 16:23:09 GMT, Damon Nguyen wrote: >> test/jdk/javax/swing/JMenuItem/bug4839464.java line 220: >> >>> 218: // Check mnemonics >>> 219: int tmpInt = item.getMnemonic(); >>> 220: robo.clickMouseOn(changeMnemonicButton); >> >> @DamonGuy Util's JRobot is

Re: RFR: 8306996: Open source Swing MenuItem related tests [v2]

2023-05-10 Thread Harshitha Onkar
On Mon, 8 May 2023 09:33:40 GMT, Abhishek Kumar wrote: >> This PR is to open source JMenuItem related swing tests. > > Abhishek Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > Review comment update

Re: RFR: 8307194: Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries [v12]

2023-05-10 Thread Jiangli Zhou
On Wed, 10 May 2023 20:24:30 GMT, Erik Joelsson wrote: > This change caused all our builds but Linux to fail. Did you verify on other > platforms than Linux at all? I see you have GHA turned off. Sorry about the issue. There were failed workflows after I merged with the latest JDK master this

Re: RFR: 8306996: Open source Swing MenuItem related tests [v2]

2023-05-10 Thread Harshitha Onkar
On Mon, 8 May 2023 09:33:40 GMT, Abhishek Kumar wrote: >> This PR is to open source JMenuItem related swing tests. > > Abhishek Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > Review comment update Marked as reviewed by honkar

Re: RFR: 8307194: Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries [v12]

2023-05-10 Thread Erik Joelsson
On Wed, 10 May 2023 17:13:13 GMT, Jiangli Zhou wrote: >> This PR is branched from the makefile changes for >> https://bugs.openjdk.org/browse/JDK-8303796 and contains the following for >> handling the JDK/hotspot static libraries: >> >> - Build hotspot libjvm.a and JDK static libraries for

Re: RFR: 8306996: Open source Swing MenuItem related tests [v2]

2023-05-10 Thread Abhishek Kumar
On Mon, 8 May 2023 09:33:40 GMT, Abhishek Kumar wrote: >> This PR is to open source JMenuItem related swing tests. > > Abhishek Kumar has updated the pull request incrementally with one additional > commit since the last revision: > > Review comment update @prrace Please review.

Re: RFR: 8307779: Relax the java.awt.Robot specification [v2]

2023-05-10 Thread Phil Race
On Wed, 10 May 2023 13:04:42 GMT, Kevin Rushforth wrote: >> Updated according to your comments. >> I still left `revokeScreenCapturePermission` here for now, so that its >> documentation can be easily compared and aligned with the other >> documentation changes. >> After finalization it will

Re: RFR: 8307779: Relax the java.awt.Robot specification [v2]

2023-05-10 Thread Phil Race
On Wed, 10 May 2023 13:48:20 GMT, Alexander Zvegintsev wrote: >> src/java.desktop/share/classes/java/awt/Robot.java line 431: >> >>> 429: * to capture screen content, and the required permissions are not >>> granted, >>> 430: * then a {@code SecurityException} may be thrown, >>>

Re: RFR: 8307779: Relax the java.awt.Robot specification [v2]

2023-05-10 Thread Phil Race
On Wed, 10 May 2023 13:56:56 GMT, Alexander Zvegintsev wrote: >> We need to relax the java.awt.Robot specification according to the latest >> operating system trends. >> This should at least cover the case of Wayland, which has changed many >> familiar concepts in Linux. >> >>

Re: RFR: 8307297: Move some DnD tests to open [v5]

2023-05-10 Thread Alisen Chung
> Adding some DnD tests Alisen Chung has updated the pull request incrementally with one additional commit since the last revision: fixing timeouts - Changes: - all: https://git.openjdk.org/jdk/pull/13756/files - new:

Re: RFR: JDK-8294427 - Check boxes and radio buttons have rendering issues on Windows in High DPI env [v2]

2023-05-10 Thread Sergey Bylokhov
On Tue, 9 May 2023 18:56:49 GMT, Rajat Mahajan wrote: >So, I added this code specifically to have improved rendering for the 25% >cases, like 125%, 225%, 325% as rendering was a little off f But why it looks off if the clipRound is used? Does that size scales worse or does it cut the image at

Re: RFR: 8307299: Move more DnD tests to open [v4]

2023-05-10 Thread Sergey Bylokhov
On Tue, 9 May 2023 17:54:41 GMT, Alisen Chung wrote: >> Added DnD tests > > Alisen Chung has updated the pull request incrementally with one additional > commit since the last revision: > > fix space, add method on EDT test/jdk/java/awt/dnd/DropTargetingTest.java line 256: > 254: } >

Re: RFR: JDK-8307083: Open source some drag and drop tests 3 [v3]

2023-05-10 Thread Sergey Bylokhov
On Tue, 9 May 2023 18:05:39 GMT, Alisen Chung wrote: >> moving tests to open > > Alisen Chung has updated the pull request incrementally with two additional > commits since the last revision: > > - Merge branch 'JTREG_TEST_SPRINT3' of github.com:alisenchung/jdk into > JTREG_TEST_SPRINT3 > -

Re: RFR: 8306812: Open source several AWT Miscellaneous tests [v2]

2023-05-10 Thread Damon Nguyen
On Mon, 8 May 2023 18:05:59 GMT, Tejesh R wrote: >> Open source few AWT Miscellaneous ( Panel, Popup, robot and scrollbar) tests. > > Tejesh R has updated the pull request incrementally with one additional > commit since the last revision: > > Updated based on review comments Changes

Integrated: 8307194: Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries

2023-05-10 Thread Jiangli Zhou
On Wed, 3 May 2023 02:09:22 GMT, Jiangli Zhou wrote: > This PR is branched from the makefile changes for > https://bugs.openjdk.org/browse/JDK-8303796 and contains the following for > handling the JDK/hotspot static libraries: > > - Build hotspot libjvm.a and JDK static libraries for >

Re: RFR: 8307194: Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries [v12]

2023-05-10 Thread Jiangli Zhou
> This PR is branched from the makefile changes for > https://bugs.openjdk.org/browse/JDK-8303796 and contains the following for > handling the JDK/hotspot static libraries: > > - Build hotspot libjvm.a and JDK static libraries for > static-libs-image/static-libs-bundles targets; This change

Re: RFR: 8307194: Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries [v11]

2023-05-10 Thread Jiangli Zhou
> This PR is branched from the makefile changes for > https://bugs.openjdk.org/browse/JDK-8303796 and contains the following for > handling the JDK/hotspot static libraries: > > - Build hotspot libjvm.a and JDK static libraries for > static-libs-image/static-libs-bundles targets; This change

Re: RFR: 8307194: Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries

2023-05-10 Thread Jiangli Zhou
On Wed, 3 May 2023 13:34:12 GMT, Erik Joelsson wrote: >> This PR is branched from the makefile changes for >> https://bugs.openjdk.org/browse/JDK-8303796 and contains the following for >> handling the JDK/hotspot static libraries: >> >> - Introduce new make target(s) for creating image/bundle

Integrated: 8307799: Newly added java/awt/dnd/MozillaDnDTest.java has invalid jtreg `@requires` clause

2023-05-10 Thread Daniel D . Daugherty
On Wed, 10 May 2023 14:23:38 GMT, Daniel D. Daugherty wrote: > A trivial fix to adjust an `@requires` tag into the proper format. This pull request has now been integrated. Changeset: 1bca05ea Author:Daniel D. Daugherty URL:

Re: RFR: 8307799: Newly added java/awt/dnd/MozillaDnDTest.java has invalid jtreg `@requires` clause

2023-05-10 Thread Daniel D . Daugherty
On Wed, 10 May 2023 14:26:48 GMT, Roger Riggs wrote: >> A trivial fix to adjust an `@requires` tag into the proper format. > > Marked as reviewed by rriggs (Reviewer). @RogerRiggs and @azvegint - Thanks for the lightning fast reviews! - PR Comment:

Re: RFR: 8307799: Newly added java/awt/dnd/MozillaDnDTest.java has invalid jtreg `@requires` clause

2023-05-10 Thread Roger Riggs
On Wed, 10 May 2023 14:23:38 GMT, Daniel D. Daugherty wrote: > A trivial fix to adjust an `@requires` tag into the proper format. Marked as reviewed by rriggs (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/13906#pullrequestreview-1420776595

Re: RFR: 8307799: Newly added java/awt/dnd/MozillaDnDTest.java has invalid jtreg `@requires` clause

2023-05-10 Thread Alexander Zvegintsev
On Wed, 10 May 2023 14:23:38 GMT, Daniel D. Daugherty wrote: > A trivial fix to adjust an `@requires` tag into the proper format. Marked as reviewed by azvegint (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/13906#pullrequestreview-1420777471

RFR: 8307799: Newly added java/awt/dnd/MozillaDnDTest.java has invalid jtreg `@requires` clause

2023-05-10 Thread Daniel D . Daugherty
A trivial fix to adjust an `@requires` tag into the proper format. - Commit messages: - Fix a bad copy-and-paste. - 8307799: Newly added java/awt/dnd/MozillaDnDTest.java has invalid jtreg `@requires` clause Changes: https://git.openjdk.org/jdk/pull/13906/files Webrev:

Re: RFR: 8307779: Relax the java.awt.Robot specification [v2]

2023-05-10 Thread Alexander Zvegintsev
On Wed, 10 May 2023 04:40:31 GMT, Prasanta Sadhukhan wrote: >> Alexander Zvegintsev has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - rephrase wording about mouse pointer may not visually move >> - Remove mention of mouse cursor for

Re: RFR: 8307779: Relax the java.awt.Robot specification [v2]

2023-05-10 Thread Alexander Zvegintsev
On Wed, 10 May 2023 12:54:37 GMT, Kevin Rushforth wrote: >> Alexander Zvegintsev has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - rephrase wording about mouse pointer may not visually move >> - Remove mention of mouse cursor for

Re: RFR: 8307779: Relax the java.awt.Robot specification [v2]

2023-05-10 Thread Alexander Zvegintsev
> We need to relax the java.awt.Robot specification according to the latest > operating system trends. > This should at least cover the case of Wayland, which has changed many > familiar concepts in Linux. > > https://bugs.openjdk.org/browse/JDK-8280982 [Wayland] [XWayland] > java.awt.Robot

Re: RFR: 8307779: Relax the java.awt.Robot specification

2023-05-10 Thread Kevin Rushforth
On Tue, 9 May 2023 20:14:56 GMT, Alexander Zvegintsev wrote: >>> I do not understand why we need this method. >> >> We need it in order to be able to revoke the saved permission. >> >> For example, it may be necessary if the user has two monitors, but the user >> only allowed access to one

Re: RFR: 8307779: Relax the java.awt.Robot specification

2023-05-10 Thread Kevin Rushforth
On Thu, 4 May 2023 16:26:47 GMT, Alexander Zvegintsev wrote: > We need to relax the java.awt.Robot specification according to the latest > operating system trends. > This should at least cover the case of Wayland, which has changed many > familiar concepts in Linux. > >

Re: RFR: 8307194: Add make target for optionally building a complete set of all JDK and hotspot libjvm static libraries [v10]

2023-05-10 Thread Erik Joelsson
On Tue, 9 May 2023 23:06:23 GMT, Jiangli Zhou wrote: >> This PR is branched from the makefile changes for >> https://bugs.openjdk.org/browse/JDK-8303796 and contains the following for >> handling the JDK/hotspot static libraries: >> >> - Introduce new make target(s) for creating image/bundle