Skip to content

Remove Material/Cupertino references in error messages - #191305

Merged
auto-submit[bot] merged 4 commits into
flutter:masterfrom
justinmc:material-in-error-message
Aug 23, 2026
Merged

auto-submit[bot] merged 4 commits into
flutter:masterfrom
justinmc:material-in-error-message

Conversation

@justinmc

@justinmc justinmc commented Aug 18, 2026 •

Copy link
Copy Markdown
Contributor

This removes references to Material and Cupertino widgets in framework error messages, instead using the generic Widgets library versions.

Fixes #190947

@justinmc justinmc self-assigned this Aug 18, 2026
@justinmc justinmc added the CICD Run CI/CD label Aug 18, 2026
@justinmc
justinmc marked this pull request as ready for review August 18, 2026 21:15
@flutter-dashboard

Copy link
Copy Markdown

It looks like this pull request may not have tests. Please make sure to add tests or get an explicit test exemption before merging.

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. If you believe this PR qualifies for a test exemption, contact "@test-exemption-reviewer" in the #hackers channel in Discord (don't just cc them here, they won't see it!). The test exemption team is a small volunteer group, so all reviewers should feel empowered to ask for tests, without delegating that responsibility entirely to the test exemption group.

@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 updates error messages and debug hints across several widget files, including debug.dart, focus_traversal.dart, navigator.dart, overlay.dart, restoration.dart, and shared_app_data.dart. The changes simplify references to MaterialApp and CupertinoApp by consolidating them to WidgetsApp, which introduces the relevant behaviors implicitly, and updates a route creation reference from showDialog to showRawDialog. There are no review comments, and I have no additional feedback to provide.

@github-actions github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. f: routes Navigator, Router, and related APIs. f: focus Focus traversal, gaining or losing focus labels Aug 18, 2026
@Piinks Piinks added the a: error message Error messages from the Flutter framework label Aug 19, 2026
Piinks
Piinks previously approved these changes Aug 19, 2026

@Piinks Piinks 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

@stuartmorgan-g

Copy link
Copy Markdown
Contributor

test-exempt: only modifies informational messages

@Piinks Piinks added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 20, 2026
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Aug 20, 2026
@auto-submit

auto-submit Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/flutter/191305, because - The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

  • The status or check suite ci.yaml validation has failed. Please fix the issues identified (or deflake) before re-applying this label.

@justinmc

Copy link
Copy Markdown
Contributor Author

The Google testing failure appeared to be a timeout.

@Piinks Piinks added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 20, 2026
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Aug 20, 2026
@auto-submit

auto-submit Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/flutter/191305, because - The status or check suite Dashboard Checks has failed. Please fix the issues identified (or deflake) before re-applying this label.

@navaronbracke navaronbracke added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 23, 2026
@auto-submit
auto-submit Bot added this pull request to the merge queue Aug 23, 2026
Merged via the queue into flutter:master with commit 1f2af33 Aug 23, 2026
22 checks passed
@flutter-dashboard flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Aug 23, 2026
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Aug 24, 2026
flutter/flutter@65c9a8d...9a82789

