Skip to content

Commit

Permalink
Remove task.scheduler tracing category from slow-reports
Browse files Browse the repository at this point in the history
This category was not useful. It's adding a trace event for each
top-level post-task, which is about doubling the size of a trace.

Let remove it.

R=oysteine@chromium.org

Change-Id: I271f5314179e7ae6299835c36a0f92a6506eae08
Reviewed-on: https://chromium-review.googlesource.com/c/1301838
Reviewed-by: Eric Seckler <eseckler@chromium.org>
Reviewed-by: oysteine <oysteine@chromium.org>
Commit-Queue: Etienne Bergeron <etienneb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#603163}
  • Loading branch information
bergeret authored and Commit Bot committed Oct 26, 2018
1 parent 0f53027 commit 3a1d4f2
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions content/browser/tracing/background_tracing_manager_impl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -676,7 +676,6 @@ TraceConfig BackgroundTracingManagerImpl::GetConfigForCategoryPreset(
auto config = TraceConfig(
"benchmark,toplevel,ipc,base,browser,navigation,omnibox,ui,shutdown,"
"safe_browsing,task_scheduler,"
"disabled-by-default-task_scheduler_diagnostics,"
"disabled-by-default-system_stats,disabled-by-default-cpu_profiler",
record_mode);
// Filter only browser process events.
Expand All @@ -692,7 +691,6 @@ TraceConfig BackgroundTracingManagerImpl::GetConfigForCategoryPreset(
"disabled-by-default-v8.gc,"
"disabled-by-default-blink_gc,"
"disabled-by-default-renderer.scheduler,"
"disabled-by-default-task_scheduler_diagnostics,"
"disabled-by-default-system_stats,disabled-by-default-cpu_profiler",
record_mode);
case BackgroundTracingConfigImpl::CategoryPreset::BLINK_STYLE:
Expand Down

0 comments on commit 3a1d4f2

Please sign in to comment.