Skip to content

[framework] ensure ink sparkle is disposed (#104569) #105144

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.

Sign up for GitHub

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

Jump to bottom
Merged
itsjustkevin merged 4 commits into from
Jun 2, 2022
Merged

[framework] ensure ink sparkle is disposed (#104569) #105144

itsjustkevin merged 4 commits into from
Jun 2, 2022

Conversation

@dnfield
Copy link
Contributor

@dnfield dnfield commented Jun 1, 2022

Cherry pick #104569 into stable

Fixes #104580

@dnfield dnfield requested a review from itsjustkevin June 1, 2022 16:30
@flutter-dashboard flutter-dashboard bot changed the base branch from stable to master June 1, 2022 16:30
@flutter-dashboard flutter-dashboard bot requested a review from keyonghan as a code owner June 1, 2022 16:30
@flutter-dashboard
Copy link

flutter-dashboard bot commented Jun 1, 2022

This pull request was opened against a branch other than master. Since Flutter pull requests should not normally be opened against branches other than master, I have changed the base to master. If this was intended, you may modify the base back to stable. See the Release Process for information about how other branches get updated.

Reviewers: Use caution before merging pull requests to branches other than master, unless this is an intentional hotfix/cherrypick.

@flutter-dashboard flutter-dashboard bot added a: text input Entering text in a text field or keyboard related problems d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos documentation engine flutter/engine related. See also e: labels. f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels. labels Jun 1, 2022
@dnfield dnfield changed the base branch from master to stable June 1, 2022 16:31
@dnfield dnfield changed the base branch from stable to flutter-2.13-candidate.0 June 1, 2022 16:33
@flutter-dashboard
Copy link

flutter-dashboard bot commented Jun 1, 2022

This pull request was opened against a branch other than master. Since Flutter pull requests should not normally be opened against branches other than master, I have changed the base to master. If this was intended, you may modify the base back to flutter-2.13-candidate.0. See the Release Process for information about how other branches get updated.

Reviewers: Use caution before merging pull requests to branches other than master, unless this is an intentional hotfix/cherrypick.

@flutter-dashboard flutter-dashboard bot changed the base branch from flutter-2.13-candidate.0 to master June 1, 2022 16:33
@dnfield dnfield changed the base branch from master to 3.0.1 June 1, 2022 16:37
@flutter-dashboard flutter-dashboard bot changed the base branch from 3.0.1 to master June 1, 2022 16:37
@flutter-dashboard
Copy link

flutter-dashboard bot commented Jun 1, 2022

This pull request was opened against a branch other than master. Since Flutter pull requests should not normally be opened against branches other than master, I have changed the base to master. If this was intended, you may modify the base back to 3.0.1. See the Release Process for information about how other branches get updated.

Reviewers: Use caution before merging pull requests to branches other than master, unless this is an intentional hotfix/cherrypick.

@dnfield dnfield changed the base branch from master to flutter-2.13-candidate.0 June 1, 2022 16:37
itsjustkevin
itsjustkevin approved these changes Jun 2, 2022
View reviewed changes
Copy link
Contributor

@itsjustkevin itsjustkevin left a comment

Choose a reason for hiding this comment

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

LGTM

@itsjustkevin itsjustkevin merged commit 8572eac into flutter:flutter-2.13-candidate.0 Jun 2, 2022
@itsjustkevin itsjustkevin mentioned this pull request Jun 7, 2022
Merged
@laiiihz laiiihz mentioned this pull request Jun 9, 2022
Closed
bernard-lee added a commit to bernard-lee/flutter that referenced this pull request Jun 29, 2022
@bernard-lee
Merge branch 'stable' of https://github.com/flutter/flutter into stable
bad536b 
* 'stable' of https://github.com/flutter/flutter: (1203 commits)
  'Update Engine revision to ffe7b86a1e5b5cb63c8385ae1adc759e372ee8f4 for stable release 3.0.3' (flutter#106431)
  [flutter_releases] Removing minor iOS version filter value from ci.yaml (flutter#105059) (flutter#105629)
  [flutter_releases] Flutter stable 3.0.2 Framework Cherrypicks (flutter#105528)
  [framework] ensure ink sparkle is disposed (flutter#104569) (flutter#105144)
  [CP] Fix Flutter doctor crash on Windows caused by bad UTF8 from vswhere.exe (flutter#105153)
  [tool][web] Use 'constructor' instead of 'public field declarations' in flutter.js (Safari 13) (flutter#105162)
  [flutter_releases] Cherrypicks for SliverReorderableList and Slider regressions (flutter#105141)
  'Update Engine revision to caaafc5604ee9172293eb84a381be6aadd660317 for stable release 3.0.1' (flutter#104213)
  [flutter_releases] Flutter stable 2.13.0 Framework Cherrypicks (flutter#103375)
  [flutter_releases] Flutter beta 2.13.0-0.4.pre Framework Cherrypicks (flutter#103101)
  Enforce cpu explicitly for Mac devicelab test beds (flutter#101871) (flutter#102685)
  [flutter_releases] Flutter beta 2.13.0-0.3.pre Framework Cherrypicks (flutter#102620)
  [flutter_releases] Upgrade dwds to 12.1.1 (flutter#101546)
  [flutter_releases] Flutter beta 2.13.0-0.2.pre Framework Cherrypicks (flutter#102193)
  [flutter_releases] Flutter beta 2.12.0-4.1.pre Framework Cherrypicks (flutter#101771)
  [flutter_releases] Cherrypick engine to c341645 (flutter#101608)
  Revert "Refactor `ToggleButtons` (remove `RawMaterialButton`) (flutter#99493)" (flutter#101538)
  [Cherrypick] Partial revert of super params in tools (flutter#101436) (flutter#101527)
  Roll Engine from e7e7ca1 to b48d65e (10 revisions) (flutter#101370)
  [flutter_tools] port bash script to use sysctl not uname on macOS (flutter#101308)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@itsjustkevin itsjustkevin itsjustkevin approved these changes

@keyonghan keyonghan Awaiting requested review from keyonghan

Assignees

No one assigned

Labels

a: text input Entering text in a text field or keyboard related problems c: contributor-productivity Team-specific productivity, code health, technical debt. d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos engine flutter/engine related. See also e: labels. f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[CP] dispose ink sparkle

2 participants

@dnfield @itsjustkevin