Hi Sergey, The test case was leading my reasoning astray! Yes you are right that we post event even for the same item selected again. I have updated the fix and the test case well, which checks if an ItemEvent is generated when a new item is selected and then it is not generated when the same item is selected again. Here is the updated webrev: http://cr.openjdk.java.net/~kaddepalli/8215921/webrev02 <http://cr.openjdk.java.net/~kaddepalli/8215921/webrev02>
Thanks, Krishna > On 17-Jan-2019, at 1:30 AM, Sergey Bylokhov <sergey.bylok...@oracle.com> > wrote: > > > On 16/01/2019 09:59, Krishna Addepalli wrote: >> Hi Sergey, >> I checked the code, by putting logs and can confirm that an event is not >> generated even if the same is reselected. >> I have updated the test case to reflect this fact, so here is the new >> webrev: http://cr.openjdk.java.net/~kaddepalli/8215921/webrev01/ > > But why it is not generated? How (!firstPress || wasDragged ) prevents it to > be posted? > From the code it looks like when "wasDragged==true" or "firstPress==false" we > will post an event. > > > -- > Best regards, Sergey.