Remove WTF::currentTime() / WTF::monotonicallyIncreasingTime() uses from cc

Remove WTF::currentTime() / WTF::monotonicallyIncreasingTime() uses from cc

These are implemented on top of WebKit::Platform via webkit_wtf_support and need
to go.

BUG=154451
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=164580

[jamesr, 2012-10-18 23:27:40]

There are a handful of isolated CurrentTime.h that aren't related to this. 
After I kill those, I believe I can break our dependency on webkit_wtf_support
completely!

[jamesr, 2012-10-18 23:31:13]

Looks like I need a rebase.

FYI, the whole animation system needs to be rewritten to use base::Time concepts
instead of doubles, but this patch was already getting huge so I left it alone. 
The animation system itself never queries the current time - it relies on
passed-in values - so the seperation isn't too horrible.

[enne (OOO), 2012-10-19 05:18:32]

lgtm

[commit-bot: I haz the power, 2012-10-19 05:19:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamesr@chromium.org/11191068/2012

[commit-bot: I haz the power, 2012-10-19 05:19:32]

Failed to apply patch for cc/layer_tree_host.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file cc/layer_tree_host.cc
  Hunk #4 FAILED at 775.
  1 out of 6 hunks FAILED -- saving rejects to file cc/layer_tree_host.cc.rej

Patch:       cc/layer_tree_host.cc
Index: cc/layer_tree_host.cc
diff --git a/cc/layer_tree_host.cc b/cc/layer_tree_host.cc
index
f7cdfbe06fbc5473905994b5047edf4c8e2542a8..35267d7f63ea1ac5d2713593a04c7f2e425a4f96
100644
--- a/cc/layer_tree_host.cc
+++ b/cc/layer_tree_host.cc
@@ -222,11 +222,11 @@ void CCLayerTreeHost::acquireLayerTextures()
     m_proxy->acquireLayerTextures();
 }
 
-void CCLayerTreeHost::updateAnimations(double monotonicFrameBeginTime)
+void CCLayerTreeHost::updateAnimations(base::TimeTicks frameBeginTime)
 {
     m_animating = true;
-    m_client->animate(monotonicFrameBeginTime);
-    animateLayers(monotonicFrameBeginTime);
+    m_client->animate((frameBeginTime - base::TimeTicks()).InSecondsF());
+    animateLayers(frameBeginTime);
     m_animating = false;
 
     m_renderingStats.numAnimationFrames++;
@@ -377,7 +377,7 @@ bool CCLayerTreeHost::commitRequested() const
     return m_proxy->commitRequested();
 }
 
-void CCLayerTreeHost::setAnimationEvents(scoped_ptr<CCAnimationEventsVector>
events, double wallClockTime)
+void CCLayerTreeHost::setAnimationEvents(scoped_ptr<CCAnimationEventsVector>
events, base::Time wallClockTime)
 {
     DCHECK(CCThreadProxy::isMainThread());
     setAnimationEventsRecursive(*events.get(), m_rootLayer.get(),
wallClockTime);
@@ -436,9 +436,9 @@ void CCLayerTreeHost::setVisible(bool visible)
     m_proxy->setVisible(visible);
 }
 
-void CCLayerTreeHost::startPageScaleAnimation(const IntSize& targetPosition,
bool useAnchor, float scale, double durationSec)
+void CCLayerTreeHost::startPageScaleAnimation(const IntSize& targetPosition,
bool useAnchor, float scale, base::TimeDelta duration)
 {
-    m_proxy->startPageScaleAnimation(targetPosition, useAnchor, scale,
durationSec);
+    m_proxy->startPageScaleAnimation(targetPosition, useAnchor, scale,
duration);
 }
 
 void CCLayerTreeHost::loseContext(int numTimes)
@@ -775,22 +775,23 @@ void CCLayerTreeHost::setDeviceScaleFactor(float
deviceScaleFactor)
     setNeedsCommit();
 }
 
-void CCLayerTreeHost::animateLayers(double monotonicTime)
+void CCLayerTreeHost::animateLayers(base::TimeTicks time)
 {
     if (!CCSettings::acceleratedAnimationEnabled() || !m_needsAnimateLayers)
         return;
 
     TRACE_EVENT0("cc", "CCLayerTreeHostImpl::animateLayers");
-    m_needsAnimateLayers = animateLayersRecursive(m_rootLayer.get(),
monotonicTime);
+    m_needsAnimateLayers = animateLayersRecursive(m_rootLayer.get(), time);
 }
 
-bool CCLayerTreeHost::animateLayersRecursive(LayerChromium* current, double
monotonicTime)
+bool CCLayerTreeHost::animateLayersRecursive(LayerChromium* current,
base::TimeTicks time)
 {
     if (!current)
         return false;
 
     bool subtreeNeedsAnimateLayers = false;
     CCLayerAnimationController* currentController =
current->layerAnimationController();
+    double monotonicTime = (time - base::TimeTicks()).InSecondsF();
     currentController->animate(monotonicTime, 0);
 
     // If the current controller still has an active animation, we must
continue animating layers.
@@ -798,14 +799,14 @@ bool
CCLayerTreeHost::animateLayersRecursive(LayerChromium* current, double mono
          subtreeNeedsAnimateLayers = true;
 
     for (size_t i = 0; i < current->children().size(); ++i) {
-        if (animateLayersRecursive(current->children()[i].get(),
monotonicTime))
+        if (animateLayersRecursive(current->children()[i].get(), time))
             subtreeNeedsAnimateLayers = true;
     }
 
     return subtreeNeedsAnimateLayers;
 }
 
-void CCLayerTreeHost::setAnimationEventsRecursive(const
CCAnimationEventsVector& events, LayerChromium* layer, double wallClockTime)
+void CCLayerTreeHost::setAnimationEventsRecursive(const
CCAnimationEventsVector& events, LayerChromium* layer, base::Time wallClockTime)
 {
     if (!layer)
         return;
@@ -813,9 +814,9 @@ void CCLayerTreeHost::setAnimationEventsRecursive(const
CCAnimationEventsVector&
     for (size_t eventIndex = 0; eventIndex < events.size(); ++eventIndex) {
         if (layer->id() == events[eventIndex].layerId) {
             if (events[eventIndex].type == CCAnimationEvent::Started)
-                layer->notifyAnimationStarted(events[eventIndex],
wallClockTime);
+                layer->notifyAnimationStarted(events[eventIndex],
wallClockTime.ToDoubleT());
             else
-                layer->notifyAnimationFinished(wallClockTime);
+                layer->notifyAnimationFinished(wallClockTime.ToDoubleT());
         }
     }

[commit-bot: I haz the power, 2012-10-28 19:53:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamesr@chromium.org/11191068/8001

[commit-bot: I haz the power, 2012-10-29 00:06:32]

Change committed as 164580