user32/listbox: Fix redraw after LB_SETCOUNT message.

3ef790b9 accidentally dropped updating LBS_DISPLAYCHANGED flag that had been done in LISTBOX_InvalidateItems() if the count value was changed. In fact, the newly added test shows LB_SETCOUNT message always causes WM_DRAWITEM requests regardless of the count value. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=51591Signed-off-by: Akihiro Sagawa <sagawa.aki@gmail.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

user32/listbox: Fix redraw after LB_SETCOUNT message.
247185ba · Akihiro Sagawa · Alexandre Julliard · a70ed810 · 247185ba · 247185ba
Commit 247185ba authored Aug 12, 2021 by Akihiro Sagawa Committed by Alexandre Julliard Aug 12, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

listbox.c dlls/user32/listbox.c +2 -0

listbox.c dlls/user32/tests/listbox.c +0 -1

No files found.
--- a/dlls/user32/listbox.c
+++ b/dlls/user32/listbox.c
@@ -1839,6 +1839,8 @@ static LRESULT LISTBOX_SetCount( LB_DESCR *descr, UINT count )
    if (!resize_storage(descr, count))
        return LB_ERRSPACE;
    descr->nb_items = count;
+    if (descr->style & LBS_NOREDRAW)
+        descr->style |= LBS_DISPLAYCHANGED;

    if (count)
    {

--- a/dlls/user32/tests/listbox.c
+++ b/dlls/user32/tests/listbox.c
@@ -447,7 +447,6 @@ static void test_ownerdraw(void)
        got_drawitem = 0;
        ret = RedrawWindow(hLB, NULL, 0, RDW_UPDATENOW);
        ok(ret, "RedrawWindow failed\n");
-        todo_wine_if(testcase[i].message == LB_SETCOUNT)
        ok(got_drawitem == testcase[i].drawitem, "expected %u, got %u\n", testcase[i].drawitem, got_drawitem);

        DestroyWindow(hLB);