1685888 - Firefox screenshot looks different than what is displayed

Status: RESOLVED FIXED

(Core :: Web Painting, defect)

Description

[Benjamin Neff [:supertux88]]

Attachment: firefox_screenshots_garmin.png

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:86.0) Gecko/20100101 Firefox/86.0

Steps to reproduce:

I used the built-in screenshot tool of Firefox to take a screenshot of a Garmin activity, but the weather-info on the top-right of the map isn't fully visible on the screenshot, but is visible in Firefox or when using an external screenshot tool.

I found a random public activity on google (not mine) where you can reproduce it without the need of an account: https://connect.garmin.com/modern/activity/5726837845

Actual results:

The screenshot looks different than what is actually displayed in Firefox. See attachment with comparison of how the weather-info looked like when using the Firefox screenshot tool, the right half of the weather info is missing somehow (and they are also slightly blurry), and how it was actually displayed in Firefox (screenshots with an external tool when I created the above screenshots).

Expected results:

The screenshot should look the same as it is displayed in Firefox or when using an external screenshot tool.

Comment 1

[Emma Malysz]

Hey Matt,

Do you know why browser.tabs.captureTab is not capturing these images? I'm not too sure which component this belongs in

STR:

1. Open https://connect.garmin.com/modern/activity/5726837845
2. Try taking a screenshot of the upper right hand corner of the map (including the images next to the 11.1 indicator)

ER:
-Images are included in the screenshot

AR:
-Images are not included

Comment 2

[Matt Woodrow (:mattwoodrow)]

Attachment: Bug 1685888 - Use the correct offset to reference frame for items wrapping a transform item. r?mstange

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b63a8619b481
Use the correct offset to reference frame for items wrapping a transform item. r=mstange

Comment 4

[Alexandru Michis [:malexandru]]

Backed out changeset b63a8619b481 (Bug 1685888) for causing crashtest assertion failures in 1507961-1.html
Backout link: https://hg.mozilla.org/integration/autoland/rev/c2e2ebf84fd090d5453a62a0beca79b9f13669ef
Push with failures, failure log.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/67abb07887f7
Use the correct offset to reference frame for items wrapping a transform item. r=mstange

Comment 6

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/67abb07887f7