New issue
Advanced search Search tips

Issue 640222 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

page_cycler_v2 needs to add top_level tracing category for computing TimeToInteractive metric

Project Member Reported by nedngu...@google.com, Aug 23 2016

Issue description

I just run a telemetry job to gather TimeToInteractive metrics on top1k sites & realize that almost all the timeInteractiveMetric are the same as FirstMeaningfulPaint.

The bug turns out to be that  timeInteractiveMetric relies on "TaskQueueManager::ProcessTaskFromWorkQueues" slices which requires toplevel tracing category.

I check page_cycler_v2 benchmarks & found that we don't specify toplevel  tracing cateogry there either. 
 
Allisonb...@hotmail.com
Project Member

Comment 2 by sheriffbot@chromium.org, Aug 24 2016

Labels: Hotlist-Google
Status: Assigned (was: Untriaged)
Project Member

Comment 4 by sheriffbot@chromium.org, Aug 25 2016

Labels: Hotlist-Google
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 25 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/54a95838ec059e34f9f8e8952a1eb484998e5102

commit 54a95838ec059e34f9f8e8952a1eb484998e5102
Author: kouhei <kouhei@chromium.org>
Date: Thu Aug 25 22:50:07 2016

[PCv2] Enable toplevel tracing category

BUG= 640222 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

Review-Url: https://codereview.chromium.org/2273243002
Cr-Commit-Position: refs/heads/master@{#414568}

[modify] https://crrev.com/54a95838ec059e34f9f8e8952a1eb484998e5102/tools/perf/benchmarks/page_cycler_v2.py

Status: Fixed (was: Assigned)

Sign in to add a comment