tree 9704fe024f8519f211171333dcaf95576515478f
parent d55ad99af9759c6eeeba3290226b463a4655bce7
author Oystein Eftevaag <oysteine@chromium.org> 1557938415 +0000
committer Commit Bot <commit-bot@chromium.org> 1557938415 +0000

Reland "Background tracing: Use the new Perfetto Consumer mojo endpoint"

This reverts commit ff99845cd65afe2e9c574bf7bfb5a40147d21d52.

Reason for revert: Relanding with fix for use-after-free crashes.

Original change's description:
> Revert "Reland "Background tracing: Use the new Perfetto Consumer mojo endpoint""
>
> This reverts commit 996da9c1dd4245041e74370782197b18f94e2d05.
>
> Reason for revert: Revering due to https://crbug.com/962478
>
> Original change's description:
> > Reland "Background tracing: Use the new Perfetto Consumer mojo endpoint"
> >
> > This reverts commit 5116072f4c3af4f01c04c70abaa71163653256e1.
> >
> > Reason for revert: Relanding with fix for fully waiting for trace shutdown.
> >
> > Original change's description:
> > > Revert "Background tracing: Use the new Perfetto Consumer mojo endpoint"
> > >
> > > This reverts commit 67f4c2f3d0643b2cdbbff06241fec7b531287e6e.
> > >
> > > Reason for revert: Revert while investigating new TSan warnings
> > >
> > > Original change's description:
> > > > Background tracing: Use the new Perfetto Consumer mojo endpoint
> > > > rather than the TracingController.
> > > >
> > > > This is in preparation for being able to handle proto traces
> > > > in addition to JSON.
> > > >
> > > > R=​ssid@chromium.org
> > > > BUG=925151
> > > >
> > > > Change-Id: I91ed3c9648587fb19a5102b869a5d5ab688e0187
> > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546146
> > > > Commit-Queue: oysteine <oysteine@chromium.org>
> > > > Reviewed-by: Avi Drissman <avi@chromium.org>
> > > > Reviewed-by: ssid <ssid@chromium.org>
> > > > Cr-Commit-Position: refs/heads/master@{#657413}
> > >
> > > TBR=avi@chromium.org,oysteine@chromium.org,ssid@chromium.org
> > >
> > > Change-Id: I92db1fa46353cfb960071f44d110a5faa0c25421
> > > No-Presubmit: true
> > > No-Tree-Checks: true
> > > No-Try: true
> > > Bug: 925151
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1600305
> > > Reviewed-by: oysteine <oysteine@chromium.org>
> > > Commit-Queue: oysteine <oysteine@chromium.org>
> > > Cr-Commit-Position: refs/heads/master@{#657520}
> >
> > TBR=avi@chromium.org,oysteine@chromium.org,ssid@chromium.org
> >
> > Change-Id: Ic3cc44a563a98871de8003221cb015fbc3b890d0
> > Bug: 925151
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1601555
> > Reviewed-by: oysteine <oysteine@chromium.org>
> > Reviewed-by: ssid <ssid@chromium.org>
> > Reviewed-by: Avi Drissman <avi@chromium.org>
> > Commit-Queue: oysteine <oysteine@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#657925}
>
> TBR=avi@chromium.org,oysteine@chromium.org,ssid@chromium.org
>
> # Not skipping CQ checks because original CL landed > 1 day ago.
>
> Bug: 925151
> Change-Id: Id214d9547e18cb35b5ecf987f611b204944e51a7
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1610301
> Reviewed-by: oysteine <oysteine@chromium.org>
> Commit-Queue: oysteine <oysteine@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#659204}

TBR=avi@chromium.org,oysteine@chromium.org,ssid@chromium.org

Change-Id: Ic0495ca0a4a245a326225b0c68bc69a84b30ff06
Bug: 925151
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1610312
Reviewed-by: ssid <ssid@chromium.org>
Reviewed-by: oysteine <oysteine@chromium.org>
Commit-Queue: oysteine <oysteine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#659970}