Bug 1341518 - We don't need to call SetNeedStyleFlush() explicitly in UpdateAnimations. r=birtles,mantaroh

DocumentTimeline ensures the next tick, which is necessary for animation events,
is processed. Actually CSSAnimation::QueueEvents() is called from WillRefresh(),
the WillRefresh() is not called if the animation is not being observed by the
DocumentTimeline.

MozReview-Commit-ID: AYdqi7x3jty

--HG--
extra : rebase_source : 074689e7ddb5e48119b5758f73fb436840daec0b
This commit is contained in:
Hiroyuki Ikezoe 2017-02-22 15:45:27 +09:00
Родитель deb9ee27b3
Коммит 4f3263f0d2
1 изменённых файлов: 0 добавлений и 8 удалений

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

@ -468,14 +468,6 @@ nsAnimationManager::UpdateAnimations(nsStyleContext* aStyleContext,
for (size_t newAnimIdx = newAnimations.Length(); newAnimIdx-- != 0; ) {
newAnimations[newAnimIdx]->CancelFromStyle();
}
// We don't actually dispatch the pending events now. We'll either
// dispatch them the next time we get a refresh driver notification
// or the next time somebody calls
// nsPresShell::FlushPendingNotifications.
if (mEventDispatcher.HasQueuedEvents()) {
mPresContext->PresShell()->SetNeedStyleFlush();
}
}
void