[COMCTL32] Address regression CORE-14622
authorJoachim Henze <Joachim.Henze@reactos.org>
Mon, 21 Oct 2019 23:10:28 +0000 (01:10 +0200)
committerJoachim Henze <Joachim.Henze@reactos.org>
Mon, 21 Oct 2019 23:10:28 +0000 (01:10 +0200)
"Avira Antivir Personal 6.35.00.243" had issues painting its
setup wizard dialog with correct background color.
That part was a regression of SVN r74136 == git
b1b42795a94d8e3cb69e2d9e5bd0af4929610361

and also did not draw its umbrella bmp, which did never work
in ros up to now afaik.

https://reactos.org/testman/compare.php?ids=69394,69478 VBOX
https://reactos.org/testman/compare.php?ids=69395,69479 KVM

Thanks to patches author Doug Lyons and HBelusca.

Wine has been reported to be affected by that bug as well:
https://bugs.winehq.org/show_bug.cgi?id=46636
Not sure whether our patch will be accepted by Wine,
to my naive eye this looks like an interim solution.

dll/win32/comctl32/propsheet.c

index 746d66c..4dd355a 100644 (file)
@@ -1470,6 +1470,9 @@ static BOOL PROPSHEET_CreatePage(HWND hwndParent,
      (psInfo->ppshheader.dwFlags & PSH_WATERMARK) &&
      (ppshpage->dwFlags & PSP_HIDEHEADER))
   {
+#ifdef __REACTOS__
+    if (psInfo->ppshheader.u4.hbmWatermark)
+#endif
       SetWindowSubclass(hwndPage, PROPSHEET_WizardSubclassProc, 1,
                         (DWORD_PTR)ppshpage);
   }