Verify that there is no scene windows when Scene is about to be destroyed
ClosedPublic

Authored by zzag on Mar 5 2019, 12:45 PM.

Details

Summary

When ~Scene is called, it's not guaranteed that backend is still valid
or that there is current opengl context. So, generally speaking,
deleting scene windows in ~Scene is a bad idea.

It's okay not to delete scene windows in ~Scene because current
implementation of Compositor::finish ensures that they don't last longer
than the scene.

Diff Detail

Repository
R108 KWin
Branch
assert-empty-windows
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 9194
Build 9212: arc lint + arc unit
zzag created this revision.Mar 5 2019, 12:45 PM
Restricted Application added a project: KWin. · View Herald TranscriptMar 5 2019, 12:45 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Mar 5 2019, 12:45 PM
zzag edited the summary of this revision. (Show Details)Mar 5 2019, 12:47 PM
davidedmundson accepted this revision.Mar 5 2019, 1:16 PM
This revision is now accepted and ready to land.Mar 5 2019, 1:16 PM
This revision was automatically updated to reflect the committed changes.