Re: [PATCH v2 0/6] MR550: comctl32/listview: Fix LVS_OWNERDATA list view multi select.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/listview.c:
> +    res = SendMessageA(hwnd, WM_KEYDOWN, VK_DOWN, 0);
> +    expect(0, res);
> +    ok_sequence(sequences, PARENT_ODSTATECHANGED_SEQ_INDEX,
> +                ownerdata_multiselect_select_3_odstatechanged_seq,
> +                "ownerdata multiselect: deselect all, select item 3 via DOWN", FALSE);
> +    res = SendMessageA(hwnd, WM_KEYUP, VK_DOWN, 0);
> +    expect(0, res);
> +    res = SendMessageA(hwnd, LVM_GETSELECTEDCOUNT, 0, 0);
> +    expect(1, res);
> +
> +    hold_key(VK_SHIFT);
> +
> +    flush_sequences(sequences, NUM_MSG_SEQUENCES);
> +
> +    /* First up then down */
> +    /* Select multiple items via SHIFT+UP */
> For consistency, let's move hold_key() to the start of each test. Right now some of them are at the start and some are at the up.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/550#note_5316