New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 899180 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature



Sign in to add a comment

TaskCostEstimator samples Now

Project Member Reported by alexclarke@chromium.org, Oct 26

Issue description

The TaskCostEstimator uses the old base::MessageLoop::TaskObserver API
and it samples Now in WillProcessTask & DidProcessTask.

This seems wasteful since we have TaskTimeObservers.  Unfortunately
TaskTimeObservers are a SequenceManager level API but TaskCostEstimator
needs a TaskQueue level API.   Alternatively we could consider a map of
TaskQueue* to TaskCostEstimator held by MainThreadSchedulerImpl and
referenced by MainThreadSchedulerImpl::OnTaskStarted / OnTaskCompleted.
 
Status: Available (was: Untriaged)

Sign in to add a comment