CC::Animations should use TimeTicks & TimeDelta to represent time

webkit/renderer/compositor_bindings/web_animation_impl.cc

Index: webkit/renderer/compositor_bindings/web_animation_impl.cc
diff --git a/webkit/renderer/compositor_bindings/web_animation_impl.cc b/webkit/renderer/compositor_bindings/web_animation_impl.cc
index c401d5c974128060ca7a6f59530fa726ff1dd463..12ceb2c52cc538ed74d535e949f8c9228a8624f2 100644
--- a/webkit/renderer/compositor_bindings/web_animation_impl.cc
+++ b/webkit/renderer/compositor_bindings/web_animation_impl.cc
@@ -81,18 +81,21 @@ int WebAnimationImpl::iterations() const { return animation_->iterations(); }
 
 void WebAnimationImpl::setIterations(int n) { animation_->set_iterations(n); }
 
-double WebAnimationImpl::startTime() const { return animation_->start_time(); }
+double WebAnimationImpl::startTime() const {
+  return animation_->ticks_inseconds(animation_->start_time());

[ajuma, 2014/04/24 20:41:36]

There's no need for a ticks_inseconds method. Just convert directly here in the
usual way (animation->start_time() - base::TimeTicks()).InSecondsF()).

[Sikugu_, 2014/05/05 07:09:19]

Done.

+}
 
 void WebAnimationImpl::setStartTime(double monotonic_time) {
-  animation_->set_start_time(monotonic_time);
+  animation_->set_start_time(base::TimeTicks::FromInternalValue(
+      monotonic_time * base::Time::kMicrosecondsPerSecond));
 }
 
 double WebAnimationImpl::timeOffset() const {
-  return animation_->time_offset();
+  return animation_->delta_inseconds(animation_->time_offset());

[ajuma, 2014/04/24 20:41:36]

There's no need for a delta_inseconds method. Use TimeDelta's conversion methods
directly.

[Sikugu_, 2014/05/05 07:09:19]

Done.

 }
 
 void WebAnimationImpl::setTimeOffset(double monotonic_time) {
-  animation_->set_time_offset(monotonic_time);
+  animation_->set_time_offset(base::TimeDelta::FromSeconds(monotonic_time));
 }
 
 bool WebAnimationImpl::alternatesDirection() const {