Fix some PA_UNSAFE_TODOs in //base/allocator/partition_allocator [chromium/src : main]
1 view
Skip to first unread message
Etienne Pierre-Doray (Gerrit)
Feb 5, 2026, 7:44:53 AM (2 days ago) Feb 5
to YISI YU, Kentaro Hara, Chromium LUCI CQ, chromium...@chromium.org, bartek...@chromium.org, lize...@chromium.org, lizeb...@chromium.org, spang...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
Attention needed from Kentaro Hara and YISI YU
Etienne Pierre-Doray voted and added 1 comment![]( "Open in Gerrit")
Votes added by Etienne Pierre-Doray
| Code-Review | +1 |
1 comment
Patchset-level comments
Related details
Attention is currently required from:
- Kentaro Hara
- YISI YU
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Kentaro Hara (Gerrit)
Feb 5, 2026, 12:30:48 PM (2 days ago) Feb 5
to YISI YU, Etienne Pierre-Doray, Chromium LUCI CQ, chromium...@chromium.org, bartek...@chromium.org, lize...@chromium.org, lizeb...@chromium.org, spang...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
Attention needed from YISI YU
Kentaro Hara voted Code-Review+1![]( "Open in Gerrit")
Attention is currently required from:
- YISI YU
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
YISI YU (Gerrit)
Feb 5, 2026, 7:12:10 PM (2 days ago) Feb 5
to Kentaro Hara, Etienne Pierre-Doray, Chromium LUCI CQ, chromium...@chromium.org, bartek...@chromium.org, lize...@chromium.org, lizeb...@chromium.org, spang...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
YISI YU voted Commit-Queue+2![]( "Open in Gerrit")
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Chromium LUCI CQ (Gerrit)
Feb 5, 2026, 7:25:25 PM (2 days ago) Feb 5
to YISI YU, Kentaro Hara, Etienne Pierre-Doray, chromium...@chromium.org, bartek...@chromium.org, lize...@chromium.org, lizeb...@chromium.org, spang...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
Chromium LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
Fix some PA_UNSAFE_TODOs in //base/allocator/partition_allocator
Use std::array instead of native array.
Bug: 40285824
Change-Id: I652094ad6a36c77bd9cfdd56e8afa772096f581a
Reviewed-by: Etienne Pierre-Doray <etie...@chromium.org>
Commit-Queue: YISI YU <hello...@gmail.com>
Reviewed-by: Kentaro Hara <har...@chromium.org>
Auto-Submit: YISI YU <hello...@gmail.com>
Cr-Commit-Position: refs/heads/main@{#1580490}
Files:
- M base/allocator/partition_allocator/src/partition_alloc/address_pool_manager.cc
- M base/allocator/partition_allocator/src/partition_alloc/address_pool_manager.h
- M base/allocator/partition_allocator/src/partition_alloc/address_pool_manager_unittest.cc
- M base/allocator/partition_allocator/src/partition_alloc/partition_stats.h
- M base/allocator/partition_allocator/src/partition_alloc/thread_cache.cc
- M base/allocator/partition_allocator/src/partition_alloc/thread_cache.h
- M base/trace_event/malloc_dump_provider.cc
Change size: M
Delta: 7 files changed, 58 insertions(+), 67 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Kentaro Hara, +1 by Etienne Pierre-Doray
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages