-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Organize leak tracking TODOs. #142460
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
Organize leak tracking TODOs. #142460
Conversation
.ci.yaml
Outdated
| - name: Windows framework_tests_libraries_leak_tracking | ||
| recipe: flutter/flutter_drone | ||
| bringup: true # New target: https://github.com/flutter/flutter/issues/140414 | ||
| # bringup: true # New target: https://github.com/flutter/flutter/issues/140414 |
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.
Is this change supposed to be in here?
|
|
||
| testWidgets('Should show event indicator for pointer events with setSurfaceSize', | ||
| // TODO(polina-c): clean up leaks, https://github.com/flutter/flutter/issues/134787 | ||
| // TODO(polina-c): clean up leaks, https://github.com/flutter/flutter/issues/134787 [leaks-to-clean] |
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.
Looks like they all have the same issue attached, could you just search for that?
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.
Nope, they are different. There are at least two (141388 and 134787). And, when we start investigating, new ones will show up.
goderbauer
left a comment
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@ace9181...75a2e5b 2024-01-30 39623254+aizatazhar@users.noreply.github.com Reset framesEnabled to default value at the end of each test (flutter/flutter#141844) 2024-01-30 katelovett@google.com Fix SliverMainAxisGroup geometry cacheExtent (flutter/flutter#142482) 2024-01-30 engine-flutter-autoroll@skia.org Roll Flutter Engine from 0e586d1c28c8 to f02a4a80a77e (3 revisions) (flutter/flutter#142528) 2024-01-30 engine-flutter-autoroll@skia.org Roll Packages from 516648a to 25abb5d (6 revisions) (flutter/flutter#142527) 2024-01-30 engine-flutter-autoroll@skia.org Roll Flutter Engine from 438e9b4d7d4e to 0e586d1c28c8 (4 revisions) (flutter/flutter#142515) 2024-01-30 26625149+0xZOne@users.noreply.github.com Using `initExpensiveAndroidView` for Android Hybrid Composition Mode (flutter/flutter#142399) 2024-01-30 engine-flutter-autoroll@skia.org Roll Flutter Engine from ed73d40a8c93 to 438e9b4d7d4e (1 revision) (flutter/flutter#142508) 2024-01-30 polinach@google.com Organize leak tracking TODOs. (flutter/flutter#142460) 2024-01-30 engine-flutter-autoroll@skia.org Roll Flutter Engine from 5584a78a439b to ed73d40a8c93 (1 revision) (flutter/flutter#142504) 2024-01-30 engine-flutter-autoroll@skia.org Roll Flutter Engine from df5f1afd4991 to 5584a78a439b (2 revisions) (flutter/flutter#142503) 2024-01-30 engine-flutter-autoroll@skia.org Roll Flutter Engine from 65bf8b1db4d1 to df5f1afd4991 (1 revision) (flutter/flutter#142501) 2024-01-30 engine-flutter-autoroll@skia.org Roll Flutter Engine from c9268c7db03c to 65bf8b1db4d1 (1 revision) (flutter/flutter#142496) 2024-01-30 engine-flutter-autoroll@skia.org Roll Flutter Engine from e21208583956 to c9268c7db03c (1 revision) (flutter/flutter#142492) 2024-01-29 engine-flutter-autoroll@skia.org Roll Flutter Engine from bedafa8794b6 to e21208583956 (1 revision) (flutter/flutter#142483) 2024-01-29 43054281+camsim99@users.noreply.github.com Revert "Reland: "Fix how Gradle resolves Android plugin" (#137115)" (flutter/flutter#142464) 2024-01-29 fluttergithubbot@gmail.com Marks Mac_pixel_7pro native_assets_android to be unflaky (flutter/flutter#141675) 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 rmistry@google.com,stuartmorgan@google.com 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
Prerequisite: https://github.com/flutter/flutter/issues/141119m should merge to G3 (done).
This PR:
[leaks-to-clean]to all remaining opt outs that should be fixed, to make the search easier