Code review comment for lp://staging/~macslow/qtmir/fix-1475678

Revision history for this message
Gerry Boland (gerboland) wrote :

void MirSurfaceManager::displayOff()
void MirSurfaceManager::displayOn()

I disapprove of this design. A surface manager should not care about display state. It's the shell's decision if a client surface should be marked invisible when the display is off.

Also this throws away any visibility state that the shell may have set on surfaces.

A remote desktop would be an example where we don't care about the actual display state.

« Back to merge proposal