Skip to content

Conversation

@ksokolovskyi
Copy link
Contributor

Description

Tests

  • Updates ticker_test.dart to test Ticker object creation and disposal dispatching.

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
  • I signed the [CLA].
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is [test-exempt].
  • All existing and new tests are passing.

@github-actions github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Nov 3, 2023
@ksokolovskyi
Copy link
Contributor Author

cc @polina-c

@polina-c polina-c added the a: leak tracking Issues and PRs related to memory leaks detected by leak_tracker label Nov 6, 2023
@goderbauer goderbauer requested a review from polina-c November 7, 2023 23:11
@polina-c polina-c merged commit ca384b8 into flutter:master Nov 8, 2023
@ksokolovskyi
Copy link
Contributor Author

@polina-c, thanks a lot for the review!

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 8, 2023
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Nov 8, 2023
flutter/flutter@5a6a322...4b4a1fe

2023-11-08 [email protected] Roll Flutter Engine from 117d47aa3f88 to b0310da3254d (1 revision) (flutter/flutter#138096)
2023-11-08 [email protected] Run a couple of iOS tests in presubmit (flutter/flutter#138089)
2023-11-08 [email protected] Roll Flutter Engine from 3e3be5e33bda to 117d47aa3f88 (20 revisions) (flutter/flutter#138091)
2023-11-08 [email protected] Roll Packages from be18d28 to 94c7623 (6 revisions) (flutter/flutter#138086)
2023-11-08 [email protected] Ticker should dispatch creation and disposal events. (flutter/flutter#137844)
2023-11-08 [email protected] [flutter_tools] Fix local engine preview device (flutter/flutter#138046)
2023-11-07 [email protected] Revert "Add no-shuffle to reorderable_list_test.dart" (flutter/flutter#137715)
2023-11-07 [email protected] Document where `Curves` curves correspond to CSS easing functions (flutter/flutter#137318)
2023-11-07 [email protected] Roll Flutter Engine from f8961d203039 to 3e3be5e33bda (1 revision) (flutter/flutter#138039)
2023-11-07 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Use no-response from cocoon." (flutter/flutter#138042)
2023-11-07 [email protected] Use no-response from cocoon. (flutter/flutter#138037)
2023-11-07 [email protected] Roll Flutter Engine from 1b20752e2a63 to f8961d203039 (6 revisions) (flutter/flutter#138034)
2023-11-07 [email protected] Added an AnimationController API doc example (flutter/flutter#137975)
2023-11-07 [email protected] [flutter_tools] toolexit when using plugins with preview device (flutter/flutter#136936)
2023-11-07 [email protected] Roll Flutter Engine from 38895fbd9468 to 1b20752e2a63 (1 revision) (flutter/flutter#138020)
2023-11-07 [email protected] Roll Flutter Engine from a9e0f9514f78 to 38895fbd9468 (1 revision) (flutter/flutter#138016)
2023-11-07 [email protected] Move Skia new_gallery_transition_perf on a02 from staging to prod (flutter/flutter#138013)
2023-11-07 [email protected] Adaptive `Switch` (flutter/flutter#130425)
2023-11-07 [email protected] Roll Flutter Engine from e2810f07abb5 to a9e0f9514f78 (1 revision) (flutter/flutter#138005)
2023-11-07 [email protected] Roll Flutter Engine from b91400976b4a to e2810f07abb5 (1 revision) (flutter/flutter#138002)
2023-11-07 [email protected] [native assets] Tool exit on build failure (flutter/flutter#137995)

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://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 Feb 16, 2024
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 framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants