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

Issue 784438 link

Starred by 1 user
Status: Fixed
Owner:
█ kraynov@chromium.org
Xoogler
Closed: Jan 2018
Cc:
altimin@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature



Sign in to add a comment

[Blink Scheduler] Remove redundant tracing counters (or convert to TraceableValue)

Project Member Reported by kraynov@chromium.org, Nov 13 2017 
E.g. use_case "counter" is redundant since RendererScheduler.UseCase has been exposed.

Comment 1 by kraynov@chromium.org, Nov 14 2017 

Example trace for https://chromium-review.googlesource.com/768809
trace_Tue_Nov_14_2017_1.48.16_PM.json.gz
1.6 MB Download
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 14 2017 

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

commit dedcfad5c2f04cb6d02e7a34e1a54c2f647355ad
Author: Greg Kraynov <kraynov@chromium.org>
Date: Tue Nov 14 18:50:14 2017

Blink Scheduler: Convert counters to TraceableState.

Now use_case, rail_mode and expensive_task_policy are easier to read.

Bug:  784438 
Change-Id: Ie07aaa06c3ad7c6dd7c37cc097d087e26050a34b
Reviewed-on: https://chromium-review.googlesource.com/768809
Commit-Queue: Greg Kraynov <kraynov@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516368}
[modify] https://crrev.com/dedcfad5c2f04cb6d02e7a34e1a54c2f647355ad/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc
[modify] https://crrev.com/dedcfad5c2f04cb6d02e7a34e1a54c2f647355ad/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.h

Comment 3 by kraynov@chromium.org, Jan 30 2018

Status: Fixed (was: Assigned)
►

Sign in to add a comment