Closed
Bug 1883717
Opened 2 years ago
Closed 2 years ago
Incorrect pixel count of screenshots on macOS with component Screenshots enabled
Categories
(Firefox :: Screenshots, defect, P3)
Tracking
()
VERIFIED
FIXED
126 Branch
| Tracking | Status | |
|---|---|---|
| firefox-esr115 | --- | unaffected |
| firefox123 | --- | unaffected |
| firefox124 | --- | unaffected |
| firefox125 | --- | wontfix |
| firefox126 | --- | verified |
People
(Reporter: sbadau, Assigned: niklas)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
Found in
- Nightly 125.0a1
Affected versions
- Nightly 125.0a1
Tested platforms
- Affected platforms: macOS 13, macOS 14
- Unaffected platforms: Ubuntu 22.04, Windows 10 x64
Steps to reproduce
- Navigate to reddit.com
- Take a screenshot of one of the photos
- Observe the pixels counter.
Expected result
- The pixel count should match the resolution of the device.
Actual result
- The size of the screenshot is larger than expected. For example, on a MacBook device with a resolution of 1792 x 1120, the recorded size of the screenshot is 1,500x1,408 px. Please see the attached screenshot for more details.
Regression range
- This is not a regression, it's reproducible only when the component version of Screenshots is enabled.
Additional notes
- When using the extension version of Screenshots ('screenshots.browser.component.enabled' set to false in about:config) or the OS integrated tool, the size of the same picture is around 750 x 705 px.
- Issue reproducible on a MacBook Pro 16-inch, 2019.
| Assignee | ||
Comment 1•2 years ago
|
||
The reason this is happening is because macOS has a device pixel ratio of 2. This is reproducible on any device with a device pixel ratio that isn't 1.
Assignee: nobody → nbaumgardner
Status: NEW → ASSIGNED
Priority: -- → P3
| Assignee | ||
Comment 2•2 years ago
|
||
Pushed by nbaumgardner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f2cf1858cea8
Correctly display screenshot size in overlay. r=sfoster,firefox-desktop-core-reviewers
Comment 4•2 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox126:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 126 Branch
Comment 5•2 years ago
|
||
The patch landed in nightly and beta is affected.
:niklas, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox125towontfix.
For more information, please visit BugBot documentation.
Flags: needinfo?(nbaumgardner)
| Assignee | ||
Updated•2 years ago
|
Flags: needinfo?(nbaumgardner)
| Reporter | ||
Comment 6•2 years ago
|
||
Verified as fixed using the latest Nightly 126.0a1 on macOS 13.2 and macOS 14.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•