Adding monotonic only API to Blink's animation delegate interface

public/platform/WebAnimationDelegate.h

 /*
  * Copyright (C) 2012 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:
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
  *     notice, this list of conditions and the following disclaimer in the
@@ skipping 15 matching lines @@
 #define WebAnimationDelegate_h
 
 #include "WebAnimation.h"
 
 #define WEB_ANIMATION_DELEGATE_TAKES_MONOTONIC_TIME 1
 
 namespace blink {
 
 class WebAnimationDelegate {
 public:
-    // FIXME: Remove wallClockTime once the legacy implementation of CSS
-    // animations and transitions is removed.
-    virtual void notifyAnimationStarted(double wallClockTime, double monotonicTime, WebAnimation::TargetProperty) = 0;
-    virtual void notifyAnimationFinished(double wallClockTime, double monotonicTime, WebAnimation::TargetProperty) = 0;
+    // TODO: Removed wallClockTime after the following file is updated;

[ajuma, 2014/03/03 15:25:21]

Nit: Blink style is to use "FIXME" rather than "TODO" (see
http://www.chromium.org/blink/coding-style#TOC-Comments)

Also, s/Removed/Remove

[mithro-old, 2014/03/04 01:39:18]

Done.

+    // webkit/renderer/compositor_bindings/web_to_cc_animation_delegate_adapter.cc
+    inline void notifyAnimationStarted(double wallClockTime, double monotonicTime, WebAnimation::TargetProperty prop)
+    {
+        notifyAnimationStarted(monotonicTime, prop);
+    }
+    inline void notifyAnimationFinished(double wallClockTime, double monotonicTime, WebAnimation::TargetProperty prop)
+    {
+        notifyAnimationFinished(monotonicTime, prop);
+    }
+    virtual void notifyAnimationStarted(double monotonicTime, WebAnimation::TargetProperty) = 0;

[ajuma, 2014/03/03 15:25:21]

If this CL lands first (as in the steps you've described), this will initially
have no implementation anywhere. I'd suggest doing the chromium side first
(guarded by #ifdefs), then do all of the Blink side (and add the appropriate
#define), and then clean up the #ifdefs on the chromium side.

[ajuma, 2014/03/03 15:30:27]

Actually, ignore the "this will have no implementation anywhere" part of this
comment, sorry. But I still suggest changing the order of the CLs to address the
Windows build error.

[mithro-old, 2014/03/04 01:39:18]

Adding BLINK_PLATFORM_EXPORT should fix windows. Testing now.

+    virtual void notifyAnimationFinished(double monotonicTime, WebAnimation::TargetProperty) = 0;
 };
 
 } // namespace blink
 
 #endif // WebAnimationDelegate_h