[reactos] 01/01: [BROWSEUI] Fix pos of taskbar toolbar right-click menu (#5421)

[Katayama Hirofumi MZ]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=b4e946f5b0147480d0df34...

commit b4e946f5b0147480d0df34654b55f9a7705787ff
Author:     Katayama Hirofumi MZ [e-mail]
AuthorDate: Tue Jul 11 05:28:50 2023 +0900
Commit:     GitHub [e-mail]
CommitDate: Tue Jul 11 05:28:50 2023 +0900

[BROWSEUI] Fix pos of taskbar toolbar right-click menu (#5421)

Don't call ClientToScreen in handling NM_RCLICK message because NMMOUSE.pt is already in screen coordinates.
    CORE-18966
---
 dll/win32/browseui/shellbars/CISFBand.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/dll/win32/browseui/shellbars/CISFBand.cpp b/dll/win32/browseui/shellbars/CISFBand.cpp
index 2d8c7d4500b..c0266aeca9c 100644
--- a/dll/win32/browseui/shellbars/CISFBand.cpp
+++ b/dll/win32/browseui/shellbars/CISFBand.cpp
@@ -353,7 +353,7 @@ HRESULT CISFBand::CreateSimpleToolbar(HWND hWndParent)
                     case NM_RCLICK:
                     {
                         HRESULT hr;
-                        POINT pt = ((LPNMMOUSE)lParam)->pt;
+                        POINT pt = ((LPNMMOUSE)lParam)->pt; // Already in screen coordinates
                         CComPtr<IContextMenu> picm;
                         HMENU fmenu = CreatePopupMenu();
                         TBBUTTON tb;
@@ -363,7 +363,6 @@ HRESULT CISFBand::CreateSimpleToolbar(HWND hWndParent)

if (chk)
                         {
-                            ClientToScreen(&pt);
                             hr = m_pISF->GetUIObjectOf(m_hWnd, 1, &pidl, IID_NULL_PPV_ARG(IContextMenu, &picm));
                             if (FAILED_UNEXPECTEDLY(hr))
                                 return hr;