Closed Bug 1608156 Opened 4 years ago Closed 4 years ago

border-image-repeat: repeat is incorrect

Categories

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

defect

Tracking

()

VERIFIED FIXED
mozilla74
Tracking Status
firefox74 --- verified

People

(Reporter: nical, Assigned: nical)

Details

Attachments

(4 files)

Look at the example at https://developer.mozilla.org/en-US/docs/Web/CSS/border-image-repeat the pattern is "squished" instead of preserving its aspect ratio as it should.

The issue is in brush_image.glsl: the stretch size is incorrectly computed in function of the uv_size, whereas it should be a function of the segment size (preserving the aspect ratio).

Attached image Before patch
Attached image After patch
Pushed by nsilva@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8b8a27242880
Fix border-image-repeat: repeat in WebRender. r=gw
https://hg.mozilla.org/integration/autoland/rev/47c89996827d
Adjust reftest expectations. r=kvark
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
Flags: qe-verify+

I tried to verify this fix, but I can't reproduce the initial issue on Nightly 74.0a1 (Build ID: 20200109094415), tried on Windows 10 x64, Ubuntu 18.04 and Mac OS X 10.15.

Nicolas, can you please confirm that the issue is fixed on your end?

Flags: needinfo?(nical.bugzilla)

Verified it is fixed.

Status: RESOLVED → VERIFIED
Flags: needinfo?(nical.bugzilla)

(In reply to Nicolas Silva [:nical] from comment #8)

Verified it is fixed.

Based on this comment, can I set the tracking flag of firefox 74 to verified?

Flags: needinfo?(nical.bugzilla)

Yep, done.

Flags: needinfo?(nical.bugzilla)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: