Skip to content

[widgets] Use super parameters in missed spots#186198

Merged
srawlins merged 1 commit into
flutter:masterfrom
srawlins:super-p-framework
May 16, 2026
Merged

[widgets] Use super parameters in missed spots#186198
srawlins merged 1 commit into
flutter:masterfrom
srawlins:super-p-framework

Conversation

@srawlins

@srawlins srawlins commented May 7, 2026

Copy link
Copy Markdown
Contributor

Work towards dart-lang/sdk#59226

The analyzer will start reporting cases where a super parameter could be used, and the type of the supar parameter is tightened in the subclass constructor. This is still a perfectly valid super parameter use case.

Pre-launch Checklist

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label May 7, 2026
@github-actions github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. f: scrolling Viewports, list views, slivers, etc. labels May 7, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request refactors various constructors across the Flutter framework and its test suites to utilize Dart's super-initializer parameters. The changes simplify constructor syntax in files such as overlay.dart, sliver_prototype_extent_list.dart, and several test utility files by replacing explicit super calls with the super.parameterName syntax. I have no feedback to provide.

@Piinks
Piinks requested a review from justinmc May 12, 2026 22:18
@srawlins
srawlins enabled auto-merge May 13, 2026 20:28

@justinmc justinmc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 . Thanks!

@srawlins
srawlins added this pull request to the merge queue May 14, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks May 15, 2026
@srawlins
srawlins added this pull request to the merge queue May 16, 2026
Merged via the queue into flutter:master with commit 467c55f May 16, 2026
85 checks passed
@srawlins
srawlins deleted the super-p-framework branch May 16, 2026 17:04
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request May 18, 2026
flutter/flutter@1ceffd1...3598686

2026-05-17 [email protected] Roll Fuchsia Linux SDK from G9xv1qcMhvXOy-9pk... to 5Ki-dBY4SpWdQMF_3... (flutter/flutter#186636)
2026-05-17 [email protected] [Impeller] Canonicalize uniform block instance names for the GL backends (flutter/flutter#186394)
2026-05-16 [email protected] [widgets] Use super parameters in missed spots (flutter/flutter#186198)
2026-05-16 [email protected] refactor: Remove material imports from Widget tests  (flutter/flutter#185078)
2026-05-16 [email protected] Roll pub packages (flutter/flutter#186602)
2026-05-16 [email protected] [gn] Fix typo in comment (flutter/flutter#186549)
2026-05-16 [email protected] Roll Fuchsia Linux SDK from y6uQHA5xUN83IF395... to G9xv1qcMhvXOy-9pk... (flutter/flutter#186599)
2026-05-15 [email protected] Removes bringup from passing macos/sdf tests (flutter/flutter#186527)
2026-05-15 [email protected] [web] Fix several WebParagraph bugs (flutter/flutter#186403)
2026-05-15 [email protected] Display the team ID and name when selecting a signing certificate (flutter/flutter#184665)
2026-05-15 [email protected] [iOS] Improve documentation on FlutterVSyncClient and FlutterDisplayLink (flutter/flutter#186456)
2026-05-15 [email protected] Increase the run time of text field integration tests to 10 seconds (flutter/flutter#186475)
2026-05-15 [email protected] Fix web-server hot restart/reload not applying changes for entrypoints outside lib (flutter/flutter#183838)
2026-05-15 [email protected] Update dart_skills_lint dependency to e449787 and optimize skills validation test (flutter/flutter#186528)
2026-05-15 [email protected] Roll Packages from 2ec2236 to 32c84d6 (3 revisions) (flutter/flutter#186583)
2026-05-15 [email protected] Dispose TextEditingController in IndexedStack example (flutter/flutter#186375)
2026-05-15 [email protected] refactor: update filename for sliver semantic widget (flutter/flutter#185917)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
…r#11724)

flutter/flutter@1ceffd1...3598686

2026-05-17 [email protected] Roll Fuchsia Linux SDK from G9xv1qcMhvXOy-9pk... to 5Ki-dBY4SpWdQMF_3... (flutter/flutter#186636)
2026-05-17 [email protected] [Impeller] Canonicalize uniform block instance names for the GL backends (flutter/flutter#186394)
2026-05-16 [email protected] [widgets] Use super parameters in missed spots (flutter/flutter#186198)
2026-05-16 [email protected] refactor: Remove material imports from Widget tests  (flutter/flutter#185078)
2026-05-16 [email protected] Roll pub packages (flutter/flutter#186602)
2026-05-16 [email protected] [gn] Fix typo in comment (flutter/flutter#186549)
2026-05-16 [email protected] Roll Fuchsia Linux SDK from y6uQHA5xUN83IF395... to G9xv1qcMhvXOy-9pk... (flutter/flutter#186599)
2026-05-15 [email protected] Removes bringup from passing macos/sdf tests (flutter/flutter#186527)
2026-05-15 [email protected] [web] Fix several WebParagraph bugs (flutter/flutter#186403)
2026-05-15 [email protected] Display the team ID and name when selecting a signing certificate (flutter/flutter#184665)
2026-05-15 [email protected] [iOS] Improve documentation on FlutterVSyncClient and FlutterDisplayLink (flutter/flutter#186456)
2026-05-15 [email protected] Increase the run time of text field integration tests to 10 seconds (flutter/flutter#186475)
2026-05-15 [email protected] Fix web-server hot restart/reload not applying changes for entrypoints outside lib (flutter/flutter#183838)
2026-05-15 [email protected] Update dart_skills_lint dependency to e449787 and optimize skills validation test (flutter/flutter#186528)
2026-05-15 [email protected] Roll Packages from 2ec2236 to 32c84d6 (3 revisions) (flutter/flutter#186583)
2026-05-15 [email protected] Dispose TextEditingController in IndexedStack example (flutter/flutter#186375)
2026-05-15 [email protected] refactor: update filename for sliver semantic widget (flutter/flutter#185917)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC [email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD f: scrolling Viewports, list views, slivers, etc. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants