Closed Bug 1521656 Opened 6 years ago Closed 6 years ago

Perspective rendering has recently regressed quite a lot

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: emilio, Assigned: kvark)

References

Details

Attachments

(1 file)

All the the demos from bug 1504590 / bug 1505222 / etc have regressed on Nightly (before my patches too, heh :P).

Textures are deformed and what not, looks pretty bad. Will run mozregression ASAP.

Flags: needinfo?(emilio)
Priority: -- → P2

Culprit is bug 1505934.

Blocks: 1505934
Flags: needinfo?(dmalyshau)

See discussion above^

Flags: needinfo?(dmalyshau)
Depends on: 1522555

Perspective quality issues regressions with plane splitting should be addressed by https://phabricator.services.mozilla.com/D17854

Actually, there are still issues with those (i.e. the change addresses some things but not the others), not sure which ones are regressions and which aren't.

Clamp the UV interpolated from image quad corners in the
plane-splitting shader. In some cases, the values produced by Rust side
are outside of the target range. Coupled with the fact that Rust side
doesn't properly do near-plane splitting when evaluating those, it's
reasonable for the shader to do a little cleanup before using them.

Attachment #9040196 - Attachment description: WR clamp plane split UV coordinates → WR switch image UV quad coordinates into homogeneous space
Pushed by dmalyshau@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/82f168e2222d WR switch image UV quad coordinates into homogeneous space r=gw
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67

You rock Dzmytri :)

Flags: needinfo?(emilio)
Assignee: emilio → dmalyshau

(In reply to Emilio Cobos Álvarez (:emilio) from comment #9)

You rock Dzmytri :)

Err, I got the spelling wrong, but you still rock :)

Thanks Emilio!
If you haven't already created a bug specifically for local space scaling, let's make one. It would be very useful to have.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: