Bug 1150810 part 15 - Add a comment about need to store more than just relevant animations; r=jwatt

We'll likely address this as part of bug 1151731 when we sample animations from
their timeline.

--HG--
extra : commitid : 9g00bBtDIue
extra : rebase_source : 12d9de2524eb3133bef5a5bcf4c84d4759ccbbca
This commit is contained in:
Brian Birtles 2015-06-15 11:05:43 +09:00
Родитель 0c7ff86e21
Коммит 918f55389f
1 изменённых файлов: 6 добавлений и 0 удалений

Просмотреть файл

@ -815,6 +815,12 @@ Animation::UpdateTiming(SeekFlag aSeekFlag)
// have no way of notifying its animations). For now, however, we can // have no way of notifying its animations). For now, however, we can
// simply store just the relevant animations. // simply store just the relevant animations.
if (mTimeline) { if (mTimeline) {
// FIXME: Once we expect animations to go back and forth betweeen being
// inactive and active, we will need to store more than just relevant
// animations on the timeline. This is because an animation might be
// deemed irrelevant because its timeline is inactive. If it is removed
// from the timeline at that point the timeline will have no way of
// getting the animation to add itself again once it becomes active.
if (IsRelevant()) { if (IsRelevant()) {
mTimeline->AddAnimation(*this); mTimeline->AddAnimation(*this);
} else { } else {