Closed Bug 1427968 Opened 2 years ago Closed 2 years ago

Flickering playing media.ccc.de videos in fullscreen on macOS

Categories

(Core :: Web Painting, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox-esr52 --- unaffected
firefox57 --- unaffected
firefox58 --- unaffected
firefox59 --- fixed

People

(Reporter: freesamael, Assigned: mattwoodrow)

References

Details

(Keywords: regression)

Attachments

(2 files)

STR:
1. Play https://media.ccc.de/v/34c3-8941-console_security_-_switch/#video&t=9 (or any other videos on that website) on macOS.
2. Click the fullscreen icon.

Actual Result:
Flickering at the top and bottom of the video. Looks like web content overlaps the video. Screen recording attached.

There weren't enough macOS builds for mozregression to narrow down:

https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=0843e4e6cb1d6e32a35ce4d7014bd250c24f74fa&tochange=1a76c0899bc68acd6b77dd1796870eac3fb0f112

git-bisect from there eventually points me to 

Bug 1344971 - Part 5: Share DisplayItemData lookups when we can in FrameLayerBuilder. r=mstange
Assignee: nobody → matt.woodrow
Previously we stored the OutOutFlowDisplayData for all visible frames, regardless of whether they were contained in the dirty rect or not.

In the current state, we only return OutOfFlowDisplayData to callers if we have the force descend into flag, which is true for frames that were both visible and dirty.

The callers behave differently when we don't return the display data (even when the dirty rect on it is empty), so this takes us back to always returning it when we can.
Attachment #8940546 - Flags: review?(mikokm)
Priority: -- → P2
Comment on attachment 8940546 [details] [diff] [review]
Don't check for the force descend flag when returning the OOFDisplayData

Review of attachment 8940546 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM.
Attachment #8940546 - Flags: review?(mikokm) → review+
Pushed by mwoodrow@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9fc13d9e3092
Don't check for the force descend flag when checking for the presence of OutOfFlowDisplayData. r=miko
https://hg.mozilla.org/mozilla-central/rev/9fc13d9e3092
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Duplicate of this bug: 1427558
QA Whiteboard: [good first verify]
I tried reproducing this issue on Firefox Nightly 59.0a1 (2018-01-03) under macOS 10.12.6, using the steps from Comment 0, but that webpage uses another video player, on which the issue is not reproducible.

Samael Wang, have you reproduced the issue on a specific version of macOS? Are there any pref changes involved?

Please let me know if you are still able to reproduce the issue.
Flags: needinfo?(freesamael)
Flags: needinfo?(freesamael)
You need to log in before you can comment on or make changes to this bug.