Closed Bug 1659418 Opened 4 years ago Closed 4 years ago

Text flickers on 3D CSS cube with WebRender on

Categories

(Core :: Graphics: WebRender, defect, P3)

defect

Tracking

()

VERIFIED FIXED
83 Branch
Tracking Status
firefox-esr68 --- disabled
firefox-esr78 --- disabled
firefox79 --- wontfix
firefox80 --- wontfix
firefox81 --- wontfix
firefox82 --- wontfix
firefox83 --- verified

People

(Reporter: rdoghi, Assigned: kvark)

References

(Blocks 1 open bug)

Details

(Keywords: correctness, regression)

Attachments

(4 files)

Attached video 2020-08-17_14h10_56.mp4

[Affected versions]:
Nightly 81.0a1 (2020-08-16)

[Affected platforms]:
Platforms: Mac Os, Windows 10

[Preconditions]
Have gfx.webrender.all - true

Steps :

  1. Launch the Firefox browser, reach https://paulrhayes.com/experiments/cube-3d/
  2. Hit the UP arrow button once and then the Left arrow button twice.

Expected Results :
Text should remain static no flickers or visual glitches should occur.

Actual Results :
The Text inside the Cube flickers when the Cube rotates.

Dzmitry, could you have a look?

Severity: -- → S3
Flags: needinfo?(dmalyshau)
Priority: -- → P3

First bad is 2018-09-11 (as in bug 1655732 comment 1).

Has STR: --- → yes
OS: Windows 10 → All
Hardware: Desktop → All
See Also: → 1655732
Assignee: nobody → dmalyshau
Status: NEW → ASSIGNED
Flags: needinfo?(dmalyshau)

Figured out the problem is in rectangle re-projection within get_raster_rects. It's not a plane splitting issue.

Fun looking at the inverse projection points from a rect. The interesting bit here is one of the points in the left top corner.

Attached file cube.html.zip

Reduced test case (has HTML and a few CSS)

Our inverse projection logic was correct, just not fully complete.
The problem is that it didn't take into account that the projected point can be in negative hemisphere.
Now we are checking for this and bailing out of building a tight rect if this is the case.

Pushed by dmalyshau@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cc76c766bd81
Handle inverse projection negative hemisphere r=nical
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch

This issue is Verified as fixed in our Latest Beta build 83.0b4 on Windows as well as Mac osX 10.15.

Status: RESOLVED → VERIFIED
Regressions: 1696807
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: