[Extensions] Ensure iterator stability for EventListenerMap [chromium/src : main]
0 views
Skip to first unread message
Andrea Orru (Gerrit)
Apr 1, 2026, 1:03:53 PM (yesterday) Apr 1
to Dominic Battre, AyeAye, Chromium LUCI CQ, Keishi Hattori, chromium-a...@chromium.org, extension...@chromium.org
Attention needed from Dominic Battre
Andrea Orru voted Code-Review+1![]( "Open in Gerrit")
| Code-Review | +1 |
Related details
Attention is currently required from:
- Dominic Battre
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Dominic Battre (Gerrit)
Apr 1, 2026, 2:21:49 PM (yesterday) Apr 1
to Andrea Orru, AyeAye, Chromium LUCI CQ, Keishi Hattori, chromium-a...@chromium.org, extension...@chromium.org
Dominic Battre 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)
6:03 AM (9 hours ago) 6:03 AM
to Dominic Battre, Andrea Orru, AyeAye, Keishi Hattori, chromium-a...@chromium.org, extension...@chromium.org
Chromium LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
[Extensions] Ensure iterator stability for EventListenerMap
By moving from flat_map to std::map, we can guarantee that iterators
remain stable in the presence of modifying the map.
Bug: b:497977591
Change-Id: I81bd322f69c8628d3d0b3cedcbaaf7e26a6a6964
Commit-Queue: Dominic Battre <bat...@chromium.org>
Reviewed-by: Andrea Orru <andre...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1609075}
Files:
- M extensions/browser/event_listener_map.cc
- M extensions/browser/event_listener_map.h
Change size: S
Delta: 2 files changed, 17 insertions(+), 5 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Andrea Orru
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages