[Bug 42566] New: Edit control uses WM_PAINT instead of WM_ERASEBKGND

https://bugs.winehq.org/show_bug.cgi?id=42566

Bug ID: 42566
           Summary: Edit control uses WM_PAINT instead of WM_ERASEBKGND
           Product: Wine
           Version: 2.0
          Hardware: x86
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: user32
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

Created attachment 57478
  --> https://bugs.winehq.org/attachment.cgi?id=57478
Test Program

In Windows, the edit control uses the WM_ERASEBKGND message to erase the
background but the Wine edit control uses the WM_PAINT message. Test app
included..

[Bug 42566] Edit control uses WM_PAINT instead of WM_ERASEBKGND

https://bugs.winehq.org/show_bug.cgi?id=42566

James Tabor  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |https://jira.reactos.org/br
                   |                            |owse/CORE-12814

[Bug 42566] Edit control uses WM_PAINT instead of WM_ERASEBKGND

https://bugs.winehq.org/show_bug.cgi?id=42566

--- Comment #1 from Nikolay Sivov  ---
Could you attach test case sources? Preferably for C/C++ application, instead
of VB6 one.

[Bug 42566] Edit control uses WM_PAINT instead of WM_ERASEBKGND

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=42566

--- Comment #2 from Nikolay Sivov  ---
Created attachment 65943
  --> https://bugs.winehq.org/attachment.cgi?id=65943
patch

Confirming. Attached patch makes it look better. Remaining issue that text area
should be erased regardless, even with empty text. That's probably why erasing
code was unusually merged with WM_PAINT originally. This needs more work.

[Bug 42566] Edit control uses WM_PAINT instead of WM_ERASEBKGND

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=42566

Alistair Leslie-Hughes  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch

[Bug 42566] Edit control uses WM_PAINT instead of WM_ERASEBKGND

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=42566

--- Comment #3 from Janne  ---
Created attachment 79077
  --> http://bugs.winehq.org/attachment.cgi?id=79077
screenshot of test application

Tested with Wine 10.9 on Ubuntu 22.04 running on VMware.

Created fresh prefix and installed vb6run with winetricks.

Started test application and typed some text.
Switching between colors didn't do anything.

attached screenshot of application.

[Bug 42566] Edit control uses WM_PAINT instead of WM_ERASEBKGND

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=42566

--- Comment #4 from Janne  ---
Created attachment 79078
  --> http://bugs.winehq.org/attachment.cgi?id=79078
Test application on Windows11

Tested test application on Windows11.

In Windows 11 background color changes in areas where there is no text.

attached screenshot of test application on Windows 11.

[Bug 42566] Edit control uses WM_PAINT instead of WM_ERASEBKGND

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=42566

Maotong Zhang  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #5 from Maotong Zhang  ---
Does the test program have a clear execution flow? WM_ERASEBKGND appears to be
called only once and is not triggered when switching colors.

[Bug 42566] Edit control uses WM_PAINT instead of WM_ERASEBKGND

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=42566

KRosUser  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #6 from KRosUser  ---
Shouldn't the "component" be here changed to comctl32 instead of user32 ?