-
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
[canvaskit] Optimize CanvasKit platform views in special cases #28087
[canvaskit] Optimize CanvasKit platform views in special cases #28087
Conversation
/// on which direction the list is scrolling. One or more views will be added | ||
/// to the beginning of the list, and one or more views will be removed from | ||
/// the end of the list, with the order of the unchanged middle views | ||
/// remaining the same. |
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.
Checkout _insertChildDomNodes. With the same amount of code (reusing longestIncreasingSubsequence
) we should be able to find the optimal solution for all situations, including the scroll case.
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.
Added a TODO. That refactor would also necessitate copying more code, for instance to generate the index maps.
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.
Diff the old and new platform view list and attempt to elminate most DOM manipulation in the common case.
Addresses flutter/flutter#86868 for CanvasKit
Pre-launch Checklist
writing and running engine tests.
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.