Changing animate to beginFrame and use struct rather than naked double to allow extension.

public/platform/WebFrameTime.h

 /*
- * Copyright (C) 2009 Google Inc. All rights reserved.
+ * Copyright (C) 2014 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
  * copyright notice, this list of conditions and the following disclaimer
  * in the documentation and/or other materials provided with the
  * distribution.
  *     * Neither the name of Google Inc. nor the names of its
  * contributors may be used to endorse or promote products derived from
  * this software without specific prior written permission.
  *
  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
  * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
  * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * 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 WebPoint_h
-#define WebPoint_h
+#ifndef WebFrameTime_h
+#define WebFrameTime_h
 
 #include "WebCommon.h"
 
-#if INSIDE_BLINK
-#include "platform/geometry/IntPoint.h"
-#else
-#include <ui/gfx/point.h>
-#endif
-
 namespace blink {
 
-struct WebPoint {
-    int x;
-    int y;
-
-    WebPoint()
-        : x(0)
-        , y(0)
+struct WebFrameTime {
+    WebFrameTime(double flt, uint64_t d, uint64_t fdt)

[abarth-chromium, 2014/06/11 19:33:01]

Please use complete words in variable names.

[mithro-old, 2014/06/18 03:36:44]

Done.

+        : frameLastTime(flt)
+        , renderDeadline(d)
+        , frameDisplayTime(fdt)
     {
     }
 
-    WebPoint(int x, int y)
-        : x(x)
-        , y(y)
+    // FIXME(mithro): Upgrade frame_time, deadline and frame_dispaly_time to TimeTick class.

[abarth-chromium, 2014/06/11 19:33:01]

frame_time -> frameTime
frame_dispaly_time -> frameDisplayTime

+
+    // Time in CLOCK_MONOTONIC that is the most recent vsync time.
+    double frameLastTime;

[abarth-chromium, 2014/06/11 19:33:01]

lastFrameTime?

[mithro-old, 2014/06/18 03:36:44]

Done.

+
+    // Time in CLOCK_MONOTONIC by which rendering must be finished by to not
+    // stall the pipeline.
+    double renderDeadline;
+
+    // Time in CLOCK_MONOTONIC that the frame will hit the glass of the display
+    // if rendered within the deadline. A value of NaN means we do not know and
+    // you should use frameLastTime instead.
+    double frameDisplayTime;
+
+    inline bool frameDisplayTimeValid()

[abarth-chromium, 2014/06/11 19:33:01]

The keyword |inline| doesn't do anything there.

[mithro-old, 2014/06/18 03:36:44]

I removed these functions and just made sure displayTime was always populated
(with lastFrameTime if we don't know otherwise).

     {
+        return (frameDisplayTime == frameDisplayTime) && frameDisplayTime >= frameLastTime;
     }
 
-#if INSIDE_BLINK
-    WebPoint(const WebCore::IntPoint& p)
-        : x(p.x())
-        , y(p.y())
+    inline double frameTime()

[abarth-chromium, 2014/06/11 19:33:01]

ditto

[mithro-old, 2014/06/18 03:36:44]

Done.

     {
+        if (frameDisplayTimeValid())
+            return frameDisplayTime;
+        return frameLastTime;
     }
-
-    WebPoint& operator=(const WebCore::IntPoint& p)
-    {
-        x = p.x();
-        y = p.y();
-        return *this;
-    }
-
-    operator WebCore::IntPoint() const
-    {
-        return WebCore::IntPoint(x, y);
-    }
-#else
-    WebPoint(const gfx::Point& p)
-        : x(p.x())
-        , y(p.y())
-    {
-    }
-
-    WebPoint& operator=(const gfx::Point& p)
-    {
-        x = p.x();
-        y = p.y();
-        return *this;
-    }
-
-    operator gfx::Point() const
-    {
-        return gfx::Point(x, y);
-    }
-#endif
 };
 
-inline bool operator==(const WebPoint& a, const WebPoint& b)
-{
-    return a.x == b.x && a.y == b.y;
-}
-
-inline bool operator!=(const WebPoint& a, const WebPoint& b)
-{
-    return !(a == b);
-}
-
 } // namespace blink
 
 #endif