[x11] Check pending release when mapping

Authored by scao on Oct 23 2019, 6:38 AM.



when unmap notify is followed by a map, the old Unmanaged will get released and never be managed again. by checking if there is a pending release operation, we can safely re-manage the window again.

BUG: 413350

Diff Detail

R108 KWin
Lint Skipped
Unit Tests Skipped
scao created this revision.Oct 23 2019, 6:38 AM
Restricted Application added a subscriber: kwin. · View Herald TranscriptOct 23 2019, 6:38 AM
scao requested review of this revision.Oct 23 2019, 6:38 AM
zzag added a subscriber: zzag.Oct 23 2019, 9:17 AM

That's a good discovery! Do you have a test client by any chance?

I have write a simple qt demo with only one override-redirected window, and clicking the button will produce the effect.

zzag accepted this revision.Oct 23 2019, 3:13 PM
zzag retitled this revision from check pending release when mapping to [x11] Check pending release when mapping.
zzag edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Oct 23 2019, 3:13 PM
zzag added a comment.Oct 23 2019, 3:25 PM

Do you have commit access?

I believe I had, but a long time has passed since I last committed.

should I commit it now?

zzag added a comment.Oct 24 2019, 7:53 AM

Go for it.

scao closed this revision.Oct 24 2019, 10:20 AM