Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@jason-simmons
Copy link
Member

@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie on the #hackers channel in Chat.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@GaryQian
Copy link
Contributor

Implementation LGTM, can you add a quick test for this?

@jason-simmons
Copy link
Member Author

Added a test

Copy link
Contributor

@GaryQian GaryQian left a comment

Choose a reason for hiding this comment

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

Thanks!

@jason-simmons jason-simmons added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Sep 22, 2020
@fluttergithubbot fluttergithubbot merged commit 11befe4 into flutter:master Sep 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 22, 2020
@pcsosinski
Copy link

removing cp label in favor of the issue which is tagged

christopherfujino added a commit that referenced this pull request Sep 24, 2020
* Update 1.22 engine to use Dart 2.10.0-110.5.beta
* Dark mode friendly iOS debugging message (#21277)
* Update FlutterViewController.mm (#21362)
* Tweaked the label here a little for style ("apps", not "application" -- plural and shorter); and "launching" rather than "re-launching"?
* Fix iOS platform view's mask view blocking touch events. (#21286)
* Disconnect the view's AndroidKeyProcessor when detaching from the engine (#21307)
* Remove extraneous window inset call on IME animation (#21213)
* Retain the WindowInsetsAnimation callback if code shrinking is enabled (#21330)

Co-authored-by: Jenn Magder <[email protected]>
Co-authored-by: Tim Sneath <[email protected]>
Co-authored-by: Chris Yang <[email protected]>
Co-authored-by: Jason Simmons <[email protected]>
Co-authored-by: Gary Qian <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes platform-android waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Add2App] Android leaks the FlutterView from 1.22.0-12.0.pre

5 participants