Avoid duplicate map lookups. [chromium/src : main]
0 views
Skip to first unread message
Yuwei Huang (Gerrit)
Mar 27, 2026, 8:19:03 PM (7 days ago) Mar 27
to Jeffrey Yu, AyeAye, Chromium LUCI CQ, chromotin...@chromium.org
Attention needed from Jeffrey Yu
Yuwei Huang voted Code-Review+1![]( "Open in Gerrit")
| Code-Review | +1 |
Related details
Attention is currently required from:
- Jeffrey 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. |
Jeffrey Yu (Gerrit)
Mar 30, 2026, 4:09:24 PM (4 days ago) Mar 30
to Joe Downing, Yuwei Huang, AyeAye, Chromium LUCI CQ, chromotin...@chromium.org
Attention needed from Joe Downing
Jeffrey Yu voted Auto-Submit+1![]( "Open in Gerrit")
Attention is currently required from:
- Joe Downing
Submit Requirements:
Code-Coverage
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Joe Downing (Gerrit)
Mar 30, 2026, 6:58:47 PM (4 days ago) Mar 30
to Jeffrey Yu, Yuwei Huang, AyeAye, Chromium LUCI CQ, chromotin...@chromium.org
Attention needed from Jeffrey Yu
Joe Downing voted Code-Review+1![]( "Open in Gerrit")
Attention is currently required from:
- Jeffrey 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. |
Jeffrey Yu (Gerrit)
Mar 31, 2026, 11:56:16 AM (3 days ago) Mar 31
to Joe Downing, Yuwei Huang, AyeAye, Chromium LUCI CQ, chromotin...@chromium.org
Jeffrey 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)
Mar 31, 2026, 12:15:44 PM (3 days ago) Mar 31
to Jeffrey Yu, Joe Downing, Yuwei Huang, AyeAye, chromotin...@chromium.org
Chromium LUCI CQ submitted the change![]( "Open in Gerrit")
Change information
Commit message:
Avoid duplicate map lookups.
Instead of using `.count(key)` followed by a lookup, use `.find(key)`,
and reuse the iterator to access the value.
Bug: 452950815
Change-Id: I72156a3c28101b9cf025b85796c613fb37430f83
Reviewed-by: Joe Downing <joe...@chromium.org>
Commit-Queue: Jeffrey Yu <yu...@chromium.org>
Reviewed-by: Yuwei Huang <yuw...@chromium.org>
Auto-Submit: Jeffrey Yu <yu...@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1607906}
Files:
- M remoting/host/desktop_resizer_win.cc
Change size: S
Delta: 1 file changed, 6 insertions(+), 4 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Yuwei Huang, +1 by Joe Downing
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages