Don't flush client connection when sending frame callbacks

This way, the compositor can batch more frame callbacks before flushing
the client connection. We attempted this before, but it broke tests.

Now, it seems like the tests pass, so we can remove the manual flush.
This commit is contained in:
Vlad Zahorodnii 2021-06-19 10:35:30 +03:00
parent 669d7fd652
commit ac60e35818

View file

@ -404,7 +404,6 @@ QList<SurfaceInterface *> SurfaceInterface::surfaces()
void SurfaceInterface::frameRendered(quint32 msec)
{
// notify all callbacks
const bool needsFlush = !d->current.frameCallbacks.isEmpty();
while (!d->current.frameCallbacks.isEmpty()) {
KWaylandFrameCallback *frameCallback = d->current.frameCallbacks.takeFirst();
frameCallback->send_done(msec);
@ -413,9 +412,6 @@ void SurfaceInterface::frameRendered(quint32 msec)
for (auto it = d->current.children.constBegin(); it != d->current.children.constEnd(); ++it) {
(*it)->surface()->frameRendered(msec);
}
if (needsFlush) {
client()->flush();
}
}
bool SurfaceInterface::hasFrameCallbacks() const