Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(257)

Unified Diff: third_party/WebKit/Source/core/animation/CompositorPendingAnimations.cpp

Issue 2724083002: [SPv2] Decomposite otherwise-compositable animations that paint nothing. (Closed)
Patch Set: Update expectations. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/animation/CompositorPendingAnimations.cpp
diff --git a/third_party/WebKit/Source/core/animation/CompositorPendingAnimations.cpp b/third_party/WebKit/Source/core/animation/CompositorPendingAnimations.cpp
index b99515d56dd95e8465816d595152251f87ccdf97..974e112fc239aa5162ab145a3f8125760eb57f52 100644
--- a/third_party/WebKit/Source/core/animation/CompositorPendingAnimations.cpp
+++ b/third_party/WebKit/Source/core/animation/CompositorPendingAnimations.cpp
@@ -54,7 +54,9 @@ void CompositorPendingAnimations::add(Animation* animation) {
}
}
-bool CompositorPendingAnimations::update(bool startOnCompositor) {
+bool CompositorPendingAnimations::update(
+ const Optional<CompositorElementIdSet>& compositedElementIds,
+ bool startOnCompositor) {
HeapVector<Member<Animation>> waitingForStartTime;
bool startedSynchronizedOnCompositor = false;
@@ -74,7 +76,7 @@ bool CompositorPendingAnimations::update(bool startOnCompositor) {
// Animations with a start time do not participate in compositor start-time
// grouping.
if (animation->preCommit(animation->hasStartTime() ? 1 : compositorGroup,
- startOnCompositor)) {
+ compositedElementIds, startOnCompositor)) {
if (animation->hasActiveAnimationsOnCompositor() &&
!hadCompositorAnimation) {
startedSynchronizedOnCompositor = true;

Powered by Google App Engine
This is Rietveld 408576698