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

Unified Diff: components/scheduler/renderer/renderer_scheduler_impl.cc

Issue 1636513003: Remove the code that prevents task blocking if not expecting frames (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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
« no previous file with comments | « no previous file | components/scheduler/renderer/renderer_scheduler_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/scheduler/renderer/renderer_scheduler_impl.cc
diff --git a/components/scheduler/renderer/renderer_scheduler_impl.cc b/components/scheduler/renderer/renderer_scheduler_impl.cc
index 7334f743ab705546a41af5e11aab06d33f8224ec..2ea3db6a4f9c7c11eb93500324fb6962bbb66275 100644
--- a/components/scheduler/renderer/renderer_scheduler_impl.cc
+++ b/components/scheduler/renderer/renderer_scheduler_impl.cc
@@ -632,16 +632,12 @@ void RendererSchedulerImpl::UpdatePolicyLocked(UpdateType update_type) {
bool loading_tasks_seem_expensive = false;
bool timer_tasks_seem_expensive = false;
- // Only deem tasks to be exensive (which may cause them to be preemptively
- // blocked) if we are expecting frames.
- if (!MainThreadOnly().begin_frame_not_expected_soon) {
- loading_tasks_seem_expensive =
- MainThreadOnly().loading_task_cost_estimator.expected_task_duration() >
- longest_jank_free_task_duration;
- timer_tasks_seem_expensive =
- MainThreadOnly().timer_task_cost_estimator.expected_task_duration() >
- longest_jank_free_task_duration;
- }
+ loading_tasks_seem_expensive =
+ MainThreadOnly().loading_task_cost_estimator.expected_task_duration() >
+ longest_jank_free_task_duration;
+ timer_tasks_seem_expensive =
+ MainThreadOnly().timer_task_cost_estimator.expected_task_duration() >
+ longest_jank_free_task_duration;
MainThreadOnly().timer_tasks_seem_expensive = timer_tasks_seem_expensive;
MainThreadOnly().loading_tasks_seem_expensive = loading_tasks_seem_expensive;
« no previous file with comments | « no previous file | components/scheduler/renderer/renderer_scheduler_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698