tree 195b297bf0df92dc2ca4049ca792ddbff95022eb
parent d06e7a567f592167a275f49af4e58fb723a5f705
author Jakob Johnson <johnsonjakob99@gmail.com> 1659364487 -0700
committer Copybara-Service <copybara-worker@google.com> 1659387360 -0700

[trace] Replace TraceCursorUP with TraceCursorSP

The use of `std::unique_ptr` with `TraceCursor` adds unnecessary complexity to adding `SBTraceCursor` bindings
Specifically, since `TraceCursor` is an abstract class there's no clean
way to provide "deep clone" semantics for `TraceCursorUP` short of
creating a pure virtual `clone()` method (afaict).

After discussing with @wallace, we decided there is no strong reason to
favor wrapping `TraceCursor` with `std::unique_ptr` over `std::shared_ptr`, thus this diff
replaces all usages of `std::unique_ptr<TraceCursor>` with `std::shared_ptr<TraceCursor>`.

This sets the stage for future diffs to introduce `SBTraceCursor`
bindings in a more clean fashion.

Test Plan:

Differential Revision: https://reviews.llvm.org/D130925

GitOrigin-RevId: 3bec33b16db11c67d43bda134520a2132ff606c9