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

Unified Diff: cc/animation/layer_animation_controller.h

Issue 231133002: CC::Animations should use TimeTicks & TimeDelta to represent time (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Code changed as per review comments Created 6 years, 7 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
« no previous file with comments | « cc/animation/animation_unittest.cc ('k') | cc/animation/layer_animation_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/animation/layer_animation_controller.h
diff --git a/cc/animation/layer_animation_controller.h b/cc/animation/layer_animation_controller.h
index bf54a148b11c2bd6d455fd9074352ab7917b6e96..72f4c46a9cc9fe51f8d7ed79c015f4c92e419ea1 100644
--- a/cc/animation/layer_animation_controller.h
+++ b/cc/animation/layer_animation_controller.h
@@ -40,7 +40,7 @@ class CC_EXPORT LayerAnimationController
int id() const { return id_; }
void AddAnimation(scoped_ptr<Animation> animation);
- void PauseAnimation(int animation_id, double time_offset);
+ void PauseAnimation(int animation_id, base::TimeDelta time_offset);
void RemoveAnimation(int animation_id);
void RemoveAnimation(int animation_id,
Animation::TargetProperty target_property);
@@ -52,8 +52,8 @@ class CC_EXPORT LayerAnimationController
virtual void PushAnimationUpdatesTo(
LayerAnimationController* controller_impl);
- void Animate(double monotonic_time);
- void AccumulatePropertyUpdates(double monotonic_time,
+ void Animate(base::TimeTicks monotonic_time);
+ void AccumulatePropertyUpdates(base::TimeTicks monotonic_time,
AnimationEventsVector* events);
void UpdateState(bool start_ready_animations,
@@ -152,15 +152,15 @@ class CC_EXPORT LayerAnimationController
void PushPropertiesToImplThread(
LayerAnimationController* controller_impl) const;
- void StartAnimations(double monotonic_time);
- void PromoteStartedAnimations(double monotonic_time,
+ void StartAnimations(base::TimeTicks monotonic_time);
+ void PromoteStartedAnimations(base::TimeTicks monotonic_time,
AnimationEventsVector* events);
- void MarkFinishedAnimations(double monotonic_time);
- void MarkAnimationsForDeletion(double monotonic_time,
+ void MarkFinishedAnimations(base::TimeTicks monotonic_time);
+ void MarkAnimationsForDeletion(base::TimeTicks monotonic_time,
AnimationEventsVector* events);
void PurgeAnimationsMarkedForDeletion();
- void TickAnimations(double monotonic_time);
+ void TickAnimations(base::TimeTicks monotonic_time);
enum UpdateActivationType {
NormalActivation,
@@ -193,7 +193,7 @@ class CC_EXPORT LayerAnimationController
// This is used to ensure that we don't spam the registrar.
bool is_active_;
- double last_tick_time_;
+ base::TimeTicks last_tick_time_;
ObserverList<LayerAnimationValueObserver> value_observers_;
ObserverList<LayerAnimationEventObserver> event_observers_;
« no previous file with comments | « cc/animation/animation_unittest.cc ('k') | cc/animation/layer_animation_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698