Fix PopupMenuButton crashes when being hidden immediately #171970

huycozy · 2025-07-10T17:05:50Z

Fix PopupMenuButton broken in Flutter 3.32.5 #171422 (and maybe also [go_router] Open PopUpMenuButton causes crash when navigating to new base route using pageBuilder #167180, and PopupMenuRoute crashes sometimes on drawFrame #163477)
Investigation: Per discussion in PopupMenuButton broken in Flutter 3.32.5 #171422, the issue is probably happening due to accessing inherited widgets via context in positionBuilder(like PopupMenuTheme) when PopupMenu's State is inactive. This PR proposed a fix by caching the necessary objects in didChangeDependencies method. Some of RenderBox are also cached to avoid another exception like Cannot get renderObject of inactive element..This solution can also be seen at another widget like CalendarDatePicker:

flutter/packages/flutter/lib/src/material/calendar_date_picker.dart

Lines 225 to 232 in d6b2a95

    
           @override 
        
           void didChangeDependencies() { 
        
             super.didChangeDependencies(); 
        
             assert(debugCheckHasMaterial(context)); 
        
             assert(debugCheckHasMaterialLocalizations(context)); 
        
             assert(debugCheckHasDirectionality(context)); 
        
             _localizations = MaterialLocalizations.of(context); 
        
             _textDirection = Directionality.of(context);

Demo

before	after
before.mov	after.mov

Pre-launch Checklist

I read the Contributor Guide and followed the process outlined there for submitting PRs.
I read the Tree Hygiene wiki page, which explains my responsibilities.
I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
I signed the CLA.
I listed at least one issue that this PR fixes in the description above.
I updated/added relevant documentation (doc comments with ///).
I added new tests to check the change I am making, or this PR is test-exempt.
I followed the breaking change policy and added Data Driven Fixes where supported.
All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Signed-off-by: huycozy <[email protected]>

dkwingsmt

LGTM.

Awesome change! Thank you so much for the investigation.

…lutter/flutter#171970)

flutter/flutter@70cdc0c...ee0cc66 2025-07-22 [email protected] Licenses cpp ignore whitespace (flutter/flutter#172512) 2025-07-22 [email protected] feat: start using the content aware hash for downloading artifacts (flutter/flutter#172552) 2025-07-22 [email protected] fix: correct typo in analysis_options.yaml comment (disabeling -> disabling) (flutter/flutter#170206) 2025-07-22 [email protected] Hide `flutter assemble` unless `--verbose` (flutter/flutter#172508) 2025-07-22 [email protected] Fix PopupMenuButton crashes when being hidden immediately (flutter/flutter#171970) 2025-07-22 [email protected] Add hitTestChildren to baseline-aligned stack (flutter/flutter#172485) 2025-07-22 [email protected] Roll Skia from 4aba494b8108 to eea1c33fbe84 (11 revisions) (flutter/flutter#172515) 2025-07-22 [email protected] Replaces legacy licenses check with licenses_cpp (flutter/flutter#172486) 2025-07-22 [email protected] Omit `w` (widget tree), `t` (render tree) from `flutter run --profile` `h` (flutter/flutter#172506) 2025-07-22 [email protected] Refactor checks for `--machine` (flutter/flutter#172504) 2025-07-22 [email protected] feat(IconButton): add property to specify a states controller (flutter/flutter#169821) 2025-07-21 98614782+auto-submit[bot]@users.noreply.github.com Reverts "feat: start using the content aware hash for downloading artifacts (#171927)" (flutter/flutter#172505) 2025-07-21 [email protected] Render Flutter in its own EGL context (flutter/flutter#172330) 2025-07-21 [email protected] Feat: Add momentary varient of CupertinoSlidingSegmentedControl (flutter/flutter#164262) 2025-07-21 [email protected] Roll Packages from cb8fef6 to 9c85e5e (10 revisions) (flutter/flutter#172479) 2025-07-21 [email protected] Wrap `ShutdownHook` calls to avoid crashing the tool (flutter/flutter#171917) 2025-07-21 [email protected] fix: macOS Mojave Crash on Flutter Desktop Startup (flutter/flutter#172282) 2025-07-21 [email protected] Roll Skia from 9587301e33bc to 4aba494b8108 (41 revisions) (flutter/flutter#172471) 2025-07-21 [email protected] Add CLA note to tree hygiene (flutter/flutter#172475) 2025-07-21 [email protected] Add `radioInnerRadius` to `RadioListTile` (flutter/flutter#171851) 2025-07-21 [email protected] Use baseUri always when doing a hot reload or hot restart (flutter/flutter#172271) 2025-07-21 [email protected] feat: start using the content aware hash for downloading artifacts (flutter/flutter#171927) 2025-07-21 [email protected] Add support for `/?` to alias to `--help` (flutter/flutter#172328) 2025-07-21 [email protected] Upgrade flutter_lints in project template pubspec.yaml files (flutter/flutter#172400) 2025-07-21 [email protected] Use `$dartSdkVersion` when creating "dummy" pubspec for `create_api_docs` (flutter/flutter#172327) 2025-07-21 [email protected] Adds `(defaults to ___)` when using `flutter config --help` (flutter/flutter#172329) 2025-07-21 [email protected] Add a warning on usage of `dartPluginClass: 'none'`. (flutter/flutter#172315) 2025-07-21 [email protected] Skip [tool] crash reporting on `--local-engine` (flutter/flutter#172326) 2025-07-21 [email protected] Remove IDE validation from `flutter doctor` (flutter/flutter#171924) 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] 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

…r#9662) flutter/flutter@70cdc0c...ee0cc66 2025-07-22 [email protected] Licenses cpp ignore whitespace (flutter/flutter#172512) 2025-07-22 [email protected] feat: start using the content aware hash for downloading artifacts (flutter/flutter#172552) 2025-07-22 [email protected] fix: correct typo in analysis_options.yaml comment (disabeling -> disabling) (flutter/flutter#170206) 2025-07-22 [email protected] Hide `flutter assemble` unless `--verbose` (flutter/flutter#172508) 2025-07-22 [email protected] Fix PopupMenuButton crashes when being hidden immediately (flutter/flutter#171970) 2025-07-22 [email protected] Add hitTestChildren to baseline-aligned stack (flutter/flutter#172485) 2025-07-22 [email protected] Roll Skia from 4aba494b8108 to eea1c33fbe84 (11 revisions) (flutter/flutter#172515) 2025-07-22 [email protected] Replaces legacy licenses check with licenses_cpp (flutter/flutter#172486) 2025-07-22 [email protected] Omit `w` (widget tree), `t` (render tree) from `flutter run --profile` `h` (flutter/flutter#172506) 2025-07-22 [email protected] Refactor checks for `--machine` (flutter/flutter#172504) 2025-07-22 [email protected] feat(IconButton): add property to specify a states controller (flutter/flutter#169821) 2025-07-21 98614782+auto-submit[bot]@users.noreply.github.com Reverts "feat: start using the content aware hash for downloading artifacts (#171927)" (flutter/flutter#172505) 2025-07-21 [email protected] Render Flutter in its own EGL context (flutter/flutter#172330) 2025-07-21 [email protected] Feat: Add momentary varient of CupertinoSlidingSegmentedControl (flutter/flutter#164262) 2025-07-21 [email protected] Roll Packages from cb8fef6 to 9c85e5e (10 revisions) (flutter/flutter#172479) 2025-07-21 [email protected] Wrap `ShutdownHook` calls to avoid crashing the tool (flutter/flutter#171917) 2025-07-21 [email protected] fix: macOS Mojave Crash on Flutter Desktop Startup (flutter/flutter#172282) 2025-07-21 [email protected] Roll Skia from 9587301e33bc to 4aba494b8108 (41 revisions) (flutter/flutter#172471) 2025-07-21 [email protected] Add CLA note to tree hygiene (flutter/flutter#172475) 2025-07-21 [email protected] Add `radioInnerRadius` to `RadioListTile` (flutter/flutter#171851) 2025-07-21 [email protected] Use baseUri always when doing a hot reload or hot restart (flutter/flutter#172271) 2025-07-21 [email protected] feat: start using the content aware hash for downloading artifacts (flutter/flutter#171927) 2025-07-21 [email protected] Add support for `/?` to alias to `--help` (flutter/flutter#172328) 2025-07-21 [email protected] Upgrade flutter_lints in project template pubspec.yaml files (flutter/flutter#172400) 2025-07-21 [email protected] Use `$dartSdkVersion` when creating "dummy" pubspec for `create_api_docs` (flutter/flutter#172327) 2025-07-21 [email protected] Adds `(defaults to ___)` when using `flutter config --help` (flutter/flutter#172329) 2025-07-21 [email protected] Add a warning on usage of `dartPluginClass: 'none'`. (flutter/flutter#172315) 2025-07-21 [email protected] Skip [tool] crash reporting on `--local-engine` (flutter/flutter#172326) 2025-07-21 [email protected] Remove IDE validation from `flutter doctor` (flutter/flutter#171924) 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] 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

…lutter/flutter#171970)

Fix PopupMenuButton crashes when being hidden immediately

f1b7f9b

Signed-off-by: huycozy <[email protected]>

huycozy self-assigned this Jul 10, 2025

github-actions bot added framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels Jul 10, 2025

dkwingsmt self-requested a review July 10, 2025 23:25

dkwingsmt approved these changes Jul 22, 2025

View reviewed changes

Merge branch 'master' into fix-PopupMenuButton-crash-in-pop-animation

5f78dd2

huycozy added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 22, 2025

auto-submit bot added this pull request to the merge queue Jul 22, 2025

Merged via the queue into flutter:master with commit db38e71 Jul 22, 2025
72 of 73 checks passed

flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jul 22, 2025

huycozy deleted the fix-PopupMenuButton-crash-in-pop-animation branch July 22, 2025 04:27

This was referenced Jul 22, 2025

PopupMenuRoute crashes sometimes on drawFrame #163477

Closed

[go_router] Open PopUpMenuButton causes crash when navigating to new base route using pageBuilder #167180

Closed

engine-flutter-autoroll mentioned this pull request Jul 22, 2025

Roll Flutter from 70cdc0c933d6 to d16e5b9971e7 (26 revisions) flutter/packages#9659

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 22, 2025

db38e71bf Fix PopupMenuButton crashes when being hidden immediately (f…

2596186

…lutter/flutter#171970)

engine-flutter-autoroll mentioned this pull request Jul 22, 2025

Roll Flutter from 70cdc0c933d6 to 404fb3f14fdd (27 revisions) flutter/packages#9660

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 22, 2025

db38e71bf Fix PopupMenuButton crashes when being hidden immediately (f…

0781628

…lutter/flutter#171970)

engine-flutter-autoroll mentioned this pull request Jul 22, 2025

Roll Flutter from 70cdc0c933d6 to 404fb3f14fdd (27 revisions) flutter/packages#9661

Closed

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 22, 2025

db38e71bf Fix PopupMenuButton crashes when being hidden immediately (f…

c65f9c5

…lutter/flutter#171970)

engine-flutter-autoroll mentioned this pull request Jul 22, 2025

Roll Flutter from 70cdc0c933d6 to ee0cc6652be7 (29 revisions) flutter/packages#9662

Merged

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jul 22, 2025

db38e71bf Fix PopupMenuButton crashes when being hidden immediately (f…

ef2b3d6

…lutter/flutter#171970)

huycozy mentioned this pull request Aug 18, 2025

[go_router] After the fix for #171970, when tapping the PopupMenuButton and navigating to another screen, the popup does not close. #173851

Closed

ksokolovskyi mentioned this pull request Oct 6, 2025

Fix PopupMenu does not update when PopupMenuTheme in Theme changes. #175513

Merged

9 tasks

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 12, 2025

db38e71bf Fix PopupMenuButton crashes when being hidden immediately (f…

2072a72

…lutter/flutter#171970)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Fix PopupMenuButton crashes when being hidden immediately #171970

Fix PopupMenuButton crashes when being hidden immediately #171970

huycozy commented Jul 10, 2025

Uh oh!

dkwingsmt left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	@override
	void didChangeDependencies() {
	super.didChangeDependencies();
	assert(debugCheckHasMaterial(context));
	assert(debugCheckHasMaterialLocalizations(context));
	assert(debugCheckHasDirectionality(context));
	_localizations = MaterialLocalizations.of(context);
	_textDirection = Directionality.of(context);

Fix PopupMenuButton crashes when being hidden immediately #171970

Fix PopupMenuButton crashes when being hidden immediately #171970

Conversation

huycozy commented Jul 10, 2025

Pre-launch Checklist

Uh oh!

dkwingsmt left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants