Perfetto: Fixed a crash when the Mojo binding got torn down before the interfaceptr The ProducerHost would reset the producer_endpoint_ member on connection errors, but as these are connection errors on the InterfacePtr (i.e. outgoing calls) rather than the Mojo binding (owned by a StrongBindingSet in the PerfettoService), additional calls could still happen and would crash when accessing the already-reset producer_endpoint_. The only reason to reset this member is as a safeguard against edgecases where callbacks come from Perfetto while the ProducerHost is in a half-destructed state; we can accomplish the same by just manually resetting the producer_endpoint_ in the destructor. R=ssid@chromium.org Bug: 945982, 947449 Change-Id: I44c54f4409ab384243b89ab161b171477849849b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545082 Auto-Submit: oysteine <oysteine@chromium.org> Reviewed-by: ssid <ssid@chromium.org> Commit-Queue: oysteine <oysteine@chromium.org> Cr-Commit-Position: refs/heads/master@{#646032}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .