commit 1a114c9f4bab60463189e3ecb89766d4d2017d53
author Piotr Caban <piotr@codeweavers.com> Wed Oct 04 12:38:47 2017 +0200
committer Alexandre Julliard <julliard@winehq.org> Fri Oct 06 21:48:48 2017 +0200
tree 9eafc28a0aff940c49fd09111dfe4b0e2611db2f
parent 9cb29e5182622f7db8316f62fe82e114df73d570

user32: Don't invalidate ComboBox on CB_SETCURSEL message.

Signed-off-by: Piotr Caban <piotr@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

dlls/user32/combo.c

diff --git a/dlls/user32/combo.c b/dlls/user32/combo.c
index ee9ce0a..149a495 100644
--- a/dlls/user32/combo.c
+++ b/dlls/user32/combo.c
@@ -2142,10 +2142,7 @@
 	            SendMessageW(lphc->hWndLBox, LB_SETTOPINDEX, wParam, 0);
 
 		/* no LBN_SELCHANGE in this case, update manually */
-		if( lphc->wState & CBF_EDIT )
-		    CBUpdateEdit( lphc, (INT)wParam );
-		else
-		    InvalidateRect(lphc->self, &lphc->textRect, TRUE);
+                CBPaintText( lphc, NULL );
 		lphc->wState &= ~CBF_SELCHANGE;
 	        return  lParam;
 	case CB_GETLBTEXT:

dlls/user32/tests/msg.c

diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c
index 1d34fd2..1b4f6fa 100644
--- a/dlls/user32/tests/msg.c
+++ b/dlls/user32/tests/msg.c
@@ -6937,7 +6937,7 @@
     log_all_parent_messages++;
     SendMessageA(combo, CB_SETCURSEL, 0, 0);
     log_all_parent_messages--;
-    ok_sequence(SetCurSelComboSeq, "CB_SETCURSEL on a ComboBox", TRUE);
+    ok_sequence(SetCurSelComboSeq, "CB_SETCURSEL on a ComboBox", FALSE);
 
     DestroyWindow(combo);
     DestroyWindow(parent);