Skip to content

Conversation

@Hixie
Copy link
Contributor

@Hixie Hixie commented Jul 31, 2023

No description provided.

@github-actions github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Jul 31, 2023
@morgwai
Copy link

morgwai commented Aug 1, 2023

related issue: #40940
as many ppl stated in the above issue, we think that Flutter platform should put its best effort to call dispose() as reliably as feasible.

@Piinks Piinks added the d: api docs Issues with https://api.flutter.dev/ label Aug 1, 2023
Copy link
Contributor

@Piinks Piinks left a comment

Choose a reason for hiding this comment

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

This LGTM - although it looks like conversation is ongoing in the issue?

@Hixie
Copy link
Contributor Author

Hixie commented Aug 2, 2023

yeah this is just a stopgap while we continue to figure out how to resolve it. (Even if we can resolve the issue, it's still useful information for most ways that an app can shut down.)

@Hixie Hixie added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 2, 2023
@auto-submit
Copy link
Contributor

auto-submit bot commented Aug 2, 2023

auto label is removed for flutter/flutter, pr: 131637, due to - The status or check suite Linux framework_tests_libraries has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Aug 2, 2023
@Piinks
Copy link
Contributor

Piinks commented Aug 2, 2023

Failure appears to be another instance of https://g-issues.skia.org/issues/40045532, following up with the Gold team. I kicked the task off again.

@Piinks Piinks added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 2, 2023
@auto-submit auto-submit bot merged commit 47b188e into flutter:master Aug 2, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Aug 3, 2023
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Aug 3, 2023
flutter/flutter@b3f99ff...c00d241

2023-08-03 [email protected] Manual roll Flutter Engine from 9304c0074d29 to 4c1157b9da54 (24 revisions) (flutter/flutter#131830)
2023-08-03 [email protected] Assert against infinite values of control points in CatmullRomSpline (flutter/flutter#131820)
2023-08-02 [email protected] Update stack_frame.dart to parse unexpected error format to null. (flutter/flutter#131786)
2023-08-02 [email protected] Replace TextField.canRequestFocus with TextField.focusNode.canRequestFocus (flutter/flutter#130164)
2023-08-02 [email protected] Mention that the widget tree is not disposed on exit (flutter/flutter#131637)
2023-08-02 [email protected] [framework] Add Look Up to selection controls for iOS  (flutter/flutter#131798)
2023-08-02 [email protected] Add documentation in flutter.groovy noting that we always use the latest available android version (flutter/flutter#131705)
2023-08-02 [email protected] [framework] lerp images in a save layer. (flutter/flutter#131703)
2023-08-02 [email protected] Roll Packages from 3dc00c1 to 4e4961a (1 revision) (flutter/flutter#131788)
2023-08-02 [email protected] add gem dependency to the ios_app_with_extensions_test (flutter/flutter#131713)
2023-08-02 [email protected] Roll Flutter Engine from d6b962d0e36d to 9304c0074d29 (4 revisions) (flutter/flutter#131790)
2023-08-02 [email protected] [flutter_tools] set terminal.singleCharMode to false after attach finishes (flutter/flutter#131723)

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App d: api docs Issues with https://api.flutter.dev/ framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants