DescriptionRevert of Add more trace events to "benchmark" category (patchset #4 id:50001 of https://codereview.chromium.org/855553002/)
Reason for revert:
This CL has perf implication and currently all the android perf bots are red.
So lets delay the landing until we get perf bots back to normal
Original issue's description:
> Add more trace events to "benchmark" category
>
> This is to make the trace events captured during smoothness test more
> useful in debugging smoothness/latency issues.
>
> These added trace events are supposed to be giving most useful information
> about the input/scheduling/rendering pipeline and have low occurrence in
> each frame so as to minimize the performance overhead.
>
> Following are the list for the added trace events roughly in the order
> of the input->rendering pipeline:
>
> LatencyInfo.Flow // so we can follow the input latency flow
> RenderWidgetHostViewAndroid::OnVSync // when frame is started
> InputHandlerProxy::HandleInputEvent // when the input event is handled on impl
> RenderWidget::OnHandleInputEvent // when the input event is handled on main
> Scheduler::BeginFrame // cc decides to schedule a frame
> Scheduler::BeginRetroFrame // cc starts a retro frame
> Scheduler::BeginImplFrame // impl frame is scheduled
> Scheduler::OnBeginImplFrameDeadline // impl frame actually starts
> OnSwapCompositorFrame // browser gets notified that renderer frame is ready
> CompositorImpl::PostComposit // browser schedules a frame
> SingleThreadProxy::CompositeImmediately // browser starts a frame
> SingleThreadProxy::DidSwapBuffersCompleteOnImplThread // browser frame is done
>
> BUG=449258
>
> Committed: https://crrev.com/0190b029a83fad58e31a039bd8bba62d9d12ee5a
> Cr-Commit-Position: refs/heads/master@{#312664}
TBR=sievers@chromium.org,brianderson@chromium.org,skyostil@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=449258
Committed: https://crrev.com/ef1c46ab8a359ae309b49b3222c10bb2ef0bcf09
Cr-Commit-Position: refs/heads/master@{#312668}
Patch Set 1 #
Created: 5 years, 11 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|