Commit 327e1f37 authored by Etienne Bergeron's avatar Etienne Bergeron Committed by Commit Bot

Filter browser process events in navigation slow-reports

This CL is adding a tracing filter to only keep browser process events.
For that scenario, we only care about browser process. There is no need
to collect renderers/GPU events.

R=ssid@chromium.org, oysteine@chromium.org

Change-Id: Ifd0538df42c8f237da23ebc6a082e57dbfab9056
Reviewed-on: https://chromium-review.googlesource.com/1187505Reviewed-by: default avataroysteine <oysteine@chromium.org>
Reviewed-by: default avatarSiddhartha S <ssid@chromium.org>
Commit-Queue: Etienne Bergeron <etienneb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#586290}
parent 223b5eef
...@@ -662,11 +662,17 @@ TraceConfig BackgroundTracingManagerImpl::GetConfigForCategoryPreset( ...@@ -662,11 +662,17 @@ TraceConfig BackgroundTracingManagerImpl::GetConfigForCategoryPreset(
case BackgroundTracingConfigImpl::CategoryPreset:: case BackgroundTracingConfigImpl::CategoryPreset::
BENCHMARK_EXECUTION_METRIC: BENCHMARK_EXECUTION_METRIC:
return TraceConfig("blink.console,v8", record_mode); return TraceConfig("blink.console,v8", record_mode);
case BackgroundTracingConfigImpl::CategoryPreset::BENCHMARK_NAVIGATION: case BackgroundTracingConfigImpl::CategoryPreset::BENCHMARK_NAVIGATION: {
return TraceConfig( auto config = TraceConfig(
"benchmark,toplevel,ipc,base,browser,navigation,omnibox," "benchmark,toplevel,ipc,base,browser,navigation,omnibox,"
"safe_browsing,disabled-by-default-system_stats", "safe_browsing,disabled-by-default-system_stats",
record_mode); record_mode);
// Filter only browser process events.
base::trace_event::TraceConfig::ProcessFilterConfig process_config(
{base::GetCurrentProcId()});
config.SetProcessFilterConfig(process_config);
return config;
}
case BackgroundTracingConfigImpl::CategoryPreset::BLINK_STYLE: case BackgroundTracingConfigImpl::CategoryPreset::BLINK_STYLE:
return TraceConfig("blink_style", record_mode); return TraceConfig("blink_style", record_mode);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment