Make the web-animations engine use the passed in blink::WebFrameTime values.

Source/core/animation/AnimationClock.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef AnimationClock_h
 #define AnimationClock_h
 
-#include "wtf/CurrentTime.h"
+#include "public/platform/WebFrameTime.h"
 #include "wtf/Noncopyable.h"
 #include "wtf/PassOwnPtr.h"
 #include <limits>
 
 namespace WebCore {
 
 class AnimationClock {
     WTF_MAKE_NONCOPYABLE(AnimationClock);
 public:
-    explicit AnimationClock(WTF::TimeFunction monotonicallyIncreasingTime = WTF::monotonicallyIncreasingTime)
-        : m_monotonicallyIncreasingTime(monotonicallyIncreasingTime)
-        , m_time(0)
+    explicit AnimationClock()
+        : m_frameTime(-1, -1, -1, -1)

[eseidel, 2014/07/07 16:31:50]

Can this be a default constructor on WebFrameTime instead?

[mithro-old, 2014/07/07 17:47:15]

Open to better suggestions on how to deal with the below.

The only place which should really be constructing WebFrameTime objects is the
Chrome code which calls into Blink so don't want to really encourage people to
make these objects. (There is already a possible issue in
Source/core/svg/graphics/SVGImageChromeClient.cpp which I have another
experimental patch to fix).

As this is a member variable C++ doesn't let us have this value as being "unset"
so we need some way to mark it as being unset (and using a pointer seems bad).

I can't use (0, 0, 0, 0) - the most logical choice as a many unit tests depend
on the first valid time being t=0.0; I ended up choosing (-1, -1, -1, -1)
because that seemed like the next logical choice.

         , m_currentTask(std::numeric_limits<unsigned>::max())
     {
     }
 
-    void updateTime(double time);
+    void updateTime(blink::WebFrameTime);
     double currentTime();
-    void resetTimeForTesting();
-
     static void notifyTaskStart() { ++s_currentTask; }
 
+    void resetTimeForTesting();
+    void tickTimeForTesting();
+    void clearTimeForTesting();
+
 private:
-    WTF::TimeFunction m_monotonicallyIncreasingTime;
-    double m_time;
+    double internalCurrentTime();
+
+    blink::WebFrameTime m_frameTime;
     unsigned m_currentTask;
     static unsigned s_currentTask;
 };
 
 } // namespace WebCore
 
 #endif // AnimationClock_h