-
Notifications
You must be signed in to change notification settings - Fork 6k
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
Make sure FlutterViewController flushs all pending touches when no longer active #8400
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From UIEvent documentation:
You should never retain an event object or any object returned from an event object. If you need to retain data outside of the responder method you use to process that data, copy that data from the UITouch or UIEvent object to your local data structures.
Thanks. Done. |
@@ -39,6 +39,7 @@ @implementation FlutterViewController { | |||
BOOL _initialized; | |||
BOOL _viewOpaque; | |||
BOOL _engineNeedsLaunch; | |||
NSMutableSet* _ongoingTouches; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please add a type to this or just use an std::set<int64_t>
please? Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
…en no longer active (flutter/engine#8400)
flutter/engine@219bf59...8b74cba git log 219bf59..8b74cba --no-merges --oneline 8b74cba Make sure FlutterViewController flushs all pending touches when no longer active (flutter/engine#8400) 06cab81 Roll src/third_party/skia e4628b10722f..51874e3e3712 (2 commits) (flutter/engine#8445) 5460926 Roll src/third_party/dart 9814cb1b9c..896f1c4bb9 (4 commits) 694fc56 Build windows engine on GCE (flutter/engine#8442) The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff (chinmaygarde@google.com), and stop the roller if necessary.
…en no longer active (flutter#8400)" This reverts commit b2a488d.
…en no longer active (flutter#8400)" This reverts commit b2a488d.
Fixes flutter/flutter#30342
Background #6145
@kangwang1988 try this. I wasn't really good at repro'ing but which this patch and your demo project, I can't reproduce the issue anymore.