<AWT Dev> Review request for 8033936: java.awt.List events are not sent properly to handleEvent or ItemListener

[Sergey Bylokhov]

Looks fine to me.

On 25.03.16 12:50, Ambarish Rapte wrote:
> Hi Sergey,
> 	
> 	I could verify that this is regression of JDK-6806217.
> 	Issue does NOT occur with JDK6 & occurs with JDK7,
> 	Updated information to the bug.
>
> 	Verified with JDK versions,
>
> 	JDK6:	java version "1.6.0_115"
> 		Java(TM) SE Runtime Environment (build 1.6.0_115-b07)
> 		Java HotSpot(TM) 64-Bit Server VM (build 20.115-b07, mixed mode)
>
> 	JDK7:	java version "1.7.0_01"
> 		Java(TM) SE Runtime Environment (build 1.7.0_01-b04)
> 		Java HotSpot(TM) 64-Bit Server VM (build 21.1-b02, mixed mode)
>
>
> Thanks,
> Ambarish
> 	
>
> -----Original Message-----
> From: Sergey Bylokhov
> Sent: Tuesday, March 22, 2016 5:49 PM
> To: Ambarish Rapte; Prasanta Sadhukhan; Semyon Sadetsky; awt-dev at openjdk.java.net
> Subject: Re: <AWT Dev> Review request for 8033936: java.awt.List events are not sent properly to handleEvent or ItemListener
>
> Hi, Ambarish.
> Can you confirm this is regression of JDK-6806217 or not? add  this information to the CR.
>
> On 18.03.16 11:34, 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
>>
>
>
> --
> Best regards, Sergey.
>


-- 
Best regards, Sergey.