[BRP] Remove TraceLog::OptionalAutoLock exception [chromium/src : main]
0 views
Skip to first unread message
Bartek Nowierski (Gerrit)
Jun 25, 2024, 8:44:09 AM (6 days ago) Jun 25
to Stephen Nusko, Keishi Hattori, chromium...@chromium.org, Hans Wennborg, spang...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
Attention needed from Stephen Nusko
Bartek Nowierski voted![]( "Open in Gerrit")
| Auto-Submit | +1 |
| Commit-Queue | +1 |
Related details
Attention is currently required from:
- Stephen Nusko
Submit Requirements:
Code-Coverage
Code-Review
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Stephen Nusko (Gerrit)
Jun 25, 2024, 9:14:49 AM (6 days ago) Jun 25
to Bartek Nowierski, Chromium LUCI CQ, Keishi Hattori, chromium...@chromium.org, Hans Wennborg, spang...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
Attention needed from Bartek Nowierski
Stephen Nusko voted![]( "Open in Gerrit")
Attention is currently required from:
- Bartek Nowierski
Submit Requirements:
Code-Coverage
Code-Review
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chromium LUCI CQ (Gerrit)
Jun 25, 2024, 9:49:30 AM (6 days ago) Jun 25
to Bartek Nowierski, Stephen Nusko, Keishi Hattori, chromium...@chromium.org, Hans Wennborg, spang...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
Chromium LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
[BRP] Remove TraceLog::OptionalAutoLock exception
The embedding class can be marked STACK_ALLOCATED, and we generally don't rewrite these
Bug: 348793153
Change-Id: I2215c5e9113f25ad63116a80a569ff7a83579652
Commit-Queue: Bartek Nowierski <bar...@chromium.org>
Commit-Queue: Stephen Nusko <nus...@chromium.org>
Reviewed-by: Stephen Nusko <nus...@chromium.org>
Auto-Submit: Bartek Nowierski <bar...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1319114}
Files:
- M base/trace_event/trace_log.cc
- M tools/clang/rewrite_raw_ptr_fields/manual-fields-to-ignore.txt
Change size: S
Delta: 2 files changed, 8 insertions(+), 11 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Stephen Nusko
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages