1863148 - Firefox saves screenshots under wrong name (doesn't include correct page title)

Status: RESOLVED WORKSFORME

(Firefox :: Screenshots, defect, P3)

Description

[Przemysław Siekiera]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/119.0

Steps to reproduce:

Example steps to reproduce:

1. Go to https://www.youtube.com/
2. Accept or reject the cookies if it asks you (if no cookie modal was displayed, continue to the next step)
3. Right-click on any video and select “Open Link in New Window”
4. (New widow should open up with the video) Click on some other video to the right of the video player (from the list on the right side)
5. When the page with the next video loads, right-click on page background area and select “Take Screenshot” → “Save visible” → “Download”

Actual results:

The screenshot file name includes the title of the first video you opened, not the current one.

Expected results:

The screenshot name should include the page title of currently viewed video.

Comment 1

[Przemysław Siekiera]

Attachment: Screenshot 2023-11-04 150731.png

Comment 2

[Przemysław Siekiera]

Attachment: Screenshot 2023-11-04 150753.png

The screenshot name includes “THE FALL GUY Trailer (2024) Ryan Gosling Emily Blunt” (the page title I opened three pages ago) even though I'm currently viewing the page for the “The Witcher / Blaviken Market Fight Scene (Geralt Butchers Renfri's Gang)” video.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Firefox::Screenshots' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 4

[Ardelean Oana, Desktop Test Engineering [:oardelean]]

Managed to reproduce this issue on:

• Firefox 119.0.1;
• Firefox 120.0b7;
• Nightly 121.0a1;

Tested and reproduced on:

• Ubuntu 22;
• Windows 10;
• macOS 12;

Setting as NEW so the developing team can have a look.

Comment 5

[Sam Foster [:sfoster] (he/him)]

Thanks for filing this. It looks like this issue is fixed in the new (not yet default) Screenshots implementation. You can use this by going to about:config and setting the preference screenshots.browser.component.enabled to true.

Comment 6

[Niklas Baumgardner [:niklas]]

This is fixed in the screenshots component. We enabled the component in bug 1870127