<AWT Dev> Review request for 8033936: java.awt.List events are not sent properly to handleEvent or ItemListener
Ambarish Rapte
ambarish.rapte at oracle.com
Tue Mar 22 12:15:27 UTC 2016
Hi Prasanta,
Thanks for the review.
The test class is extended from Frame, hence call to dispose works.
PerformTest is called twice for the below two scenarios,
1. To test Event received by List's parent Frame's handleEvent() -> Called testHandleEvent()
2. To test ItemEvent received by ItemListener registered by the List. -> Called testItemListener()
Thanks,
Ambarish
From: prasanta sadhukhan
Sent: Tuesday, March 22, 2016 2:11 PM
To: Ambarish Rapte; Semyon Sadetsky; awt-dev at openjdk.java.net
Subject: Re: Review request for 8033936: java.awt.List events are not sent properly to handleEvent or ItemListener
FIx looks ok to me.
But in testcase, there is no dispose() method but you are still calling dispose(). Also, why there is a need to call performTest() twice?
Regards
Prasanta
On 3/18/2016 2:04 PM, Ambarish Rapte wrote:
Hi,
Please review the fix for JDK9,
Bug: https://bugs.openjdk.java.net/browse/JDK-8033936
Webrev: http://cr.openjdk.java.net/~arapte/8033936/webrev.00/
Issue:
In a multi select list, when multiple items are selected and then deselected,
the ItemEvent received does not have correct information about the deselected item.
Cause:
In a multi select List, while deselecting an item the focus caret is not updated to recently clicked item.
But the focus caret remains at the last item clicked while selecting the items.
Fix:
Update the caret index while deselecting the items to recently clicked item.
Verification:
Verified the regression and JCK tests after the fix - there are no failures due to this change.
Verified the newly included test on Windows, Ubuntu & Mac.
Regards,
Ambarish
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20160322/a325600e/attachment.html>
More information about the awt-dev
mailing list