Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Reland "[segmentation_platform] Fix nesting of model execution trace …
…events" This reverts commit c52c5f7. Currently the ModelExecutionImpl::ExecuteModel trace event would finish after its parent ModelExecutionImpl::ExecutionState event. The parent is closed within RunModelExecutionCallback, whereas the child is closed when ModelExecutionImpl::ExecuteModel ends since it is on the stack. This could lead to confusing timestamps in the trace log. This CL changes the implementation of this to ensure that the order of closing the events is correct. Original CL: https://chromium-review.googlesource.com/c/chromium/src/+/4098382 Changes from original CL: - perfetto::Track is no longer used when looking up trace tracks. (cherry picked from commit 6e61ffa) Bug: 1393919, 1401661 Change-Id: I17549dc8a740c0c7497a6c86e168cf3b893e6b72 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4114468 Reviewed-by: Siddhartha S <ssid@chromium.org> Commit-Queue: Tommy Nyquist <nyquist@chromium.org> Cr-Original-Commit-Position: refs/heads/main@{#1084691} Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4148308 Commit-Queue: Krishna Govind <govind@chromium.org> Reviewed-by: Eric Seckler <eseckler@chromium.org> Cr-Commit-Position: refs/branch-heads/5481@{#245} Cr-Branched-From: 130f3e4-refs/heads/main@{#1084008}
- Loading branch information