2026-08-24 [email protected] Roll Packages from 252bb33 to df2ba94 (5 revisions) (flutter/flutter#191578)
2026-08-24 [email protected] Roll Dart SDK from cb056da580b6 to 78bbc37b6ff6 (13 revisions) (flutter/flutter#191574)
2026-08-24 [email protected] Roll Skia from 50452b13f05b to 57968d087d18 (1 revision) (flutter/flutter#191572)
2026-08-24 [email protected] Move more RSE rendering cases to complex_rse.frag (flutter/flutter#191235)
2026-08-24 [email protected] Roll Skia from 925e13559532 to 50452b13f05b (2 revisions) (flutter/flutter#191570)
2026-08-24 [email protected] iOS: Add stylus pointer tests (flutter/flutter#191564)
2026-08-24 [email protected] iOS: Improve FlutterEngineGroup tests (flutter/flutter#191563)
2026-08-24 [email protected] Roll Skia from c78782c7c1c2 to 925e13559532 (5 revisions) (flutter/flutter#191568)
2026-08-24 [email protected] Roll Skia from 886302c7c68e to c78782c7c1c2 (1 revision) (flutter/flutter#191566)
2026-08-24 [email protected] Revert "Improve initial size and placement logic for popups and toolt…ips on Linux (#189994) (flutter/flutter#191498)
2026-08-23 [email protected] Roll Fuchsia Linux SDK from Cqs-NyeELd60hqv1e... to 8Xu4ujBJniC0nQGx3... (flutter/flutter#191561)
2026-08-23 [email protected] Fixes MSAA on the linux embedder. (flutter/flutter#191379)
2026-08-23 [email protected] Remove Material/Cupertino references in error messages (flutter/flutter#191305)
2026-08-23 [email protected] Roll Skia from a1825062f832 to 886302c7c68e (1 revision) (flutter/flutter#191559)
2026-08-23 [email protected] Roll Skia from ad2106c0bb64 to a1825062f832 (1 revision) (flutter/flutter#191555)
2026-08-22 [email protected] Roll Fuchsia Linux SDK from ic6GjOSn-KN508XyK... to Cqs-NyeELd60hqv1e... (flutter/flutter#191547)
2026-08-22 [email protected] [WebParagraph] Fixing minor problem with getBoxesForRange (flutter/flutter#191419)

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
@justinmc
justinmc deleted the material-in-error-message branch August 24, 2026 23:20
jagadeesh8682 pushed a commit to jagadeesh8682/packages that referenced this pull request Sep 2, 2026
…r#12578)

flutter/flutter@65c9a8d...9a82789

2026-08-24 [email protected] Roll Packages from 252bb33 to df2ba94 (5 revisions) (flutter/flutter#191578)
2026-08-24 [email protected] Roll Dart SDK from cb056da580b6 to 78bbc37b6ff6 (13 revisions) (flutter/flutter#191574)
2026-08-24 [email protected] Roll Skia from 50452b13f05b to 57968d087d18 (1 revision) (flutter/flutter#191572)
2026-08-24 [email protected] Move more RSE rendering cases to complex_rse.frag (flutter/flutter#191235)
2026-08-24 [email protected] Roll Skia from 925e13559532 to 50452b13f05b (2 revisions) (flutter/flutter#191570)
2026-08-24 [email protected] iOS: Add stylus pointer tests (flutter/flutter#191564)
2026-08-24 [email protected] iOS: Improve FlutterEngineGroup tests (flutter/flutter#191563)
2026-08-24 [email protected] Roll Skia from c78782c7c1c2 to 925e13559532 (5 revisions) (flutter/flutter#191568)
2026-08-24 [email protected] Roll Skia from 886302c7c68e to c78782c7c1c2 (1 revision) (flutter/flutter#191566)
2026-08-24 [email protected] Revert "Improve initial size and placement logic for popups and toolt…ips on Linux (#189994) (flutter/flutter#191498)
2026-08-23 [email protected] Roll Fuchsia Linux SDK from Cqs-NyeELd60hqv1e... to 8Xu4ujBJniC0nQGx3... (flutter/flutter#191561)
2026-08-23 [email protected] Fixes MSAA on the linux embedder. (flutter/flutter#191379)
2026-08-23 [email protected] Remove Material/Cupertino references in error messages (flutter/flutter#191305)
2026-08-23 [email protected] Roll Skia from a1825062f832 to 886302c7c68e (1 revision) (flutter/flutter#191559)
2026-08-23 [email protected] Roll Skia from ad2106c0bb64 to a1825062f832 (1 revision) (flutter/flutter#191555)
2026-08-22 [email protected] Roll Fuchsia Linux SDK from ic6GjOSn-KN508XyK... to Cqs-NyeELd60hqv1e... (flutter/flutter#191547)
2026-08-22 [email protected] [WebParagraph] Fixing minor problem with getBoxesForRange (flutter/flutter#191419)

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

a: error message Error messages from the Flutter framework CICD Run CI/CD f: focus Focus traversal, gaining or losing focus f: routes Navigator, Router, and related APIs. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Audit Material / Cupertino references in widgets, for FlutterError error messages

4 participants