comctl32/edit: Don't repaint immediately when getting focus.

Fix MSXML 3.0 installer username edit control missing part of its edge. The application has a static control overlapping an edit control. Repainting the edit control immediately will cause the edit control to be painted under the static control. Signed-off-by: Zhiyi Zhang <zzhang@codeweavers.com>

comctl32/edit: Don't repaint immediately when getting focus.
a336b43a · Zhiyi Zhang · Alexandre Julliard · 2a27ffe4 · a336b43a · a336b43a
Commit a336b43a authored Aug 12, 2022 by Zhiyi Zhang Committed by Alexandre Julliard Aug 18, 2022
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

edit.c dlls/comctl32/edit.c +1 -1

edit.c dlls/comctl32/tests/edit.c +1 -1

No files found.
--- a/dlls/comctl32/edit.c
+++ b/dlls/comctl32/edit.c
@@ -3684,7 +3684,7 @@ static void EDIT_WM_NCPaint(HWND hwnd, HRGN region)
 */
 static void EDIT_WM_SetFocus(HTHEME theme, EDITSTATE *es)
 {
-    UINT flags = RDW_INVALIDATE | RDW_UPDATENOW;
+    UINT flags = RDW_INVALIDATE;

    es->flags |= EF_FOCUSED;


--- a/dlls/comctl32/tests/edit.c
+++ b/dlls/comctl32/tests/edit.c
@@ -3458,7 +3458,7 @@ static void test_change_focus(void)
    flush_sequences(sequences, NUM_MSG_SEQUENCES);
    SetFocus(hwnd);
    ok_sequence(sequences, COMBINED_SEQ_INDEX, setfocus_sent_only_combined_seq,
-                "Set focus sent only", TRUE);
+                "Set focus sent only", FALSE);

    flush_sequences(sequences, NUM_MSG_SEQUENCES);
    SetFocus(parent_wnd);