Skip to content

ScrollDragController should dispatch creation and disposal events.#138800

Merged
polina-c merged 2 commits intoflutter:masterfrom
ksokolovskyi:scroll-drag-controller-instrumentation
Dec 7, 2023
Merged

ScrollDragController should dispatch creation and disposal events.#138800
polina-c merged 2 commits intoflutter:masterfrom
ksokolovskyi:scroll-drag-controller-instrumentation

Conversation

@ksokolovskyi
Copy link
Copy Markdown
Contributor

Description

Tests

  • Updates scroll_activity_test.dart to test ScrollDragController object creation and disposal events 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 framework flutter/packages/flutter repository. See also f: labels. f: scrolling Viewports, list views, slivers, etc. labels Nov 21, 2023
@ksokolovskyi
Copy link
Copy Markdown
Contributor Author

cc @polina-c

@goderbauer goderbauer requested a review from polina-c November 21, 2023 23:13
@polina-c polina-c added the a: leak tracking Issues and PRs related to memory leaks detected by leak_tracker label Nov 23, 2023
@polina-c
Copy link
Copy Markdown
Contributor

This PR should hold till design discussion for leak_tracker (early December)

@polina-c polina-c merged commit 6d4d012 into flutter:master Dec 7, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Dec 8, 2023
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 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