| commit | 5b15d3db3b3d742aeca967550882634e43f8244f | [log] [tgz] |
|---|---|---|
| author | Stephen Nusko <[email protected]> | Mon Dec 01 19:12:21 2025 |
| committer | Copybara-Service <[email protected]> | Mon Dec 01 19:19:52 2025 |
| tree | e51d7a5713e57806efed0abcf0389c7431d4c647 | |
| parent | 3df3c1ef84a04c5f1985dee333578a6a2ad2c8fe [diff] |
spanification: Clean up unneeded parameters in components This was the result of asking gemini to remove all of the redundant variables and update call sites. I have also gone ahead and updated all the for loops that the linter complained were not modern c++ (to range for loops). The aim of this patch should be no behaviour changes, however well I've manually reviewed it, since this patch was generated by AI please review carefully. These checks have been in the code for 2 stable releases, and M143 is about to go to Stable, which means this has been fully fatal in Canary/Dev. All of these should be completely redundant. This patch was split with `git cl split --target-range 8 32 --max-depth=2 --no-reviewers -d ~/description.txt` This is part of auto spanification see internal doc about it: https://docs.google.com/document/u/1/d/1poEhNARfFas2osqPoXGzVZtBgnOQGQtQNUFTA-MlADs/edit?tab=t.0 This CL was uploaded by an experimental version of git cl split (https://crbug.com/389069356). Bug: 431824301 Change-Id: I98d8a92bde8812a7272b452977f47e92925a1bb5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7201723 Reviewed-by: Xiangdong Kong <[email protected]> Auto-Submit: Stephen Nusko <[email protected]> Reviewed-by: Ryan Hamilton <[email protected]> Commit-Queue: Orin Jaworski <[email protected]> Reviewed-by: Orin Jaworski <[email protected]> Cr-Commit-Position: refs/heads/main@{#1552287} NOKEYCHECK=True GitOrigin-RevId: aa88a22a5ecb133ca7a51298e13b801fa05c074a