[PATCH v2 0/1] MR8988: win32u: change the stretch mode of dst hdc from BlackOnWhite to ColorOnColor...

[Kun Yang (＠yangkun)]

win32u: change the stretch mode of dst hdc from BlackOnWhite to ColorOnColor when using StretchBlt in TransparentBlt.

In NtGdiTransparentBlt function, the StretchBlt mode of dst hdc is not set. The Default StretchBlt Mode in wine is BlackOnWhite. It is not correct to use this StretchBltMode in TransparentBlt dealing with color bitmap. According to MSDN, it should be converted to ColorOnColor.
My test code shows that, the image converted by wine generates a lot of chromatic noise.

[test_blt.c](/uploads/26101bc7bdb27400d02c8ff5bf741864/test_blt.c)

[test_blt.exe](/uploads/98ee3ab7ce0d590e1511bd8a5fb5a1bb/test_blt.exe)

![source](/uploads/0c56c5e6d1280040ba08cd3dee7acd95/source.bmp)

![result_wrong](/uploads/2076396e069f852914313a8fb86dd5e2/result_wrong.bmp)

![result](/uploads/cf3ec00bf0f0e7f3ea7fe1e2085385d6/result.bmp)

--
  v2: win32u: change the stretch mode of dst hdc from BlackOnWhite to ColorOnColor when using StretchBlt in TransparentBlt.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8988

[PATCH v2 1/1] win32u: change the stretch mode of dst hdc from BlackOnWhite to ColorOnColor when using StretchBlt in TransparentBlt.

[yangkun]

From: yangkun <yangkun(a)uniontech.com>

Signed-off-by: yangkun <yangkun(a)uniontech.com>
Change-Id: Ic3c64a5d2ffef7d890f461b089223fbd7a972ffd
---
 dlls/win32u/bitblt.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/dlls/win32u/bitblt.c b/dlls/win32u/bitblt.c
index a9f6f70daf6..55379628927 100644
--- a/dlls/win32u/bitblt.c
+++ b/dlls/win32u/bitblt.c
@@ -857,6 +857,7 @@ BOOL WINAPI NtGdiTransparentBlt( HDC hdcDest, int xDest, int yDest, int widthDes
     int oldStretchMode;
     DIBSECTION dib;
     DC *dc_src;
+    DC *dc_work;
 
     if(widthDest < 0 || heightDest < 0 || widthSrc < 0 || heightSrc < 0) {
         TRACE("Cannot mirror\n");
@@ -890,6 +891,10 @@ BOOL WINAPI NtGdiTransparentBlt( HDC hdcDest, int xDest, int yDest, int widthDes
     }
     else bmpWork = NtGdiCreateCompatibleBitmap( hdcDest, widthDest, heightDest );
     oldWork = NtGdiSelectBitmap(hdcWork, bmpWork);
+
+    if (!(dc_work = get_dc_ptr(hdcWork))) goto error;
+    dc_work->attr->stretch_blt_mode = COLORONCOLOR;
+
     if (!NtGdiStretchBlt( hdcWork, 0, 0, widthDest, heightDest, hdcSrc, xSrc, ySrc,
                           widthSrc, heightSrc, SRCCOPY, 0 ))
     {
@@ -937,6 +942,8 @@ error:
     release_dc_ptr( dc_src );
     NtGdiGetAndSetDCDword( hdcDest, NtGdiSetBkColor, oldBackground, NULL );
     NtGdiGetAndSetDCDword( hdcDest, NtGdiSetTextColor, oldForeground, NULL );
+    if(dc_work) release_dc_ptr( dc_work );
+
     if(hdcWork) {
         NtGdiSelectBitmap(hdcWork, oldWork);
         NtGdiDeleteObjectApp( hdcWork );
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/8988