Commit 738648ed authored by Minoru Chikamune's avatar Minoru Chikamune Committed by Commit Bot

[MBI] Trace each AgentGroupScheduler scope separately

Previously, each AgentGroupScheduler trace was not separated.
With this CL, each AgentGroupScheduler trace will be recorded separately.

Bug: 1105403
Change-Id: Ifb95ff2ff714fc281a6fca7a9903ba33b50b3186
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2440317Reviewed-by: default avatarTal Pressman <talp@chromium.org>
Reviewed-by: default avatarKentaro Hara <haraken@chromium.org>
Reviewed-by: default avatarAlexander Timin <altimin@chromium.org>
Commit-Queue: Minoru Chikamune <chikamune@chromium.org>
Cr-Commit-Position: refs/heads/master@{#814498}
parent 08f4d6ff
...@@ -19,6 +19,7 @@ ...@@ -19,6 +19,7 @@
#include "base/optional.h" #include "base/optional.h"
#include "base/strings/string_number_conversions.h" #include "base/strings/string_number_conversions.h"
#include "base/threading/thread_task_runner_handle.h" #include "base/threading/thread_task_runner_handle.h"
#include "base/trace_event/common/trace_event_common.h"
#include "base/trace_event/trace_event.h" #include "base/trace_event/trace_event.h"
#include "base/trace_event/traced_value.h" #include "base/trace_event/traced_value.h"
#include "build/build_config.h" #include "build/build_config.h"
...@@ -2357,19 +2358,25 @@ void MainThreadSchedulerImpl::SetCurrentAgentGroupScheduler( ...@@ -2357,19 +2358,25 @@ void MainThreadSchedulerImpl::SetCurrentAgentGroupScheduler(
AgentGroupSchedulerImpl* agent_group_scheduler_impl) { AgentGroupSchedulerImpl* agent_group_scheduler_impl) {
helper_.CheckOnValidThread(); helper_.CheckOnValidThread();
if (current_agent_group_scheduler_) { if (current_agent_group_scheduler_) {
TRACE_EVENT_NESTABLE_ASYNC_END0( TRACE_EVENT_NESTABLE_ASYNC_END1(
TRACE_DISABLED_BY_DEFAULT("renderer.scheduler"), "ASG_scope", this); TRACE_DISABLED_BY_DEFAULT("renderer.scheduler"),
"scheduler.agent_scope", current_agent_group_scheduler_,
"agent_group_scheduler", current_agent_group_scheduler_);
} else { } else {
TRACE_EVENT_NESTABLE_ASYNC_END0( TRACE_EVENT_NESTABLE_ASYNC_END0(
TRACE_DISABLED_BY_DEFAULT("renderer.scheduler"), "MTS_scope", this); TRACE_DISABLED_BY_DEFAULT("renderer.scheduler"),
"scheduler.thread_scope", this);
} }
current_agent_group_scheduler_ = agent_group_scheduler_impl; current_agent_group_scheduler_ = agent_group_scheduler_impl;
if (current_agent_group_scheduler_) { if (current_agent_group_scheduler_) {
TRACE_EVENT_NESTABLE_ASYNC_BEGIN0( TRACE_EVENT_NESTABLE_ASYNC_BEGIN1(
TRACE_DISABLED_BY_DEFAULT("renderer.scheduler"), "ASG_scope", this); TRACE_DISABLED_BY_DEFAULT("renderer.scheduler"),
"scheduler.agent_scope", current_agent_group_scheduler_,
"agent_group_scheduler", current_agent_group_scheduler_);
} else { } else {
TRACE_EVENT_NESTABLE_ASYNC_BEGIN0( TRACE_EVENT_NESTABLE_ASYNC_BEGIN0(
TRACE_DISABLED_BY_DEFAULT("renderer.scheduler"), "MTS_scope", this); TRACE_DISABLED_BY_DEFAULT("renderer.scheduler"),
"scheduler.thread_scope", this);
} }
} }
......
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