Skip to content

Conversation

@polina-c
Copy link
Contributor

@polina-c polina-c commented Jan 29, 2024

Prerequisite: https://github.com/flutter/flutter/issues/141119m should merge to G3 (done).

This PR:

  1. Adds/removes opt outs for leak tracking
  2. Adds tag [leaks-to-clean] to all remaining opt outs that should be fixed, to make the search easier

@github-actions github-actions bot added a: text input Entering text in a text field or keyboard related problems framework flutter/packages/flutter repository. See also f: labels. a: animation Animation APIs f: material design flutter/packages/flutter/material repository. a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) f: scrolling Viewports, list views, slivers, etc. f: cupertino flutter/packages/flutter/cupertino repository labels Jan 29, 2024
@polina-c polina-c marked this pull request as draft January 29, 2024 18:13
@polina-c polina-c marked this pull request as ready for review January 29, 2024 23:18
@polina-c polina-c requested review from goderbauer and kenzieschmoll and removed request for goderbauer January 29, 2024 23:18
.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
Copy link
Member

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]
Copy link
Member

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?

Copy link
Contributor Author

@polina-c polina-c Jan 30, 2024

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.

@polina-c polina-c requested a review from goderbauer January 30, 2024 00:29
Copy link
Member

@goderbauer goderbauer left a comment

Choose a reason for hiding this comment

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

LGTM

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 30, 2024
stuartmorgan-g pushed a commit to flutter/packages that referenced this pull request Feb 1, 2024
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
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) a: animation Animation APIs a: text input Entering text in a text field or keyboard related problems f: cupertino flutter/packages/flutter/cupertino repository f: material design flutter/packages/flutter/material repository. f: scrolling Viewports, list views, slivers, etc. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants