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
fix: copy pixels in AddImageSkiaRepFromBuffer #17843
Conversation
Release Notes Persisted
|
I was unable to backport this PR to "3-1-x" cleanly; |
I was unable to backport this PR to "4-1-x" cleanly; |
I was unable to backport this PR to "5-0-x" cleanly; |
/trop run backport-to 5-0-x |
The backport process for this PR has been manually initiated, |
I was unable to backport this PR to "5-0-x" cleanly; |
A maintainer has manually backported this PR to "5-0-x", please check out #17863 |
A maintainer has manually backported this PR to "4-1-x", please check out #17862 |
A maintainer has manually backported this PR to "3-1-x", please check out #17861 |
Description of Change
Changing buffer data should not modify the
nativeImage
pixels, makes it consistent with PNG / JPEG behavior, which always copies the data.Checklist
npm test
passesRelease Notes
Notes: Fixed inconsistent behavior where modifying the buffer would change
nativeImage
pixels, while decoded PNG / JPEG data is a copy.