Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2025-02-27 21:00:50 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1188251 part 1 - Remove transitions cleanup from FlushTransitions; r=dholbert

Browse Source 
There is no longer anything in FlushTransitions that modifies the set of
transitions. I believe this changed as of bug 960465, specifically changeset
https://hg.mozilla.org/mozilla-central/rev/b2ee72589c18, so that this code is
no longer needed.

By removing this we can further align FlushAnimations and FlushTransitions.
This commit is contained in:
Brian Birtles 2015-08-17 13:59:44 +09:00
parent 12c9711fb9
commit ab1cd7b690
1 changed files with 1 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
layout/style/nsTransitionManager.cpp
View File

@ -916,8 +916,7 @@ nsTransitionManager::FlushTransitions(FlushFlags aFlags)
TimeStamp now = mPresContext->RefreshDriver()->MostRecentRefresh();
bool didThrottle = false;
// Trim transitions that have completed, post restyle events for frames that
// are still transitioning, and start transitions with delays.
// Post restyle events for frames that are transitioning.
{
PRCList *next = PR_LIST_HEAD(&mElementCollections);
while (next != &mElementCollections) {
@ -959,12 +958,6 @@ nsTransitionManager::FlushTransitions(FlushFlags aFlags)
} else {
didThrottle = true;
}
if (collection->mAnimations.IsEmpty()) {
collection->Destroy();
// |collection| is now a dangling pointer!
collection = nullptr;
}
}
}