#50778: chore: iwyu commonly-included headers in shell/
Merged
Description of Change
A small cleanup refactor that I did as a tangent for another PR that I'm working on.
This PR removes unnecessary #include calls in some of the most commonly-included headers in shell/ and fixes some transitive includes that surfaced from making this change.
CC @dsanders11 as an iwyu stakeholder :)
Checklist
- PR description included
- I have built and tested this PR
-
npm testpasses - PR release notes describe the change in a way relevant to app developers, and are capitalized, punctuated, and past tense.
Release Notes
Notes: none.
Backports
42-x-y
PendingWaiting for a manual backport
Semver Impact
Major
Breaking changes
Minor
New features
Patch
Bug fixes
None
Docs, tests, etc.
Semantic Versioning helps users understand the impact of updates:
- Major (X.y.z): Breaking changes that may require code modifications
- Minor (x.Y.z): New features that maintain backward compatibility
- Patch (x.y.Z): Bug fixes that don't change the API
- None: Changes that don't affect using facing parts of Electron