Closed Bug 1543944 Opened 5 years ago Closed 5 years ago

Twitter - background image of video tweet is not cleared while video playing

Categories

(Core :: Graphics: Layers, defect)

68 Branch
x86_64
Windows 10
defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla68
Tracking Status
firefox68 --- verified

People

(Reporter: euthanasia_waltz, Assigned: mstange)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Attached file about_support.txt

STR:

  1. Open https://twitter.com/nashiko_cos/status/1116634094685671424 for example

ER:
Background image turns into black when video start playing.

AR:
Background image remains while video playing.

This occurs with gfx.webrender.force-disabled=true on my pc.
With gfx.webrender.force-disabled=false by default has no problem.

mozregression:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=1d58cd8e6457bae9de68d0470cf5236479ea7b2c&tochange=c2b1753a416b3201552108700ecd3d17686d5d4f

Thanks for reporting and finding out the bug regressed this!

Though I can't reproduce the issue on Linux, I can reproduce it on Windows 10.

Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(matt.woodrow)
Regressed by: 1538969

The patch in bug 1538969 marks the entire bounds of the item as opaque. However, the video isn't actually displayed in the entire bounds, it is centered within the bounds depending on aspect ratio and rotation. There's code to calculate that rectangle in nsVideoFrame::BuildLayer and in CreateWebRenderCommands and it's not very straightforward - especially the handling of the rotation makes it a bit tricky.

I recommend we back out just the patch that added the GetOpaqueRegion implementation. Once Matt is back from PTO, he can take another stab at it.

Assignee: nobody → mstange
Status: NEW → ASSIGNED
Flags: needinfo?(matt.woodrow)
Pushed by mstange@themasta.com:
https://hg.mozilla.org/integration/autoland/rev/c992709a5b80
Don't mark nsDisplayVideo as opaque for now because we were using a rectangle that's too big. r=tnikkel
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
Flags: qe-verify+

Hello,
Reproduced the issue with Firefox 68.0a1 (20190412215855) on Windows 10 x64 using steps from comment 0.
The issue is verified using Firefox 68.0b7 (20190603181408) on Windows 10x64.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: