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.