corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1965106. Fix ClippedImage and DynamicImage GetIntrinsicSizeInAppUnits to actually be in app units. r=dholbert

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D249234
This commit is contained in:
Timothy Nikkel
2025-05-14 19:50:12 +00:00
committed by tnikkel@mozilla.com
parent 525d45a292
commit db08ac1033
2 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
image/ClippedImage.cpp
View File

@@ -212,8 +212,8 @@ ClippedImage::GetIntrinsicSizeInAppUnits(nsSize* aSize) {
return InnerImage()->GetIntrinsicSizeInAppUnits(aSize);
}
// XXXdholbert This probably needs to be scaled; see bug 1965106.
*aSize = nsSize(mClip.Width(), mClip.Height());
*aSize = nsSize(nsPresContext::CSSPixelsToAppUnits(mClip.Width()),
nsPresContext::CSSPixelsToAppUnits(mClip.Height()));
return NS_OK;
}

5
image/DynamicImage.cpp
View File

@@ -108,9 +108,10 @@ DynamicImage::GetIntrinsicSize(ImageIntrinsicSize* aIntrinsicSize) {
NS_IMETHODIMP
DynamicImage::GetIntrinsicSizeInAppUnits(nsSize* aSize) {
// XXXdholbert This probably needs to be scaled; see bug 1965106.
IntSize intSize(mDrawable->Size());
*aSize = nsSize(intSize.width, intSize.height);
*aSize = nsSize(nsPresContext::CSSPixelsToAppUnits(intSize.width),
nsPresContext::CSSPixelsToAppUnits(intSize.height));
return NS_OK;
}