This commit helped a lot in bringing down the CPU usage of plasmashell while Elisa is playing a song. However, plasmashell still consumes quite a bit of CPU, because line 216 (Q_EMIT Seeked(m_position);) is outside the if clause that was added by this commit. As such, Q_EMIT is still triggered a lot - even when the progress is less than 1% - and that induces load on the plasmashell. Why was it not moved inside the if clause like line 231 (QDBusConnection::sessionBus().send(mProgressIndicatorSignal);)?

Seems like a sensible change. Would you be interested in submitting a merge request to do that? Thanks!

mentioned in merge request !193 (merged)

mentioned in merge request plasma-mobile/plasma-dialer!108 (merged)