-
Notifications
You must be signed in to change notification settings - Fork 6k
All layers with raster cache should use integer CTM (#33981) #34385
All layers with raster cache should use integer CTM (#33981) #34385
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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