Waritnan Sookbuntherng : comctl32/status: Checking size grip bounds in x-axis only.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 0f12fa1e432360431392ece417ae9ad02cb00647
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=0f12fa1e432360431392ece41...

Author: Waritnan Sookbuntherng <lion328(a)hotmail.co.th>
Date:   Mon Dec 10 09:52:10 2018 +0000

comctl32/status: Checking size grip bounds in x-axis only.

Signed-off-by: Waritnan Sookbuntherng <lion328(a)hotmail.co.th>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 dlls/comctl32/status.c       |  5 +----
 dlls/comctl32/tests/status.c | 20 ++++++++++----------
 2 files changed, 11 insertions(+), 14 deletions(-)

diff --git a/dlls/comctl32/status.c b/dlls/comctl32/status.c
index 3bf72fa..6ea7d9e 100644
--- a/dlls/comctl32/status.c
+++ b/dlls/comctl32/status.c
@@ -1009,10 +1009,7 @@ STATUSBAR_WMNCHitTest (const STATUS_INFO *infoPtr, INT x, INT y)
 	pt.y = y;
 	ScreenToClient (infoPtr->Self, &pt);
 
-	rect.left = rect.right - 13;
-	rect.top += 2;
-
-	if (PtInRect (&rect, pt))
+	if (pt.x >= rect.right - GetSystemMetrics(SM_CXVSCROLL))
         {
             if (GetWindowLongW( infoPtr->Self, GWL_EXSTYLE ) & WS_EX_LAYOUTRTL) return HTBOTTOMLEFT;
 	    else return HTBOTTOMRIGHT;
diff --git a/dlls/comctl32/tests/status.c b/dlls/comctl32/tests/status.c
index b7b5fb2..c7d9989 100644
--- a/dlls/comctl32/tests/status.c
+++ b/dlls/comctl32/tests/status.c
@@ -615,17 +615,17 @@ static void test_sizegrip(void)
 
     /* check bounds when not maximized */
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.left, rc.top));
-    todo_wine expect(HTBOTTOMRIGHT, r);
+    expect(HTBOTTOMRIGHT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.left - 1, rc.top));
     expect(HTCLIENT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.left, rc.top - 1));
-    todo_wine expect(HTBOTTOMRIGHT, r);
+    expect(HTBOTTOMRIGHT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.right, rc.bottom));
-    todo_wine expect(HTBOTTOMRIGHT, r);
+    expect(HTBOTTOMRIGHT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.right + 1, rc.bottom));
-    todo_wine expect(HTBOTTOMRIGHT, r);
+    expect(HTBOTTOMRIGHT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.right, rc.bottom + 1));
-    todo_wine expect(HTBOTTOMRIGHT, r);
+    expect(HTBOTTOMRIGHT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.right - 1, rc.bottom - 1));
     expect(HTBOTTOMRIGHT, r);
 
@@ -638,17 +638,17 @@ static void test_sizegrip(void)
     rc.right = pt.x;
 
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.left, rc.top));
-    todo_wine expect(HTBOTTOMLEFT, r);
+    expect(HTBOTTOMLEFT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.left + 1, rc.top));
     expect(HTCLIENT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.left, rc.top - 1));
-    todo_wine expect(HTBOTTOMLEFT, r);
+    expect(HTBOTTOMLEFT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.right, rc.bottom));
-    todo_wine expect(HTBOTTOMLEFT, r);
+    expect(HTBOTTOMLEFT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.right - 1, rc.bottom));
-    todo_wine expect(HTBOTTOMLEFT, r);
+    expect(HTBOTTOMLEFT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.right, rc.bottom + 1));
-    todo_wine expect(HTBOTTOMLEFT, r);
+    expect(HTBOTTOMLEFT, r);
     r = SendMessageA(hwndStatus, WM_NCHITTEST, 0, MAKELPARAM(rc.right + 1, rc.bottom - 1));
     expect(HTBOTTOMLEFT, r);