After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 744684 - gdk: master-clock: force scheduling new frames for timelines alive
gdk: master-clock: force scheduling new frames for timelines alive
Status: RESOLVED FIXED
Product: clutter
Classification: Platform
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: clutter-maint
clutter-maint
Depends on:
Blocks:
 
 
Reported: 2015-02-18 00:09 UTC by Lionel Landwerlin
Modified: 2015-02-18 11:39 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
gdk: master-clock: force scheduling new frames for timelines alive (5.22 KB, patch)
2015-02-18 00:09 UTC, Lionel Landwerlin 		none Details | Review
gdk: master-clock: force scheduling new frames for timelines alive (4.43 KB, patch)
2015-02-18 00:17 UTC, Lionel Landwerlin 		committed Details | Review

Description Lionel Landwerlin 2015-02-18 00:09:06 UTC 
As long as we have timelines alive, we need to keep asking the
GdkFrameClock for new frames. Otherwise animations might stall.

This is something I didn't quite grasp in the first iteration. Sorry
for that.
Comment 1 Lionel Landwerlin 2015-02-18 00:09:11 UTC 
Created attachment 297071 [details] [review]
gdk: master-clock: force scheduling new frames for timelines alive

As long as we have timelines alive, we need to keep asking the
GdkFrameClock for new frames. Otherwise animations might stall.
Comment 2 Lionel Landwerlin 2015-02-18 00:17:18 UTC 
Created attachment 297072 [details] [review]
gdk: master-clock: force scheduling new frames for timelines alive

As long as we have timelines alive, we need to keep asking the
GdkFrameClock for new frames. Otherwise animations might stall.
Comment 3 Emmanuele Bassi (:ebassi) 2015-02-18 00:54:34 UTC 
Review of attachment 297072 [details] [review]:

looks good.
Comment 4 Lionel Landwerlin 2015-02-18 11:39:23 UTC 
Review of attachment 297072 [details] [review]:

Pushed to master.