Reland of [tracing] Adding task information to heap profiler (patchset #1 id:1 of https://codereview.chromium.org/1846333002/ )
Reason for revert:
Fixing the flakiness in trace_test by removing the event from
sequenced_worker_pool.cc. 2 trace events together somehow causes multiple
calls to UpdateDuration. I couldn't reproduce yet.
Also because 2 trace events there is not necessary, I will figure out a
way to add WorkerPool task information in next CL.
Original issue's description:
> Revert of [tracing] Adding task information to heap profiler (patchset #18 id:630001 of https://codereview.chromium.org/1784133002/ )
>
> Reason for revert:
> Caused flakiness of trace_test; see Issue 599794.
>
> Original issue's description:
> > [tracing] Adding task information to heap profiler
> >
> > This CL adds more information to the malloc heap profiler added in
> > crrev.com/1675183006. The heap profiler only shows trace events and all
> > the tasks show up as "MessageLoop::Runtask".
> >
> > This CL does:
> > - Moves TRACE_TASK_EXECUTION macro from task_annotator to
> > trace_event.h.
> > - Adds extra scoped trace event to track the task context in heap
> > profiler.
> > - The folder where the task was posted from is used to categorize
> > allocations.
> > - Uses the type_name for context temporarily till we define a new
> > context dimension for allocations.
> >
> > BUG=594803
> >
> > Committed: https://crrev.com/533c402f8b21e503ebcaf3f0de427f41076b4de1
> > Cr-Commit-Position: refs/heads/master@{#384521}
>
> TBR=jochen@chromium.org,dskiba@google.com,oysteine@chromium.org,petrcermak@chromium.org,primiano@chromium.org,skyostil@chromium.org,thakis@chromium.org,ssid@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=594803
>
> Committed: https://crrev.com/7a3a6793b5a471280e64842060d565e6f52e939f
> Cr-Commit-Position: refs/heads/master@{#384695}
TBR=jochen@chromium.org,dskiba@google.com,oysteine@chromium.org,petrcermak@chromium.org,primiano@chromium.org,skyostil@chromium.org,thakis@chromium.org,kbr@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
BUG=594803
Review URL: https://codereview.chromium.org/1851233002
Cr-Original-Commit-Position: refs/heads/master@{#384795}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: 092c384b364d376d8ef4971cf43a5b11aa2884cf
1 file changed
