Refactor TimeDomain::MaybeAdvanceTime in preparation for DoWork deduping

third_party/WebKit/Source/platform/scheduler/renderer/throttled_time_domain.cc

Index: third_party/WebKit/Source/platform/scheduler/renderer/throttled_time_domain.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/renderer/throttled_time_domain.cc b/third_party/WebKit/Source/platform/scheduler/renderer/throttled_time_domain.cc
index 8f6887cbbf7945f4728d5260322c13c40756a69c..db1d088798fc7e38e84f2f2b509b8e6d243bfa4c 100644
--- a/third_party/WebKit/Source/platform/scheduler/renderer/throttled_time_domain.cc
+++ b/third_party/WebKit/Source/platform/scheduler/renderer/throttled_time_domain.cc
@@ -23,19 +23,20 @@ void ThrottledTimeDomain::RequestWakeup(base::TimeTicks now,
   // behalf.
 }
 
-bool ThrottledTimeDomain::MaybeAdvanceTime() {
+base::Optional<base::TimeDelta> ThrottledTimeDomain::DelayTillNextTask(
+    LazyNow* lazy_now) {
   base::TimeTicks next_run_time;
   if (!NextScheduledRunTime(&next_run_time))
-    return false;
+    return base::Optional<base::TimeDelta>();
 
-  base::TimeTicks now = Now();
+  base::TimeTicks now = lazy_now->Now();
   if (now >= next_run_time)
-    return true;  // Causes DoWork to post a continuation.
+    return base::TimeDelta();  // Makes DoWork post an immediate continuation.
 
-  // Unlike RealTimeDomain::MaybeAdvanceTime we don't request a wake up here, we
-  // assume the owner (i.e. TaskQueueThrottler) will manage wakeups on our
+  // Unlike RealTimeDomain::ContinuationNeeded we don't request a wake up here,

[Sami, 2017/01/19 15:22:29]

ContinuationNeeded doesn't seem to exist?

[alex clarke (OOO till 29th), 2017/01/19 15:51:03]

Fixed the comment.

+  // we assume the owner (i.e. TaskQueueThrottler) will manage wakeups on our
   // behalf.
-  return false;
+  return base::Optional<base::TimeDelta>();
 }
 
 }  // namespace scheduler