<AWT Dev> [11][JDK-8197810]RFR: Test java/awt/Choice/SelectCurrentItemTest/SelectCurrentItemTest.html fails on Windows

[Sergey Bylokhov]

If this was done intentionally then I suggest to revert the expectation 
of the test. But I think that the test will fail on lin/mac, because one 
more related bug JDK-8014503 was not fixed.
So we should run the test on windows only, until JDK-8014503 is not fixed.

On 26/06/2018 06:58, Krishna Addepalli wrote:
> Hi Sergey,
> 
> I think the current behavior is intentionally implemented in awt in Windows. I don't know how JDK-4902933 is marked as resolved, but there is a linked issue JDK-7171412. Here is the link to the patch that was pushed as fix for this issue, and we can see that there is an explicit suppression of event propagation in case of same index being selected again. Pasting the current code for reference:
> MsgRouting AwtChoice::WmNotify(UINT notifyCode)
> {
>      if (notifyCode == CBN_SELCHANGE) {
>          int selectedIndex = (int)SendMessage(CB_GETCURSEL);
> 
>          JNIEnv *env = (JNIEnv *)JNU_GetEnv(jvm, JNI_VERSION_1_2);
>          jobject target = GetTarget(env);
>          int previousIndex = env->GetIntField(target, selectedIndexID);
> 
>          if (selectedIndex != CB_ERR && selectedIndex != previousIndex){
>              DoCallback("handleAction", "(I)V", selectedIndex);
>          }
> 
> This is the reason why the test is failing.
> 
> Hope this clarifies.
> 
> Thanks,
> Krishna
> 
> -----Original Message-----
> From: Sergey Bylokhov
> Sent: Tuesday, May 29, 2018 8:48 PM
> To: Krishna Addepalli <krishna.addepalli at oracle.com>; awt-dev at openjdk.java.net
> Subject: Re: <AWT Dev> [11][JDK-8197810]RFR: Test java/awt/Choice/SelectCurrentItemTest/SelectCurrentItemTest.html fails on Windows
> 
> Hi, Krishna.
> On 08/05/2018 02:59, Krishna Addepalli wrote:
>> The basic problem is that, the Robot mouse move is moving to position
>> where item 0 is located (which is already selected), and selecting it.
>> Since this item is already selected, there is no new item selection
>> event generated, which is why the test fails.
> As far as I understand the usecase which your describe was implemented intentionally in this test to verify the bug:
> https://bugs.openjdk.java.net/browse/JDK-4902933
> 
> Did you check what is the reason of behavior change?
> 
> 
> --
> Best regards, Sergey.
> 


-- 
Best regards, Sergey.