diff options
author | miletus <miletus@chromium.org> | 2015-04-22 17:07:02 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-04-23 00:08:10 +0000 |
commit | 7f9fd6071aeb387d5c98ca839ac54af555ba2c34 (patch) | |
tree | 6d296022fe1e8abb55a26f131357c0b551ba7911 /base | |
parent | db04346c255d4391728a541b95594d6f8a503f60 (diff) | |
download | chromium_src-7f9fd6071aeb387d5c98ca839ac54af555ba2c34.zip chromium_src-7f9fd6071aeb387d5c98ca839ac54af555ba2c34.tar.gz chromium_src-7f9fd6071aeb387d5c98ca839ac54af555ba2c34.tar.bz2 |
Append input event type as trace event's name
Previously to show input event's type in the InputLatency
trace event, we issue an async step event with the
input event type as the step event's name. However, trace
viewer's UI for showing async step event is confusing that
the step event is stacked under its parent async event.
E.g. the InputLatency for a GestureScrollUpdate trace event
will be shown as an aysnc trace event with name "InputLatency"
and then a separate async step event with name "GestureScrollUpdate".
To make it cleaner, this CL removes the async step event, and
append the event type to the name of the InputLatency's async
event. So for the above example, there will only be one async
trace event with name "InputLatency::GestureScrollUpdate".
BUG=472121
Review URL: https://codereview.chromium.org/1089243002
Cr-Commit-Position: refs/heads/master@{#326410}
Diffstat (limited to 'base')
-rw-r--r-- | base/trace_event/trace_event.h | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/base/trace_event/trace_event.h b/base/trace_event/trace_event.h index 79ec4a8..e0249f5 100644 --- a/base/trace_event/trace_event.h +++ b/base/trace_event/trace_event.h @@ -601,6 +601,12 @@ TRACE_EVENT_PHASE_ASYNC_BEGIN, category_group, name, id, \ static_cast<int>(base::PlatformThread::CurrentId()), \ timestamp, TRACE_EVENT_FLAG_NONE) +#define TRACE_EVENT_COPY_ASYNC_BEGIN_WITH_TIMESTAMP0(category_group, \ + name, id, timestamp) \ + INTERNAL_TRACE_EVENT_ADD_WITH_ID_TID_AND_TIMESTAMP( \ + TRACE_EVENT_PHASE_ASYNC_BEGIN, category_group, name, id, \ + static_cast<int>(base::PlatformThread::CurrentId()), \ + timestamp, TRACE_EVENT_FLAG_COPY) // Records a single ASYNC_STEP_INTO event for |step| immediately. If the // category is not enabled, then this does nothing. The |name| and |id| must |