Blink Platform: Remove time functions from Platform.

content/renderer/renderer_blink_platform_impl.cc

Index: content/renderer/renderer_blink_platform_impl.cc
diff --git a/content/renderer/renderer_blink_platform_impl.cc b/content/renderer/renderer_blink_platform_impl.cc
index a47eec73c01a1f1c1c50659b8011db14bd9b7c21..663a1a4a59cfc089baedaa00f768289570a97fe1 100644
--- a/content/renderer/renderer_blink_platform_impl.cc
+++ b/content/renderer/renderer_blink_platform_impl.cc
@@ -281,16 +281,6 @@ void RendererBlinkPlatformImpl::Shutdown() {
 
 //------------------------------------------------------------------------------
 
-double RendererBlinkPlatformImpl::currentTimeSeconds() {
-  return renderer_scheduler_->CurrentTimeSeconds();
-}
-
-double RendererBlinkPlatformImpl::monotonicallyIncreasingTimeSeconds() {
-  return renderer_scheduler_->MonotonicallyIncreasingTimeSeconds();

[esprehn, 2016/01/05 07:01:09]

The scheduler implements virtual time, where is that done now?

[Yuta Kitamura, 2016/01/05 09:17:46]

Good point. This was my oversight, but after looking at some
scheduler code I started to wonder if the ability to take over
Blink's clock for schedulers was really necessary.

The production implementation of CurrentTimeSeconds() is
base::Time::Now().ToDoubleT() which is same as what my patch
does, thus the sole purpose of this hijacking is for testing.
However, according to my try results, all try bots are green.
So...?

I think I need some wisdom from the author of this code about
the real intent of this... alexclarke, WDYT?

If we really need this ability, we may have to make rather
intrusive API so the modules outside Blink can control Blink's
clock, or otherwise leave the Platform interface as is.

[alex clarke (OOO till 29th), 2016/01/05 10:49:24]

This is for Headless Chrome
(https://docs.google.com/document/d/1dIMHIl1xutUXqXWRXqXrDd3bo9hachIt_ZkPK_Bsh...)
We don't have good integration testing for this (yet) sadly.
For Headless parts of blink need the real clock still but most of it needs the
virtual clock.  I don't have a strong opinion on how this stuff should be wired
up so if there's something we can do that fits in better with Onion Soup then we
should explore that.

We'd be happy to talk about this over VC.

-}
-
-//------------------------------------------------------------------------------
-
 blink::WebURLLoader* RendererBlinkPlatformImpl::createURLLoader() {
   ChildThreadImpl* child_thread = ChildThreadImpl::current();
   // There may be no child thread in RenderViewTests.  These tests can still use