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

Issue 2024823002: scheduler: Avoid false alerts about expensive task blocking (Closed)

Created:
4 years, 6 months ago by Sami
Modified:
4 years, 6 months ago
Reviewers:
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@2743
Target Ref:
refs/pending/branch-heads/2743
Project:
chromium
Visibility:
Public.

Description

scheduler: Avoid false alerts about expensive task blocking This patch adjusts the logic for logging a console message when an expensive task is blocked: 1. Don't log messages when all timer tasks are blocked (e.g., while waiting for a touchstart response). There's nothing the developer can do to avoid this type of blocking. 2. Only report blocking for tasks which exceed the 50ms RAIL threshold. These types of tasks are most likely to cause user visible jank. Lower blocking thresholds are also used for shorter periods of time, so shorter tasks are not likely to be deferred for a long time in any case. 3. Also mention network tasks in the log message. BUG=613230 Review-Url: https://codereview.chromium.org/2016943003 Cr-Commit-Position: refs/heads/master@{#396467} (cherry picked from commit c66723959e9a37f1a5bbd821366a2ee24a9537c2) Committed: https://chromium.googlesource.com/chromium/src/+/72912f97a2fd244ce81b7720694fa34a159d8b22

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+57 lines, -2 lines) Patch
M components/scheduler/renderer/renderer_scheduler_impl.cc View 2 chunks +9 lines, -2 lines 0 comments Download
M components/scheduler/renderer/renderer_scheduler_impl_unittest.cc View 1 chunk +48 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
Sami
4 years, 6 months ago (2016-05-31 09:27:35 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
72912f97a2fd244ce81b7720694fa34a159d8b22.

Powered by Google App Engine
This is Rietveld 408576698