comctl32: Set listview focus in WM_LBUTTONDOWN.

This patch reverts commit 098dc7b3 and moves focus setting after LVN_ITEMCHANGED notification is sent. Writing tests for it is tricky because native blocks inside WM_LBUTTONDOWN message. Signed-off-by: Piotr Caban <piotr@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

comctl32: Set listview focus in WM_LBUTTONDOWN.
ddcab8ac · Piotr Caban · Alexandre Julliard · a4fb95d7 · ddcab8ac
Commit ddcab8ac authored Dec 01, 2015 by Piotr Caban Committed by Alexandre Julliard Dec 02, 2015
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

listview.c dlls/comctl32/listview.c +3 -3

No files found.
--- a/dlls/comctl32/listview.c
+++ b/dlls/comctl32/listview.c
@@ -10266,6 +10266,9 @@ static LRESULT LISTVIEW_LButtonDown(LISTVIEW_INFO *infoPtr, WORD wKey, INT x, IN
      }
    }

+    if (!infoPtr->bFocus)
+        SetFocus(infoPtr->hwndSelf);
+
    if (infoPtr->dwLvExStyle & LVS_EX_ONECLICKACTIVATE)
        if(lvHitTestInfo.iItem != -1) notify_itemactivate(infoPtr,&lvHitTestInfo);
  }
@@ -10352,9 +10355,6 @@ static LRESULT LISTVIEW_LButtonUp(LISTVIEW_INFO *infoPtr, WORD wKey, INT x, INT 
            LISTVIEW_DelayedEditItem);
    }

-    if (!infoPtr->bFocus)
-        SetFocus(infoPtr->hwndSelf);
-
    return 0;
 }