[Merge] lp:~vanvugt/totem/fix-1698282 into lp:~ubuntu-desktop/totem/ubuntu

[Daniel van Vugt]

Daniel van Vugt has proposed merging lp:~vanvugt/totem/fix-1698282 into lp:~ubuntu-desktop/totem/ubuntu.

Commit message:
Reduce CPU usage and stuttery playback

For some reason bvw_update_tags gets called constantly even when none are
changing. And each call seems to trigger a clutter scene graph update.
So totem was spending 20-25% of its time in g_signal_emit and below updating the clutter scene graph and GTK widgets. With this fix, totem's
CPU usage drops by about a fifth and playback is much smoother (although
still not perfect).

This fixes the totem portion of LP: #1698282 and LP: #1698270, although
both bugs also need work done in clutter before they can be declared fully
resolved.

Requested reviews:
  Ubuntu Sponsors Team (ubuntu-sponsors)
  Ubuntu Desktop (ubuntu-desktop)
Related bugs:
  Bug #1698282 in Totem: "Totem uses dramatically higher CPU than any other video player"
  https://bugs.launchpad.net/totem/+bug/1698282

For more details, see:
https://code.launchpad.net/~vanvugt/totem/fix-1698282/+merge/329292



-- 
Your team Ubuntu Sponsors Team is requested to review the proposed merge of lp:~vanvugt/totem/fix-1698282 into lp:~ubuntu-desktop/totem/ubuntu.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 2803 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-sponsors/attachments/20170821/03ce6780/attachment.diff>