From 9ec0be3dabf01ccd2cda2f088ed5bede396f2c56 Mon Sep 17 00:00:00 2001 From: Yuhang Zhao <zhaoyuhang@rankyee.com> Date: 周一, 04 12月 2023 14:59:53 +0800 Subject: [PATCH] wip --- src/core/contexts/win32windowcontext.cpp | 472 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++-- 1 files changed, 452 insertions(+), 20 deletions(-) diff --git a/src/core/contexts/win32windowcontext.cpp b/src/core/contexts/win32windowcontext.cpp index a901cae..7c50d84 100644 --- a/src/core/contexts/win32windowcontext.cpp +++ b/src/core/contexts/win32windowcontext.cpp @@ -5,19 +5,225 @@ #include <QtCore/QHash> #include <QtCore/QAbstractNativeEventFilter> #include <QtCore/QCoreApplication> +#include <QtCore/QOperatingSystemVersion> +#include <QtCore/private/qsystemlibrary_p.h> #include <QtGui/private/qhighdpiscaling_p.h> #include "qwkcoreglobal_p.h" +#include <shellscalingapi.h> +#include <dwmapi.h> + namespace QWK { + + static constexpr const auto kAutoHideTaskBarThickness = quint8{ 2 }; // The thickness of an auto-hide taskbar in pixels. using WndProcHash = QHash<HWND, Win32WindowContext *>; // hWnd -> context Q_GLOBAL_STATIC(WndProcHash, g_wndProcHash); static WNDPROC g_qtWindowProc = nullptr; // Original Qt window proc function + struct DynamicApis { + decltype(&::DwmFlush) pDwmFlush = nullptr; + decltype(&::GetDpiForWindow) pGetDpiForWindow = nullptr; + decltype(&::GetSystemMetricsForDpi) pGetSystemMetricsForDpi = nullptr; + decltype(&::GetDpiForMonitor) pGetDpiForMonitor = nullptr; + + DynamicApis() { + QSystemLibrary user32(QStringLiteral("user32.dll")); + pGetDpiForWindow = reinterpret_cast<decltype(pGetDpiForWindow)>(user32.resolve("GetDpiForWindow")); + pGetSystemMetricsForDpi = reinterpret_cast<decltype(pGetSystemMetricsForDpi)>(user32.resolve("GetSystemMetricsForDpi")); + + QSystemLibrary shcore(QStringLiteral("shcore.dll")); + pGetDpiForMonitor = reinterpret_cast<decltype(pGetDpiForMonitor)>(shcore.resolve("GetDpiForMonitor")); + + QSystemLibrary dwmapi(QStringLiteral("dwmapi.dll")); + pDwmFlush = reinterpret_cast<decltype(pDwmFlush)>(dwmapi.resolve("DwmFlush")); + } + + ~DynamicApis() = default; + + static const DynamicApis &instance() { + static const DynamicApis inst{}; + return inst; + } + + private: + Q_DISABLE_COPY_MOVE(DynamicApis) + }; + + static inline constexpr bool operator==(const POINT &lhs, const POINT &rhs) noexcept + { + return ((lhs.x == rhs.x) && (lhs.y == rhs.y)); + } + + static inline constexpr bool operator!=(const POINT &lhs, const POINT &rhs) noexcept + { + return !operator==(lhs, rhs); + } + + static inline constexpr bool operator==(const SIZE &lhs, const SIZE &rhs) noexcept + { + return ((lhs.cx == rhs.cx) && (lhs.cy == rhs.cy)); + } + + static inline constexpr bool operator!=(const SIZE &lhs, const SIZE &rhs) noexcept + { + return !operator==(lhs, rhs); + } + + static inline constexpr bool operator>(const SIZE &lhs, const SIZE &rhs) noexcept + { + return ((lhs.cx * lhs.cy) > (rhs.cx * rhs.cy)); + } + + static inline constexpr bool operator>=(const SIZE &lhs, const SIZE &rhs) noexcept + { + return (operator>(lhs, rhs) || operator==(lhs, rhs)); + } + + static inline constexpr bool operator<(const SIZE &lhs, const SIZE &rhs) noexcept + { + return (operator!=(lhs, rhs) && !operator>(lhs, rhs)); + } + + static inline constexpr bool operator<=(const SIZE &lhs, const SIZE &rhs) noexcept + { + return (operator<(lhs, rhs) || operator==(lhs, rhs)); + } + + static inline constexpr bool operator==(const RECT &lhs, const RECT &rhs) noexcept + { + return ((lhs.left == rhs.left) && (lhs.top == rhs.top) && (lhs.right == rhs.right) && (lhs.bottom == rhs.bottom)); + } + + static inline constexpr bool operator!=(const RECT &lhs, const RECT &rhs) noexcept + { + return !operator==(lhs, rhs); + } + + static inline constexpr QPoint point2qpoint(const POINT &point) + { + return QPoint{ int(point.x), int(point.y) }; + } + + static inline constexpr POINT qpoint2point(const QPoint &point) + { + return POINT{ LONG(point.x()), LONG(point.y()) }; + } + + static inline constexpr QSize size2qsize(const SIZE &size) + { + return QSize{ int(size.cx), int(size.cy) }; + } + + static inline constexpr SIZE qsize2size(const QSize &size) + { + return SIZE{ LONG(size.width()), LONG(size.height()) }; + } + + static inline constexpr QRect rect2qrect(const RECT &rect) + { + return QRect{ QPoint{ int(rect.left), int(rect.top) }, QSize{ int(RECT_WIDTH(rect)), int(RECT_HEIGHT(rect)) } }; + } + + static inline constexpr RECT qrect2rect(const QRect &qrect) + { + return RECT{ LONG(qrect.left()), LONG(qrect.top()), LONG(qrect.right()), LONG(qrect.bottom()) }; + } + + static inline /*constexpr*/ QString hwnd2str(const WId windowId) + { + // NULL handle is allowed here. + return QLatin1String("0x") + QString::number(windowId, 16).toUpper().rightJustified(8, u'0'); + } + + static inline /*constexpr*/ QString hwnd2str(const HWND hwnd) + { + // NULL handle is allowed here. + return hwnd2str(reinterpret_cast<WId>(hwnd)); + } + + static inline bool isWin8Point1OrGreater() { + static const bool result = QOperatingSystemVersion::current() >= QOperatingSystemVersion::Windows8_1; + return result; + } + + static inline bool isWin10OrGreater() { + static const bool result = QOperatingSystemVersion::current() >= QOperatingSystemVersion::Windows10; + return result; + } + + static inline quint32 getDpiForWindow(const HWND hwnd) { + Q_ASSERT(hwnd); + if (!hwnd) { + return USER_DEFAULT_SCREEN_DPI; + } + const DynamicApis &apis = DynamicApis::instance(); + if (apis.pGetDpiForWindow) { // Win10 + return apis.pGetDpiForWindow(hwnd); + } else if (apis.pGetDpiForMonitor) { // Win8.1 + const HMONITOR monitor = ::MonitorFromWindow(hwnd, MONITOR_DEFAULTTONEAREST); + UINT dpiX{ USER_DEFAULT_SCREEN_DPI }; + UINT dpiY{ USER_DEFAULT_SCREEN_DPI }; + apis.pGetDpiForMonitor(monitor, MDT_EFFECTIVE_DPI, &dpiX, &dpiY); + return dpiX; + } else { // Win2K + const HDC hdc = ::GetDC(nullptr); + const int dpiX = ::GetDeviceCaps(hdc, LOGPIXELSX); + const int dpiY = ::GetDeviceCaps(hdc, LOGPIXELSY); + ::ReleaseDC(nullptr, hdc); + return quint32(dpiX); + } + } + + static inline quint32 getResizeBorderThickness(const HWND hwnd) { + Q_ASSERT(hwnd); + if (!hwnd) { + return 0; + } + const DynamicApis &apis = DynamicApis::instance(); + if (apis.pGetSystemMetricsForDpi) { + const quint32 dpi = getDpiForWindow(hwnd); + return apis.pGetSystemMetricsForDpi(SM_CXSIZEFRAME, dpi) + apis.pGetSystemMetricsForDpi(SM_CXPADDEDBORDER, dpi); + } else { + return ::GetSystemMetrics(SM_CXSIZEFRAME) + ::GetSystemMetrics(SM_CXPADDEDBORDER); + } + } + + static inline std::optional<MONITORINFOEXW> getMonitorForWindow(const HWND hwnd) + { + Q_ASSERT(hwnd); + if (!hwnd) { + return std::nullopt; + } + // Use "MONITOR_DEFAULTTONEAREST" here so that we can still get the correct + // monitor even if the window is minimized. + const HMONITOR monitor = ::MonitorFromWindow(hwnd, MONITOR_DEFAULTTONEAREST); + MONITORINFOEXW monitorInfo{}; + monitorInfo.cbSize = sizeof(monitorInfo); + ::GetMonitorInfoW(monitor, &monitorInfo); + return monitorInfo; + }; + + static inline bool isFullScreen(const HWND hwnd) { + Q_ASSERT(hwnd); + if (!hwnd) { + return false; + } + RECT windowRect{}; + ::GetWindowRect(hwnd, &windowRect); + const std::optional<MONITORINFOEXW> mi = getMonitorForWindow(hwnd); + // Compare to the full area of the screen, not the work area. + return (windowRect == mi.value_or(MONITORINFOEXW{}).rcMonitor); + } + static inline QPoint fromNativeLocalPosition(const QWindow *window, const QPoint &point) { + Q_ASSERT(window); + if (!window) { + return point; + } #if 1 return QHighDpi::fromNativeLocalPosition(point, window); #else @@ -60,11 +266,11 @@ return false; } const LONG_PTR styles = ::GetWindowLongPtrW(hwnd, GWL_STYLE); - if ((styles == 0) || (styles & WS_DISABLED)) { + if (styles & WS_DISABLED) { return false; } const LONG_PTR exStyles = ::GetWindowLongPtrW(hwnd, GWL_EXSTYLE); - if ((exStyles == 0) || (exStyles & WS_EX_TOOLWINDOW)) { + if (exStyles & WS_EX_TOOLWINDOW) { return false; } RECT rect = {0, 0, 0, 0}; @@ -98,6 +304,11 @@ public: bool nativeEventFilter(const QByteArray &eventType, void *message, QT_NATIVE_EVENT_RESULT_TYPE *result) override { + // It has been observed that the pointer that Qt gives us is sometimes null on some machines. + // We need to guard against it in such scenarios. + if (!result) { + return false; + } if (lastMessageHandled) { *result = static_cast<QT_NATIVE_EVENT_RESULT_TYPE>(lastMessageResult); return true; @@ -274,11 +485,10 @@ return true; } - // TODO: Uncomment and do something - // bool frameBorderVisible = Utils::isWindowFrameBorderVisible(); - - // TODO: Implement - // ... + // Main implementation + if (customWindowHandler(hWnd, message, wParam, lParam, result)) { + return true; + } return false; // Not handled } @@ -289,7 +499,7 @@ return (wParam == kMessageTag); } - static quint64 getKeyState() { + static inline quint64 getKeyState() { quint64 result = 0; const auto &get = [](const int virtualKey) -> bool { return (::GetAsyncKeyState(virtualKey) < 0); @@ -343,9 +553,7 @@ } const auto screenPos = POINT{GET_X_LPARAM(lParam), GET_Y_LPARAM(lParam)}; POINT clientPos = screenPos; - if (::ScreenToClient(hWnd, &clientPos) == FALSE) { - return 0; - } + ::ScreenToClient(hWnd, &clientPos); return MAKELPARAM(clientPos.x, clientPos.y); }(); #if 0 @@ -413,9 +621,8 @@ #undef SEND_MESSAGE } - static bool requestForMouseLeaveMessage(HWND hWnd, bool nonClient) { - TRACKMOUSEEVENT tme; - SecureZeroMemory(&tme, sizeof(tme)); + static inline void requestForMouseLeaveMessage(HWND hWnd, bool nonClient) { + TRACKMOUSEEVENT tme{}; tme.cbSize = sizeof(tme); tme.dwFlags = TME_LEAVE; if (nonClient) { @@ -423,10 +630,7 @@ } tme.hwndTrack = hWnd; tme.dwHoverTime = HOVER_DEFAULT; - if (::TrackMouseEvent(&tme) == FALSE) { - return false; - } - return true; + ::TrackMouseEvent(&tme); } bool Win32WindowContext::snapLayoutHandler(HWND hWnd, UINT message, WPARAM wParam, @@ -466,7 +670,7 @@ // WM_MOUSELEAVE. if (lastHitTestResult != WindowPart::ChromeButton && mouseLeaveBlocked) { mouseLeaveBlocked = false; - std::ignore = requestForMouseLeaveMessage(hWnd, false); + requestForMouseLeaveMessage(hWnd, false); } break; } @@ -541,7 +745,7 @@ // in advance. if (mouseLeaveBlocked) { mouseLeaveBlocked = false; - std::ignore = requestForMouseLeaveMessage(hWnd, false); + requestForMouseLeaveMessage(hWnd, false); } } else { if (mouseLeaveBlocked) { @@ -567,4 +771,232 @@ return false; } + bool Win32WindowContext::customWindowHandler(HWND hWnd, UINT message, WPARAM wParam, + LPARAM lParam, LRESULT *result) + { + switch (message) { + case WM_NCCALCSIZE: { + // Windows鏄牴鎹繖涓秷鎭殑杩斿洖鍊兼潵璁剧疆绐楀彛鐨勫鎴峰尯锛堢獥鍙d腑鐪熸鏄剧ず鐨勫唴瀹癸級 + // 鍜岄潪瀹㈡埛鍖猴紙鏍囬鏍忋�佺獥鍙h竟妗嗐�佽彍鍗曟爮鍜岀姸鎬佹爮绛塛indows绯荤粺鑷鎻愪緵鐨勯儴鍒� + // 锛屼笉杩囧浜嶲t鏉ヨ锛岄櫎浜嗘爣棰樻爮鍜岀獥鍙h竟妗嗭紝闈炲鎴峰尯鍩烘湰涔熼兘鏄嚜缁樼殑锛夌殑鑼� + // 鍥寸殑锛宭Param閲屽瓨鏀剧殑灏辨槸鏂板鎴峰尯鐨勫嚑浣曞尯鍩燂紝榛樿鏄暣涓獥鍙g殑澶у皬锛屾甯� + // 鐨勭▼搴忛渶瑕佷慨鏀硅繖涓弬鏁帮紝鍛婄煡绯荤粺绐楀彛鐨勫鎴峰尯鍜岄潪瀹㈡埛鍖虹殑鑼冨洿锛堜竴鑸潵璇村彲 + // 浠ュ畬鍏ㄤ氦缁橶indows锛岃鍏惰嚜琛屽鐞嗭紝浣跨敤榛樿鐨勫鎴峰尯鍜岄潪瀹㈡埛鍖猴級锛屽洜姝ゅ鏋� + // 鎴戜滑涓嶄慨鏀筶Param锛屽氨鍙互浣垮鎴峰尯鍏呮弧鏁翠釜绐楀彛锛屼粠鑰屽幓鎺夋爣棰樻爮鍜岀獥鍙h竟妗� + // 锛堝洜涓鸿繖浜涗笢瑗块兘琚鎴峰尯缁欑洊浣忎簡銆備絾杈规闃村奖涔熶細鍥犳鑰屼涪澶憋紝涓嶈繃鎴戜滑浼氫娇 + // 鐢ㄥ叾浠栨柟寮忓皢鍏跺甫鍥烇紝璇峰弬鑰冨叾浠栨秷鎭殑澶勭悊锛屾澶勪笉杩囧鎻愬強锛夈�備絾鏈変釜鎯呭喌瑕� + // 鐗瑰埆娉ㄦ剰锛岄偅灏辨槸绐楀彛鏈�澶у寲鍚庯紝绐楀彛鐨勫疄闄呭昂瀵镐細姣斿睆骞曠殑灏哄澶т竴鐐癸紝浠庤�屼娇 + // 鐢ㄦ埛鐪嬩笉鍒扮獥鍙g殑杈圭晫锛岃繖鏍风敤鎴峰氨涓嶈兘鍦ㄧ獥鍙f渶澶у寲鍚庤皟鏁寸獥鍙g殑澶у皬浜嗭紙铏界劧 + // 杩欎釜鍋氭硶鍚捣鏉ョ壒鍒鎬紝浣哤indows纭疄灏辨槸杩欐牱鍋氱殑锛夛紝鍥犳濡傛灉鎴戜滑瑕佽嚜琛� + // 澶勭悊绐楀彛鐨勯潪瀹㈡埛鍖猴紝灏辫鍦ㄧ獥鍙f渶澶у寲鍚庯紝灏嗙獥鍙h竟妗嗙殑瀹藉害鍜岄珮搴︼紙涓�鑸槸鐩� + // 绛夌殑锛変粠瀹㈡埛鍖鸿鍓帀锛屽惁鍒欐垜浠獥鍙f墍鏄剧ず鐨勫唴瀹瑰氨浼氳秴鍑哄睆骞曡竟鐣岋紝鏄剧ず涓嶅叏銆� + // 濡傛灉鐢ㄦ埛寮�鍚簡浠诲姟鏍忚嚜鍔ㄩ殣钘忥紝鍦ㄧ獥鍙f渶澶у寲鍚庯紝杩樿鑰冭檻浠诲姟鏍忕殑浣嶇疆銆傚洜涓� + // 濡傛灉绐楀彛鏈�澶у寲鍚庯紝鍏跺昂瀵稿拰灞忓箷灏哄鐩哥瓑锛堝洜涓轰换鍔℃爮闅愯棌浜嗭紝鎵�浠ョ獥鍙f渶澶у寲 + // 鍚庡叾瀹炴槸鍏呮弧浜嗘暣涓睆骞曪紝鍙樼浉鐨勫叏灞忎簡锛夛紝Windows浼氳涓虹獥鍙e凡缁忚繘鍏ュ叏灞忕殑 + // 鐘舵�侊紝浠庤�屽鑷磋嚜鍔ㄩ殣钘忕殑浠诲姟鏍忔棤娉曞脊鍑恒�傝閬垮厤杩欎釜鐘跺喌锛屽氨瑕佷娇绐楀彛鐨勫昂瀵� + // 灏忎簬灞忓箷灏哄銆傛垜涓嬮潰鐨勫仛娉曞弬鑰冧簡鐏嫄銆丆hromium鍜學indows Terminal + // 濡傛灉娌℃湁寮�鍚换鍔℃爮鑷姩闅愯棌锛屾槸涓嶅瓨鍦ㄨ繖涓棶棰樼殑锛屾墍浠ヨ鍏堣繘琛屽垽鏂�� + // 涓�鑸儏鍐典笅锛�*result璁剧疆涓�0锛堢浉褰撲簬DefWindowProc鐨勮繑鍥炲�间负0锛夊氨鍙互浜嗭紝 + // 鏍规嵁MSDN鐨勮娉曪紝杩斿洖0鎰忎负姝ゆ秷鎭凡缁忚绋嬪簭鑷澶勭悊浜嗭紝璁¦indows璺宠繃姝ゆ秷 + // 鎭紝鍚﹀垯Windows浼氭坊鍔犲姝ゆ秷鎭殑榛樿澶勭悊锛屽浜庡綋鍓嶈繖涓秷鎭�岃█锛屽氨鎰忓懗鐫� + // 鏍囬鏍忓拰绐楀彛杈规鍙堜細鍥炴潵锛岃繖褰撶劧涓嶆槸鎴戜滑鎯宠鐨勭粨鏋溿�傛牴鎹甅SDN锛屽綋wParam + // 涓篎ALSE鏃讹紝鍙兘杩斿洖0锛屼絾褰撳叾涓篢RUE鏃讹紝鍙互杩斿洖0锛屼篃鍙互杩斿洖涓�涓猈VR_甯� + // 閲忋�傛牴鎹瓹hromium鐨勬敞閲婏紝褰撳瓨鍦ㄩ潪瀹㈡埛鍖烘椂锛屽鏋滆繑鍥濿VR_REDRAW浼氬鑷村瓙 + // 绐楀彛/瀛愭帶浠跺嚭鐜板鎬殑bug锛堣嚜缁樻帶浠堕敊浣嶏級锛屽苟涓擫ucas鍦╓indows 10 + // 涓婃垚鍔熷鐜帮紝璇存槑杩欎釜bug鑷充粖閮芥病鏈夎В鍐炽�傛垜鏌ラ槄浜嗗ぇ閲忚祫鏂欙紝鍙戠幇鍞竴鐨勮В鍐� + // 鏂规灏辨槸杩斿洖0銆備絾濡傛灉涓嶅瓨鍦ㄩ潪瀹㈡埛鍖猴紝涓攚Param涓篢RUE锛屾渶濂借繑鍥� + // WVR_REDRAW锛屽惁鍒欑獥鍙e湪璋冩暣澶у皬鍙兘浼氫骇鐢熶弗閲嶇殑闂儊鐜拌薄銆� + // 铏界劧瀵瑰ぇ澶氭暟娑堟伅鏉ヨ锛岃繑鍥�0閮戒唬琛ㄨWindows蹇界暐姝ゆ秷鎭紝浣嗗疄闄呬笂涓嶅悓娑堟伅 + // 鑳芥帴鍙楃殑杩斿洖鍊兼槸涓嶄竴鏍风殑锛岃娉ㄦ剰鑷鏌ラ槄MSDN銆� + + // Sent when the size and position of a window's client area must be + // calculated. By processing this message, an application can + // control the content of the window's client area when the size or + // position of the window changes. If wParam is TRUE, lParam points + // to an NCCALCSIZE_PARAMS structure that contains information an + // application can use to calculate the new size and position of the + // client rectangle. If wParam is FALSE, lParam points to a RECT + // structure. On entry, the structure contains the proposed window + // rectangle for the window. On exit, the structure should contain + // the screen coordinates of the corresponding window client area. + // The client area is the window's content area, the non-client area + // is the area which is provided by the system, such as the title + // bar, the four window borders, the frame shadow, the menu bar, the + // status bar, the scroll bar, etc. But for Qt, it draws most of the + // window area (client + non-client) itself. We now know that the + // title bar and the window frame is in the non-client area and we + // can set the scope of the client area in this message, so we can + // remove the title bar and the window frame by let the non-client + // area be covered by the client area (because we can't really get + // rid of the non-client area, it will always be there, all we can + // do is to hide it) , which means we should let the client area's + // size the same with the whole window's size. So there is no room + // for the non-client area and then the user won't be able to see it + // again. But how to achieve this? Very easy, just leave lParam (the + // re-calculated client area) untouched. But of course you can + // modify lParam, then the non-client area will be seen and the + // window borders and the window frame will show up. However, things + // are quite different when you try to modify the top margin of the + // client area. DWM will always draw the whole title bar no matter + // what margin value you set for the top, unless you don't modify it + // and remove the whole top area (the title bar + the one pixel + // height window border). This can be confirmed in Windows + // Terminal's source code, you can also try yourself to verify + // it. So things will become quite complicated if you want to + // preserve the four window borders. + + // If `wParam` is `FALSE`, `lParam` points to a `RECT` that contains + // the proposed window rectangle for our window. During our + // processing of the `WM_NCCALCSIZE` message, we are expected to + // modify the `RECT` that `lParam` points to, so that its value upon + // our return is the new client area. We must return 0 if `wParam` + // is `FALSE`. + // If `wParam` is `TRUE`, `lParam` points to a `NCCALCSIZE_PARAMS` + // struct. This struct contains an array of 3 `RECT`s, the first of + // which has the exact same meaning as the `RECT` that is pointed to + // by `lParam` when `wParam` is `FALSE`. The remaining `RECT`s, in + // conjunction with our return value, can + // be used to specify portions of the source and destination window + // rectangles that are valid and should be preserved. We opt not to + // implement an elaborate client-area preservation technique, and + // simply return 0, which means "preserve the entire old client area + // and align it with the upper-left corner of our new client area". + const auto clientRect = ((wParam == FALSE) ? reinterpret_cast<LPRECT>(lParam) : &(reinterpret_cast<LPNCCALCSIZE_PARAMS>(lParam))->rgrc[0]); + if (isWin10OrGreater()) { + // Store the original top margin before the default window procedure applies the default frame. + const LONG originalTop = clientRect->top; + // Apply the default frame because we don't want to remove the whole window frame, + // we still need the standard window frame (the resizable frame border and the frame + // shadow) for the left, bottom and right edges. + // If we return 0 here directly, the whole window frame will be removed (which means + // there will be no resizable frame border and the frame shadow will also disappear), + // and that's also how most applications customize their title bars on Windows. It's + // totally OK but since we want to preserve as much original frame as possible, we + // can't use that solution. + const LRESULT hitTestResult = ::DefWindowProcW(hWnd, WM_NCCALCSIZE, wParam, lParam); + if ((hitTestResult != HTERROR) && (hitTestResult != HTNOWHERE)) { + *result = hitTestResult; + return true; + } + // Re-apply the original top from before the size of the default frame was applied, + // and the whole top frame (the title bar and the top border) is gone now. + // For the top frame, we only has 2 choices: (1) remove the top frame entirely, or + // (2) don't touch it at all. We can't preserve the top border by adjusting the top + // margin here. If we try to modify the top margin, the original title bar will + // always be painted by DWM regardless what margin we set, so here we can only remove + // the top frame entirely and use some special technique to bring the top border back. + clientRect->top = originalTop; + } + const bool max = IsMaximized(hWnd); + const bool full = isFullScreen(hWnd); + // We don't need this correction when we're fullscreen. We will + // have the WS_POPUP size, so we don't have to worry about + // borders, and the default frame will be fine. + if (max && !full) { + // When a window is maximized, its size is actually a little bit more + // than the monitor's work area. The window is positioned and sized in + // such a way that the resize handles are outside the monitor and + // then the window is clipped to the monitor so that the resize handle + // do not appear because you don't need them (because you can't resize + // a window when it's maximized unless you restore it). + const quint32 frameSize = getResizeBorderThickness(hWnd); + clientRect->top += frameSize; + if (!isWin10OrGreater()) { + clientRect->bottom -= frameSize; + clientRect->left += frameSize; + clientRect->right -= frameSize; + } + } + // Attempt to detect if there's an autohide taskbar, and if + // there is, reduce our size a bit on the side with the taskbar, + // so the user can still mouse-over the taskbar to reveal it. + // Make sure to use MONITOR_DEFAULTTONEAREST, so that this will + // still find the right monitor even when we're restoring from + // minimized. + if (max || full) { + APPBARDATA abd; + SecureZeroMemory(&abd, sizeof(abd)); + abd.cbSize = sizeof(abd); + const UINT taskbarState = ::SHAppBarMessage(ABM_GETSTATE, &abd); + // First, check if we have an auto-hide taskbar at all: + if (taskbarState & ABS_AUTOHIDE) { + bool top = false, bottom = false, left = false, right = false; + // Due to ABM_GETAUTOHIDEBAREX was introduced in Windows 8.1, + // we have to use another way to judge this if we are running + // on Windows 7 or Windows 8. + if (isWin8Point1OrGreater()) { + const std::optional<MONITORINFOEXW> monitorInfo = getMonitorForWindow(hWnd); + const RECT monitorRect = monitorInfo.value().rcMonitor; + // This helper can be used to determine if there's an + // auto-hide taskbar on the given edge of the monitor + // we're currently on. + const auto hasAutohideTaskbar = [monitorRect](const UINT edge) -> bool { + APPBARDATA abd2{}; + abd2.cbSize = sizeof(abd2); + abd2.uEdge = edge; + abd2.rc = monitorRect; + const auto hTaskbar = reinterpret_cast<HWND>(::SHAppBarMessage(ABM_GETAUTOHIDEBAREX, &abd2)); + return (hTaskbar != nullptr); + }; + top = hasAutohideTaskbar(ABE_TOP); + bottom = hasAutohideTaskbar(ABE_BOTTOM); + left = hasAutohideTaskbar(ABE_LEFT); + right = hasAutohideTaskbar(ABE_RIGHT); + } else { + int edge = -1; + APPBARDATA abd2{}; + abd2.cbSize = sizeof(abd2); + abd2.hWnd = ::FindWindowW(L"Shell_TrayWnd", nullptr); + const HMONITOR windowMonitor = ::MonitorFromWindow(hWnd, MONITOR_DEFAULTTONEAREST); + const HMONITOR taskbarMonitor = ::MonitorFromWindow(abd2.hWnd, MONITOR_DEFAULTTOPRIMARY); + if (taskbarMonitor == windowMonitor) { + ::SHAppBarMessage(ABM_GETTASKBARPOS, &abd2); + edge = abd2.uEdge; + } + top = (edge == ABE_TOP); + bottom = (edge == ABE_BOTTOM); + left = (edge == ABE_LEFT); + right = (edge == ABE_RIGHT); + } + // If there's a taskbar on any side of the monitor, reduce + // our size a little bit on that edge. + // Note to future code archeologists: + // This doesn't seem to work for fullscreen on the primary + // display. However, testing a bunch of other apps with + // fullscreen modes and an auto-hiding taskbar has + // shown that _none_ of them reveal the taskbar from + // fullscreen mode. This includes Edge, Firefox, Chrome, + // Sublime Text, PowerPoint - none seemed to support this. + // This does however work fine for maximized. + if (top) { + // Peculiarly, when we're fullscreen, + clientRect->top += kAutoHideTaskBarThickness; + } else if (bottom) { + clientRect->bottom -= kAutoHideTaskBarThickness; + } else if (left) { + clientRect->left += kAutoHideTaskBarThickness; + } else if (right) { + clientRect->right -= kAutoHideTaskBarThickness; + } + } + } + // ### TODO: std::ignore = Utils::syncWmPaintWithDwm(); // This should be executed at the very last. + // By returning WVR_REDRAW we can make the window resizing look less broken. + // But we must return 0 if wParam is FALSE, according to Microsoft Docs. + // **IMPORTANT NOTE**: + // If you are drawing something manually through D3D in your window, don't + // try to return WVR_REDRAW here, otherwise Windows exhibits bugs where + // client pixels and child windows are mispositioned by the width/height + // of the upper-left non-client area. It's confirmed that this issue exists + // from Windows 7 to Windows 10. Not tested on Windows 11 yet. Don't know + // whether it exists on Windows XP to Windows Vista or not. + *result = wParam == FALSE ? FALSE : WVR_REDRAW; + return true; + } + default: + break; + } + return false; + } + } \ No newline at end of file -- Gitblit v1.9.1