[v4 PATCH 4/4] comctl32/listbox: Clear LBS_OWNERDRAWVARIABLE if LBS_OWNERDRAWFIXED is set.

Nikolay Sivov nsivov at codeweavers.com
Sat Nov 10 15:29:00 CST 2018


From: Gabriel Ivăncescu <gabrielopcode at gmail.com>

Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---
 dlls/comctl32/listbox.c       | 2 ++
 dlls/comctl32/tests/listbox.c | 3 ++-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/dlls/comctl32/listbox.c b/dlls/comctl32/listbox.c
index 2137ef86e0..d0e8e60052 100644
--- a/dlls/comctl32/listbox.c
+++ b/dlls/comctl32/listbox.c
@@ -2520,6 +2520,8 @@ static BOOL LISTBOX_Create( HWND hwnd, LPHEADCOMBO lphc )
 
     if (descr->style & LBS_OWNERDRAWFIXED)
     {
+        descr->style &= ~LBS_OWNERDRAWVARIABLE;
+
 	if( descr->lphc && (descr->lphc->dwStyle & CBS_DROPDOWN))
 	{
 	    /* WinWord gets VERY unhappy if we send WM_MEASUREITEM from here */
diff --git a/dlls/comctl32/tests/listbox.c b/dlls/comctl32/tests/listbox.c
index 5fe4becd7f..1a54bb49b1 100644
--- a/dlls/comctl32/tests/listbox.c
+++ b/dlls/comctl32/tests/listbox.c
@@ -435,6 +435,8 @@ static void test_ownerdraw(void)
         NULL, NULL, NULL, 0);
     ok(hLB != NULL, "last error 0x%08x\n", GetLastError());
 
+    ok(GetWindowLongA(hLB, GWL_STYLE) & LBS_OWNERDRAWVARIABLE, "Unexpected window style.\n");
+
     ret = SendMessageA(hLB, LB_INSERTSTRING, -1, 0);
     ok(ret == 0, "Unexpected return value %d.\n", ret);
     ret = SendMessageA(hLB, LB_INSERTSTRING, -1, 0);
@@ -450,7 +452,6 @@ static void test_ownerdraw(void)
     ok(ret == LB_OKAY, "Failed to set item height, %d.\n", ret);
 
     ret = SendMessageA(hLB, LB_GETITEMHEIGHT, 0, 0);
-todo_wine
     ok(ret == 42, "Unexpected item height %d.\n", ret);
 
     ret = SendMessageA(hLB, LB_GETITEMHEIGHT, 1, 0);
-- 
2.19.1




More information about the wine-devel mailing list