1854980 - Screenshots captured but incorrect size

Status: RESOLVED WORKSFORME

(Firefox :: Screenshots, defect, P3)

Description

[ISHIKAWA, Chiaki]

Attachment: How I tried to capture the image.

I an using FF 117.0.1 under windows 10.

I have noticed that the screen capture I tried against youtube video (while the playing stopped) is recorded in incorrect size.

This happened a few times.

I am attaching how I tried to capture the
screen by selecting the most appropriate visual element(?) recognized by FF.
But you will see that the resulting copied data is not quite right.
It only contains the upper-left portion and does not have the whole area.

The video I tried was https://www.youtube.com/watch?v=95Xq6i-ERjY
and at 09:14 (the replay was stopped.)

However, something similar happened with the following URL, too.
https://www.youtube.com/watch?v=sLA0mdnBcaI

So I think the problem is with FF.

Comment 1

[ISHIKAWA, Chiaki]

Attachment: Copied data. Too small and contains only the upper-left portion.

I chose the video screen of youtube, sort of highlighted when I moved the mouse cursor around.
But when I copied the data using copy button at the lower-right corner, the result is too small data that contains only upper-left area.

The same operation worked in older FF.

Comment 2

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

Can you try to reproduce this issue when you enable screenshots.browser.component.enabled in about:config?

Comment 3

[ISHIKAWA, Chiaki]

(In reply to Sam Foster [:sfoster] (he/him) from comment #2)

Can you try to reproduce this issue when you enable screenshots.browser.component.enabled in about:config?

Thank you.

It WORKS now.

My suggestion then would be disabling the visual component selection feature if screenshots.browser.component.enabled is false.

I wonder why this attribute is set to false by default.

Comment 4

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

The screenshots.browser.component.enabled pref enables a re-written implementation of the screenshots feature, which fixes a number of bugs in the existing (web-extension-based) implementation. We're not quite ready to switch over to the new one by default - that work and the blockers are tracked in bug 1789727 if you are interested.

I'm going to leave this bug open but tag it so we'll know to close it when the component implementation ships - or to attempt a fix in the extension if for some reason that isn't going to happen in the near future.

Comment 5

[ISHIKAWA, Chiaki]

(In reply to Sam Foster [:sfoster] (he/him) from comment #4)

The screenshots.browser.component.enabled pref enables a re-written implementation of the screenshots feature, which fixes a number of bugs in the existing (web-extension-based) implementation. We're not quite ready to switch over to the new one by default - that work and the blockers are tracked in bug 1789727 if you are interested.

I'm going to leave this bug open but tag it so we'll know to close it when the component implementation ships - or to attempt a fix in the extension if for some reason that isn't going to happen in the near future.

Thank you for the clarification. I will monitor bug 1789727.
I have a feeling that not so many people capture web page image using built-in FF screen capture feature because this problem will be so easy to reproduce on youtube video.

Thank you again.

Comment 6

[Niklas Baumgardner [:niklas]]

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