Closed Bug 1678192 Opened 4 years ago Closed 4 years ago

Color shifting (amber, blue, gray) on video (wr-sw)

Categories

(Core :: Graphics: WebRender, defect)

defect

Tracking

()

RESOLVED FIXED
85 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox83 --- unaffected
firefox84 --- unaffected
firefox85 --- fixed

People

(Reporter: yoasif, Assigned: lsalzman)

References

(Regression, )

Details

(Keywords: nightly-community, regression)

Attachments

(4 files)

Attached image image.png

Visit: https://twitter.com/genericdemocrat/status/1329224796048134160

What happens: shifting colors on video

14:08.53 INFO: Narrowed integration regression window from [b9aa322c, 6d5a7637] (3 builds) to [b9aa322c, 5e315caa] (2 builds) (~1 steps left)
14:08.54 INFO: No more integration revisions, bisection finished.
14:08.54 INFO: Last good revision: b9aa322c7e11ee52c0079401621b858d4bbb4701
14:08.54 INFO: First bad revision: 5e315caacc2730e114a86f32272e4f7d289ef19d
14:08.55 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=b9aa322c7e11ee52c0079401621b858d4bbb4701&tochange=5e315caacc2730e114a86f32272e4f7d289ef19d

Attached file about:support
Has Regression Range: --- → yes
Has STR: --- → yes
Regressed by: 1675990
Flags: needinfo?(lsalzman)
Severity: -- → S2

The SWGL fast-paths operate in terms of UVs that have already been quantized.
The quantized ivec2s were getting silently converted back to vec2s which were
then re-quantized inside the planar texture lookups. The quantization has been
moved as such to only now happen in the normal vec4 result paths as it should
have been in the first place, and the conversion constructor has been made
explicit to avoid this oversight in the future.

Assignee: nobody → lsalzman
Status: NEW → ASSIGNED

Set release status flags based on info from the regressing bug 1675990

Pushed by lsalzman@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/065da45c990f don't quantize UVs in planar texture lookups. r=jrmuizel
Pushed by lsalzman@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e5a5332c48a4 add wrench test for clipped yuv image. r=jrmuizel
Flags: needinfo?(lsalzman)
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: