Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v9]

2025-08-07 Thread Abhishek Kumar
On Wed, 6 Aug 2025 09:43:48 GMT, Ravi Gupta wrote: >> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java >> left debris on system whenever fails its required frame.dispose() in finally >> block. >> >> >> finally { >> EventQueue.invokeAndWait(ExtraButtonDrag:

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v9]

2025-08-06 Thread Ravi Gupta
> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java left > debris on system whenever fails its required frame.dispose() in finally block. > > > finally { > EventQueue.invokeAndWait(ExtraButtonDrag::disposeFrame); > } > public static void disposeFra

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v8]

2025-08-05 Thread Abhishek Kumar
On Tue, 5 Aug 2025 11:01:24 GMT, Ravi Gupta wrote: >> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java >> left debris on system whenever fails its required frame.dispose() in finally >> block. >> >> >> finally { >> EventQueue.invokeAndWait(ExtraButtonDrag:

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v8]

2025-08-05 Thread Ravi Gupta
> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java left > debris on system whenever fails its required frame.dispose() in finally block. > > > finally { > EventQueue.invokeAndWait(ExtraButtonDrag::disposeFrame); > } > public static void disposeFra

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v7]

2025-08-04 Thread Ravi Gupta
> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java left > debris on system whenever fails its required frame.dispose() in finally block. > > > finally { > EventQueue.invokeAndWait(ExtraButtonDrag::disposeFrame); > } > public static void disposeFra

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v6]

2025-07-31 Thread Phil Race
On Thu, 31 Jul 2025 05:37:59 GMT, Ravi Gupta wrote: >> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java >> left debris on system whenever fails its required frame.dispose() in finally >> block. >> >> >> finally { >> EventQueue.invokeAndWait(ExtraButtonDrag

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v6]

2025-07-31 Thread Abhishek Kumar
On Thu, 31 Jul 2025 05:37:59 GMT, Ravi Gupta wrote: >> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java >> left debris on system whenever fails its required frame.dispose() in finally >> block. >> >> >> finally { >> EventQueue.invokeAndWait(ExtraButtonDrag

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v6]

2025-07-30 Thread Ravi Gupta
> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java left > debris on system whenever fails its required frame.dispose() in finally block. > > > finally { > EventQueue.invokeAndWait(ExtraButtonDrag::disposeFrame); > } > public static void disposeFra

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v5]

2025-07-30 Thread Abhishek Kumar
On Mon, 21 Jul 2025 07:39:59 GMT, Ravi Gupta wrote: >> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java >> left debris on system whenever fails its required frame.dispose() in finally >> block. >> >> >> finally { >> EventQueue.invokeAndWait(ExtraButtonDrag

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v5]

2025-07-29 Thread Abhishek Kumar
On Mon, 21 Jul 2025 07:39:59 GMT, Ravi Gupta wrote: >> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java >> left debris on system whenever fails its required frame.dispose() in finally >> block. >> >> >> finally { >> EventQueue.invokeAndWait(ExtraButtonDrag

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v2]

2025-07-21 Thread Ravi Gupta
On Mon, 7 Jul 2025 06:31:35 GMT, Abhishek Kumar wrote: >> Ravi Gupta has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8361067: EventQueue.invokeAndWait used > > Fix to dispose the frame looks good but the test needs re-factoring like > e

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v5]

2025-07-21 Thread Ravi Gupta
> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java left > debris on system whenever fails its required frame.dispose() in finally block. > > > finally { > EventQueue.invokeAndWait(ExtraButtonDrag::disposeFrame); > } > public static void disposeFra

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v4]

2025-07-21 Thread Ravi Gupta
> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java left > debris on system whenever fails its required frame.dispose() in finally block. > > > finally { > EventQueue.invokeAndWait(ExtraButtonDrag::disposeFrame); > } > public static void disposeFra

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v3]

2025-07-10 Thread Abhishek Kumar
On Thu, 10 Jul 2025 11:49:26 GMT, Ravi Gupta wrote: >> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java >> left debris on system whenever fails its required frame.dispose() in finally >> block. >> >> >> finally { >> EventQueue.invokeAndWait(ExtraButtonDrag

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v3]

2025-07-10 Thread Ravi Gupta
> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java left > debris on system whenever fails its required frame.dispose() in finally block. > > > finally { > EventQueue.invokeAndWait(ExtraButtonDrag::disposeFrame); > } > public static void disposeFra

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v2]

2025-07-06 Thread Prasanta Sadhukhan
On Mon, 7 Jul 2025 06:26:29 GMT, Prasanta Sadhukhan wrote: >> Ravi Gupta has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8361067: EventQueue.invokeAndWait used > > test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java line

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v2]

2025-07-06 Thread Prasanta Sadhukhan
On Mon, 7 Jul 2025 06:29:17 GMT, Prasanta Sadhukhan wrote: >> test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java line 55: >> >>> 53: >>> 54: public static void main(String []s) throws >>> InvocationTargetException, InterruptedException, AWTException{ >>> 55: frame

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v2]

2025-07-06 Thread Abhishek Kumar
On Tue, 1 Jul 2025 10:12:29 GMT, Ravi Gupta wrote: >> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java >> left debris on system whenever fails its required frame.dispose() in finally >> block. >> >> >> finally { >> EventQueue.invokeAndWait(ExtraButtonDrag:

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v2]

2025-07-06 Thread Prasanta Sadhukhan
On Tue, 1 Jul 2025 10:12:29 GMT, Ravi Gupta wrote: >> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java >> left debris on system whenever fails its required frame.dispose() in finally >> block. >> >> >> finally { >> EventQueue.invokeAndWait(ExtraButtonDrag:

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v2]

2025-07-01 Thread Ravi Gupta
On Tue, 1 Jul 2025 10:12:29 GMT, Ravi Gupta wrote: >> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java >> left debris on system whenever fails its required frame.dispose() in finally >> block. >> >> >> finally { >> EventQueue.invokeAndWait(ExtraButtonDrag:

Re: RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block [v2]

2025-07-01 Thread Ravi Gupta
> Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java left > debris on system whenever fails its required frame.dispose() in finally block. > > > finally { > EventQueue.invokeAndWait(ExtraButtonDrag::disposeFrame); > } > public static void disposeFra

RFR: 8361067: Test ExtraButtonDrag.java requires frame.dispose in finally block

2025-06-30 Thread Ravi Gupta
Test test/jdk/java/awt/Mouse/MouseModifiersUnitTest/ExtraButtonDrag.java left debris on system whenever fails its required frame.dispose() in finally block. +finally { +if (frame != null) { +frame.dispose(); +frame = null; +} +