[scheduler] Speculative fix for crbug.com/813218
Bug 813218 sounds like we fail to schedule PumpThrottledTasks and
throttling machinery grinds to a halt. Reduce the probability of this
by ensuring that a call to PumpThrottledTasks is always scheduled.
R=skyostil@chromium.org
TBR=altimin@chromium.org
BUG=813218
(cherry picked from commit 5a8814f02d665603fea6dc6372005eef196275f6)
Change-Id: Ie963d1eae456e39266bc8b347fdae0fa2bd37e7e
Reviewed-on: https://chromium-review.googlesource.com/946588
Commit-Queue: Alexander Timin <altimin@chromium.org>
Reviewed-by: Sami Kyöstilä <skyostil@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#540642}
Reviewed-on: https://chromium-review.googlesource.com/948902
Reviewed-by: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/branch-heads/3325@{#656}
Cr-Branched-From: bc084a8b5afa3744a74927344e304c02ae54189f-refs/heads/master@{#530369}diff --git a/third_party/WebKit/Source/platform/scheduler/renderer/task_queue_throttler.cc b/third_party/WebKit/Source/platform/scheduler/renderer/task_queue_throttler.cc
index b76bd64..cc562ee 100644
--- a/third_party/WebKit/Source/platform/scheduler/renderer/task_queue_throttler.cc
+++ b/third_party/WebKit/Source/platform/scheduler/renderer/task_queue_throttler.cc
@@ -353,7 +353,9 @@
base::Optional<base::TimeTicks> next_wake_up =
queue->GetNextScheduledWakeUp();
- if (next_wake_up && next_wake_up.value() > unblock_until.value()) {
+ // TODO(altimin, crbug.com/813218): Find a testcase to repro freezes
+ // mentioned in the bug.
+ if (next_wake_up) {
MaybeSchedulePumpThrottledTasks(
FROM_HERE, now, GetNextAllowedRunTime(queue, next_wake_up.value()));
}
