[Bug 20040] New: After sending EM_SETREADONLY to change edit controls the screen does not get updated

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

Summary: After sending EM_SETREADONLY to change edit controls
                    the screen does not get updated
           Product: Wine
           Version: 1.1.29
          Platform: PC
               URL: http://rh-software.com/
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: gdi32
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=23588)
 --> (http://bugs.winehq.org/attachment.cgi?id=23588)
Window image showing "White" (RW) edit controls what should be "Grey" (RO)

While running SIV (http://rh-software.com/) with an updating [Volumes] page the
screen does not get repainted when I send an EM_SETREADONLY to change R/W to
R/O.  I discovered I can work-a-round this by doing an InvalidateRect() on the
child window as follows:
        SendMessage( chl, EM_SETREADONLY, FALSE, 0 );
        if( mode & MODE_WINE ) InvalidateRect( chl, NULL, TRUE );
This is the case for both Set and Clear of EM_SETREADONLY.  Note that SIV is
just changing the EM_SETREADONLY property, it is not changing the text.
It would probably be less effort for me to actually change SIV to add the
work-a-round code than report the bug in Wine!, but...

[Bug 20040] After sending EM_SETREADONLY to change edit controls the screen does not get updated

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

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download
             Status|UNCONFIRMED                 |NEW
          Component|gdi32                       |user32
     Ever Confirmed|0                           |1
              Alias|EM_SETREADONLY              |
           Severity|normal                      |trivial

--- Comment #1 from Nikolay Sivov   2009-09-14 05:32:15 ---
Confirming. Just tested with ControlSpy on XP and it does invalidation after
this message while Wine just updates style.

Not gdi32 bug, of course ).

[Bug 20040] After sending EM_SETREADONLY to change edit controls the screen does not get updated

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

--- Comment #2 from Nikolay Sivov   2009-09-22 13:59:52 ---
Fixed by commit:

http://source.winehq.org/git/wine.git/?a=commit;h=0209b02f47a13274ca4f26efff...

[Bug 20040] After sending EM_SETREADONLY to change edit controls the screen does not get updated

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

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Nikolay Sivov   2009-09-22 14:00:45 ---
Fixed.

[Bug 20040] After sending EM_SETREADONLY to change edit controls the screen does not get updated

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

--- Comment #4 from red-ray   2009-09-22 14:34:41 ---
(In reply to comment #3)
> Fixed.

Thank you.

[Bug 20040] After sending EM_SETREADONLY to change edit controls the screen does not get updated

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

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #5 from Alexandre Julliard   2009-09-25 12:26:18 ---
Closing bugs fixed in 1.1.30.