Skip to content

Conversation

@polina-c
Copy link
Contributor

@polina-c polina-c commented Jun 6, 2024

Introduced by #149763.

Repro: flutter test test/widgets/router_test.dart --dart-define LEAK_TRACKING=true

@polina-c polina-c requested a review from chunhtai June 6, 2024 15:05
@github-actions github-actions bot added framework flutter/packages/flutter repository. See also f: labels. f: routes Navigator, Router, and related APIs. labels Jun 6, 2024
@polina-c polina-c added the a: leak tracking Issues and PRs related to memory leaks detected by leak_tracker label Jun 6, 2024
Copy link
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for fixing this!

@polina-c polina-c merged commit d19d19c into flutter:master Jun 6, 2024
@polina-c polina-c deleted the leak-fix2 branch June 6, 2024 18:33
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 7, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 7, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 7, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 7, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 7, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 7, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 7, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jun 7, 2024
Manual roll Flutter from 4608a89 to fc19ecf (38 revisions)

Manual roll requested by dit@google.com

flutter/flutter@4608a89...fc19ecf

2024-06-07 tessertaha@gmail.com Fix `SegmentedButton` clipping when drawing segments (flutter/flutter#149739)
2024-06-07 zanderso@users.noreply.github.com Shift consistently passing tests on mokey to prod (flutter/flutter#149902)
2024-06-07 polinach@google.com Prepare images for tests individually to enable clean up of cache. (flutter/flutter#149693)
2024-06-07 engine-flutter-autoroll@skia.org Roll Packages from 586faa6 to 8a2c4e4 (8 revisions) (flutter/flutter#149900)
2024-06-07 engine-flutter-autoroll@skia.org Roll Flutter Engine from 98e9393bf2e9 to 1cdbebee1901 (4 revisions) (flutter/flutter#149899)
2024-06-07 engine-flutter-autoroll@skia.org Roll Flutter Engine from 087feab9922b to 98e9393bf2e9 (1 revision) (flutter/flutter#149875)
2024-06-07 21270878+elliette@users.noreply.github.com Refactor `widget_inspector_test.dart` (flutter/flutter#149850)
2024-06-07 32538273+ValentinVignal@users.noreply.github.com Add test for dropdown_menu.1.dart (flutter/flutter#149547)
2024-06-07 32538273+ValentinVignal@users.noreply.github.com Add test for standard_fab_location.0.dart (flutter/flutter#149225)
2024-06-07 dacoharkes@google.com [native_assets] Fix framework name deduplication (flutter/flutter#149761)
2024-06-07 engine-flutter-autoroll@skia.org Roll Flutter Engine from 47b15dfba1da to 087feab9922b (1 revision) (flutter/flutter#149871)
2024-06-07 engine-flutter-autoroll@skia.org Roll Flutter Engine from 5d5e0ee9102c to 47b15dfba1da (2 revisions) (flutter/flutter#149870)
2024-06-07 engine-flutter-autoroll@skia.org Roll Flutter Engine from fea8e51b0280 to 5d5e0ee9102c (2 revisions) (flutter/flutter#149867)
2024-06-07 engine-flutter-autoroll@skia.org Roll Flutter Engine from 5a0a49939c64 to fea8e51b0280 (2 revisions) (flutter/flutter#149865)
2024-06-07 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#149864)
2024-06-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from bf1a4877da0b to 5a0a49939c64 (1 revision) (flutter/flutter#149861)
2024-06-06 ian@hixie.ch Remove some vestigial /*!*/ comments (flutter/flutter#149361)
2024-06-06 jacksongardner@google.com Attempt to fix some web test flakiness. (flutter/flutter#149828)
2024-06-06 katelovett@google.com Remove abi key permanently from golden file testing (flutter/flutter#149858)
2024-06-06 dkwingsmt@users.noreply.github.com [CupertinoActionSheet] Add sliding tap gesture (flutter/flutter#149471)
2024-06-06 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#149852)
2024-06-06 hans.muller@gmail.com SliverResizingHeader (flutter/flutter#143325)
2024-06-06 15619084+vashworth@users.noreply.github.com Run tests on macOS 13 or 14 (flutter/flutter#149827)
2024-06-06 polinach@google.com Fix leaky test. (flutter/flutter#149823)
2024-06-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 31804a222f97 to bf1a4877da0b (3 revisions) (flutter/flutter#149848)
2024-06-06 vtrgalo@gmail.com Add support for setting the heading level for web semantics (#97894) (flutter/flutter#125771)
2024-06-06 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Introduce `ChipAnimationStyle` to override default chips animations durations (#149245)" (flutter/flutter#149847)
2024-06-06 kevinjchisholm@google.com 3.22.2 release notes (flutter/flutter#149818)
2024-06-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 25bcbb888939 to 31804a222f97 (3 revisions) (flutter/flutter#149837)
2024-06-06 tessertaha@gmail.com Introduce `ChipAnimationStyle` to override default chips animations durations (flutter/flutter#149245)
2024-06-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from f5a9690c6061 to 25bcbb888939 (1 revision) (flutter/flutter#149835)
2024-06-06 polinach@google.com Fix leaky test. (flutter/flutter#149822)
2024-06-06 magder@google.com Unpin archive package from tool, update to version without security advisories (flutter/flutter#149430)
2024-06-06 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#149630)
2024-06-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from d9684a37af06 to f5a9690c6061 (7 revisions) (flutter/flutter#149832)
2024-06-06 engine-flutter-autoroll@skia.org Roll Packages from 11e192a to 586faa6 (13 revisions) (flutter/flutter#149820)
2024-06-06 mdebbar@google.com Fix some links in the "Handling breaking change" section (flutter/flutter#149821)
2024-06-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from 92d0cd9370f7 to d9684a37af06 (1 revision) (flutter/flutter#149793)

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 dit@google.com,rmistry@google.com,stuartmorgan@google.com on the revert to ensure that a human
is aware of the problem.
...
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: leak tracking Issues and PRs related to memory leaks detected by leak_tracker 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.

2 participants