Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(714)

Unified Diff: third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc

Issue 2909293002: Allow tasks scheduled for the instant virtual time budges expires to run (Closed)
Patch Set: Fix Fix DevTools test Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc
diff --git a/third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc b/third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc
index f6b97061560020aeb42b1cecdba85c84b35f1335..73141792934ddb7bb960d7856067b3fc2e94f410 100644
--- a/third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc
+++ b/third_party/WebKit/Source/platform/scheduler/renderer/web_view_scheduler_impl.cc
@@ -191,11 +191,11 @@ void WebViewSchedulerImpl::DisableVirtualTimeForTesting() {
void WebViewSchedulerImpl::ApplyVirtualTimePolicyToTimers() {
if (!virtual_time_ || allow_virtual_time_to_advance_) {
altimin 2017/05/31 11:44:06 This part is slightly hard to follow. Maybe someth
alex clarke (OOO till 29th) 2017/05/31 15:11:36 I've done something similar.
if (timers_suspended_) {
- renderer_scheduler_->ResumeTimerQueue();
+ renderer_scheduler_->RemoveTimerFence();
timers_suspended_ = false;
}
} else if (!timers_suspended_) {
- renderer_scheduler_->SuspendTimerQueue();
+ renderer_scheduler_->InsertTimerFence();
timers_suspended_ = true;
}
}

Powered by Google App Engine
This is Rietveld 408576698