Skip to content
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

Fix leaking curved animation. #146644

Merged
merged 4 commits into from
Apr 11, 2024
Merged

Fix leaking curved animation. #146644

merged 4 commits into from
Apr 11, 2024

Conversation

polina-c
Copy link
Contributor

No description provided.

@github-actions github-actions bot added framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. labels Apr 11, 2024
@polina-c polina-c added the a: leak tracking Issues and PRs related to memory leaks detected by leak_tracker label Apr 11, 2024
@@ -189,7 +190,7 @@ void main() {
expect(element.dirty, isTrue);
}, skip: isBrowser); // TODO(yjbanov): cupertino does not work on the Web yet: https://github.com/flutter/flutter/issues/41920

testWidgets('RangeSlider relayout upon system fonts changes', (WidgetTester tester) async {
testWidgets('RangeSlider relayout upon system fonts changes more than once', (WidgetTester tester) async {
Copy link
Contributor

Choose a reason for hiding this comment

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

don't understand why this test name changed

Copy link
Contributor Author

@polina-c polina-c Apr 11, 2024

Choose a reason for hiding this comment

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

There are two tests with the same name in this file.

Copy link
Contributor

@jacob314 jacob314 left a comment

Choose a reason for hiding this comment

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

lgtm

@polina-c polina-c added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 11, 2024
@auto-submit auto-submit bot merged commit 5ebc940 into flutter:master Apr 11, 2024
76 checks passed
@polina-c polina-c deleted the fix-leaks1 branch April 11, 2024 21:38
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 12, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Apr 12, 2024
flutter/flutter@557fbf5...53cba24

2024-04-12 magder@google.com Allow A/B tests to run as just aggregator of local engine benchmarks (flutter/flutter#146479)
2024-04-12 chris@bracken.jp [Windows] Drop support for Windows 7/8 apps in template (flutter/flutter#146668)
2024-04-12 engine-flutter-autoroll@skia.org Roll Flutter Engine from 2d85d1225d18 to 6b37b170998e (1 revision) (flutter/flutter#146666)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from 16b0cfd85617 to 2d85d1225d18 (4 revisions) (flutter/flutter#146663)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from 0d36479e1d55 to 16b0cfd85617 (1 revision) (flutter/flutter#146662)
2024-04-11 jhy03261997@gmail.com Update the gradle task to add fallback scheme and host if needed when retrieving deep links. (flutter/flutter#146470)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from 98a8ad144597 to 0d36479e1d55 (2 revisions) (flutter/flutter#146653)
2024-04-11 polinach@google.com Fix leaking curved animation. (flutter/flutter#146644)
2024-04-11 103135467+sealesj@users.noreply.github.com Refactor verify codesigned (flutter/flutter#146450)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from d8560d495d9f to 98a8ad144597 (9 revisions) (flutter/flutter#146645)
2024-04-11 engine-flutter-autoroll@skia.org Roll Packages from 2c15d86 to e98839a (3 revisions) (flutter/flutter#146638)

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
gilnobrega pushed a commit to gilnobrega/flutter that referenced this pull request Apr 22, 2024
TecHaxter pushed a commit to TecHaxter/flutter_packages that referenced this pull request May 22, 2024
…r#6509)

flutter/flutter@557fbf5...53cba24

2024-04-12 magder@google.com Allow A/B tests to run as just aggregator of local engine benchmarks (flutter/flutter#146479)
2024-04-12 chris@bracken.jp [Windows] Drop support for Windows 7/8 apps in template (flutter/flutter#146668)
2024-04-12 engine-flutter-autoroll@skia.org Roll Flutter Engine from 2d85d1225d18 to 6b37b170998e (1 revision) (flutter/flutter#146666)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from 16b0cfd85617 to 2d85d1225d18 (4 revisions) (flutter/flutter#146663)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from 0d36479e1d55 to 16b0cfd85617 (1 revision) (flutter/flutter#146662)
2024-04-11 jhy03261997@gmail.com Update the gradle task to add fallback scheme and host if needed when retrieving deep links. (flutter/flutter#146470)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from 98a8ad144597 to 0d36479e1d55 (2 revisions) (flutter/flutter#146653)
2024-04-11 polinach@google.com Fix leaking curved animation. (flutter/flutter#146644)
2024-04-11 103135467+sealesj@users.noreply.github.com Refactor verify codesigned (flutter/flutter#146450)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from d8560d495d9f to 98a8ad144597 (9 revisions) (flutter/flutter#146645)
2024-04-11 engine-flutter-autoroll@skia.org Roll Packages from 2c15d86 to e98839a (3 revisions) (flutter/flutter#146638)

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
arc-yong pushed a commit to Arctuition/packages-arc that referenced this pull request Jun 14, 2024
…r#6509)

flutter/flutter@557fbf5...53cba24

2024-04-12 magder@google.com Allow A/B tests to run as just aggregator of local engine benchmarks (flutter/flutter#146479)
2024-04-12 chris@bracken.jp [Windows] Drop support for Windows 7/8 apps in template (flutter/flutter#146668)
2024-04-12 engine-flutter-autoroll@skia.org Roll Flutter Engine from 2d85d1225d18 to 6b37b170998e (1 revision) (flutter/flutter#146666)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from 16b0cfd85617 to 2d85d1225d18 (4 revisions) (flutter/flutter#146663)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from 0d36479e1d55 to 16b0cfd85617 (1 revision) (flutter/flutter#146662)
2024-04-11 jhy03261997@gmail.com Update the gradle task to add fallback scheme and host if needed when retrieving deep links. (flutter/flutter#146470)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from 98a8ad144597 to 0d36479e1d55 (2 revisions) (flutter/flutter#146653)
2024-04-11 polinach@google.com Fix leaking curved animation. (flutter/flutter#146644)
2024-04-11 103135467+sealesj@users.noreply.github.com Refactor verify codesigned (flutter/flutter#146450)
2024-04-11 engine-flutter-autoroll@skia.org Roll Flutter Engine from d8560d495d9f to 98a8ad144597 (9 revisions) (flutter/flutter#146645)
2024-04-11 engine-flutter-autoroll@skia.org Roll Packages from 2c15d86 to e98839a (3 revisions) (flutter/flutter#146638)

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
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 autosubmit Merge PR when tree becomes green via auto submit App f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants