-
Notifications
You must be signed in to change notification settings - Fork 28.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cover more tests with leak tracking. #134805
Conversation
@polina-c It seems that you faced the issue described here: #134205. void _scheduleRouteInformationReportingTask() {
if (_routeInformationReportingTaskScheduled || widget.routeInformationProvider == null) {
return;
}
assert(_currentIntentionToReport != null);
_routeInformationReportingTaskScheduled = true;
SchedulerBinding.instance.addPostFrameCallback(_reportRouteInformation);
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
flutter/flutter@b7d0e8c...8936504 2023-09-19 [email protected] Roll Flutter Engine from e1c784e3f841 to 589bde9a95c9 (16 revisions) (flutter/flutter#134998) 2023-09-19 [email protected] Reland Resolve breaking change of adding a method to ChangeNotifier. (flutter/flutter#134983) 2023-09-18 [email protected] Revert "Resolve breaking change of adding a method to ChangeNotifier." (flutter/flutter#134978) 2023-09-18 [email protected] Resolve breaking change of adding a method to ChangeNotifier. (flutter/flutter#134953) 2023-09-18 [email protected] Fixes focus traversal crash if the current node can't request focus (flutter/flutter#134954) 2023-09-18 [email protected] Access to fragment in router state on page refresh (flutter/flutter#131123) 2023-09-18 [email protected] Enable private field promotion for framework (flutter/flutter#134473) 2023-09-18 [email protected] Cover more test/widgets tests with leak tracking #6 (flutter/flutter#134884) 2023-09-18 [email protected] Cover more tests with leak tracking. (flutter/flutter#134805) 2023-09-18 [email protected] Cover more test/widgets tests with leak tracking #5 (flutter/flutter#134869) 2023-09-18 [email protected] Don't uninstall before retrying to connect during app launch (flutter/flutter#134542) 2023-09-18 [email protected] [flutter roll] Revert "LinkedText (Linkify)" (flutter/flutter#134955) 2023-09-18 [email protected] Fix memory leak in _DarwinViewState. (flutter/flutter#134938) 2023-09-18 [email protected] Roll Flutter Engine from be7a039c5451 to e1c784e3f841 (3 revisions) (flutter/flutter#134956) 2023-09-18 [email protected] Migrate the "Missing frames / speed related performance issues" issue template to use Github forms (flutter/flutter#134033) 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],[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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
flutter/flutter@b7d0e8c...8936504 2023-09-19 [email protected] Roll Flutter Engine from e1c784e3f841 to 589bde9a95c9 (16 revisions) (flutter/flutter#134998) 2023-09-19 [email protected] Reland Resolve breaking change of adding a method to ChangeNotifier. (flutter/flutter#134983) 2023-09-18 [email protected] Revert "Resolve breaking change of adding a method to ChangeNotifier." (flutter/flutter#134978) 2023-09-18 [email protected] Resolve breaking change of adding a method to ChangeNotifier. (flutter/flutter#134953) 2023-09-18 [email protected] Fixes focus traversal crash if the current node can't request focus (flutter/flutter#134954) 2023-09-18 [email protected] Access to fragment in router state on page refresh (flutter/flutter#131123) 2023-09-18 [email protected] Enable private field promotion for framework (flutter/flutter#134473) 2023-09-18 [email protected] Cover more test/widgets tests with leak tracking flutter#6 (flutter/flutter#134884) 2023-09-18 [email protected] Cover more tests with leak tracking. (flutter/flutter#134805) 2023-09-18 [email protected] Cover more test/widgets tests with leak tracking flutter#5 (flutter/flutter#134869) 2023-09-18 [email protected] Don't uninstall before retrying to connect during app launch (flutter/flutter#134542) 2023-09-18 [email protected] [flutter roll] Revert "LinkedText (Linkify)" (flutter/flutter#134955) 2023-09-18 [email protected] Fix memory leak in _DarwinViewState. (flutter/flutter#134938) 2023-09-18 [email protected] Roll Flutter Engine from be7a039c5451 to e1c784e3f841 (3 revisions) (flutter/flutter#134956) 2023-09-18 [email protected] Migrate the "Missing frames / speed related performance issues" issue template to use Github forms (flutter/flutter#134033) 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],[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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
No description provided.