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

Side by Side Diff: Source/core/animation/CompositorPendingAnimations.h

Issue 651103002: Web Animations: Compositor start notification should only apply to animations in a matching group (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase. Created 6 years 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 // compositing update. 42 // compositing update.
43 // For CSS Animations, used to synchronize the start of main-thread animations 43 // For CSS Animations, used to synchronize the start of main-thread animations
44 // with compositor animations when both classes of CSS Animations are triggered 44 // with compositor animations when both classes of CSS Animations are triggered
45 // by the same recalc 45 // by the same recalc
46 class CompositorPendingAnimations final { 46 class CompositorPendingAnimations final {
47 DISALLOW_ALLOCATION(); 47 DISALLOW_ALLOCATION();
48 public: 48 public:
49 49
50 CompositorPendingAnimations() 50 CompositorPendingAnimations()
51 : m_timer(this, &CompositorPendingAnimations::timerFired) 51 : m_timer(this, &CompositorPendingAnimations::timerFired)
52 , m_compositorGroup(1)
52 { } 53 { }
53 54
54 void add(AnimationPlayer*); 55 void add(AnimationPlayer*);
55 // Returns whether we are waiting for an animation to start and should 56 // Returns whether we are waiting for an animation to start and should
56 // service again on the next frame. 57 // service again on the next frame.
57 bool update(bool startOnCompositor = true); 58 bool update(bool startOnCompositor = true);
58 void notifyAnimationStarted(double monotonicAnimationStartTime, bool started OnCompositor); 59 void notifyCompositorAnimationStarted(double monotonicAnimationStartTime, in t compositorGroup = 0);
59 void notifyCompositorAnimationStarted(double monotonicAnimationStartTime);
60 60
61 void trace(Visitor*); 61 void trace(Visitor*);
62 62
63 private: 63 private:
64 void timerFired(Timer<CompositorPendingAnimations>*) { update(false); } 64 void timerFired(Timer<CompositorPendingAnimations>*) { update(false); }
65 65
66 WillBeHeapVector<RefPtrWillBeMember<AnimationPlayer> > m_pending; 66 WillBeHeapVector<RefPtrWillBeMember<AnimationPlayer> > m_pending;
67 WillBeHeapVector<RefPtrWillBeMember<AnimationPlayer> > m_waitingForComposito rAnimationStart; 67 WillBeHeapVector<RefPtrWillBeMember<AnimationPlayer> > m_waitingForComposito rAnimationStart;
68 Timer<CompositorPendingAnimations> m_timer; 68 Timer<CompositorPendingAnimations> m_timer;
69 int m_compositorGroup;
69 }; 70 };
70 71
71 } // namespace blink 72 } // namespace blink
72 73
73 #endif 74 #endif
OLDNEW
« no previous file with comments | « Source/core/animation/CompositorAnimationsTest.cpp ('k') | Source/core/animation/CompositorPendingAnimations.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698