All layers with raster cache should use integer CTM (#33981)#34385
All layers with raster cache should use integer CTM (#33981)#34385CaseyHillers merged 2 commits intoflutter:flutter-2.13-candidate.0from
Conversation
|
This pull request was opened against a branch other than main. Since Flutter pull requests should not normally be opened against branches other than main, I have changed the base to main. 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 main, unless this is an intentional hotfix/cherrypick. |
|
I had to change a lot more due to some churn from a separate layer caching change, so now this needs additional verification from me |
1b92d09 to
33b6e83
Compare
33b6e83 to
bd453a7
Compare
|
Gold has detected about 1 new digest(s) on patchset 1. |
| MockCanvas::DrawCall{ | ||
| 1, MockCanvas::ClipRectData{layer_bounds, SkClipOp::kIntersect, | ||
| MockCanvas::kSoft_ClipEdgeStyle}}, | ||
| #ifndef SUPPORT_FRACTIONAL_TRANSLATION |
There was a problem hiding this comment.
These were removed since the code to raster cache clips hasn't been added yet
|
Gold has detected about 8 new digest(s) on patchset 2. |
Cherry pick for flutter/flutter#106523 . Manually verified with the example code that the artifacts are no longer present