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
[skwasm] Clip pictures if they go beyond the bounds of the window. #50887
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
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.
LGTM. Might be worth taking a look at the preroll and paint phases of the CanvasKit renderer to see how we deal with pictures that are outside the clip.
5e0d9ba35d Roll Skia from 636e136b0da9 to 1beae57ea0dd (3 revisions) (flutter/engine#51007) 63b6117514 [skwasm] Clip pictures if they go beyond the bounds of the window. (flutter/engine#50887) cd57dc9b4c GLES for scenario_app tests using Impeller (flutter/engine#51000) ef4c205239 Roll Skia from d5ca52bc7739 to 636e136b0da9 (8 revisions) (flutter/engine#51006) 8b240244ef [Fuchsia] Use shared gn-sdk from chromium (flutter/engine#50855)
I think this is causing the skwasm benchmarks to flake. See https://ci.chromium.org/ui/p/flutter/builders/luci.flutter.prod/Linux%20web_benchmarks_skwasm I'm going to revert this for now. |
Time to revert pull request flutter/engine/50887 has elapsed. |
…ndow. (flutter#50887)" This reverts commit 63b6117. The change is causing skwasm benchmarks to flake.
Manual revert here: #51067 |
…ndow. (#50887)" (#51067) This reverts commit 63b6117. The change is causing skwasm benchmarks to flake. See https://ci.chromium.org/ui/p/flutter/builders/luci.flutter.prod/Linux%20web_benchmarks_skwasm
This fixes flutter/flutter#143800, where we are attempting to capture an image that is way too large. We only need to render the part of the image that will be visible in the window.