Closed Bug 1751913 Opened 3 years ago Closed 3 years ago

Perma REFTEST TEST-UNEXPECTED-FAIL | dom/media/test/reftest/color_quads/720p.png.bt709.bt709.pc.yuv420p.vp9.webm == dom/media/test/reftest/color_quads/720p.png | image comparison, max difference: 38, when Gecko 98 merges to Beta on 2022-02-18

Categories

(Core :: Audio/Video: Playback, defect, P2)

defect

Tracking

()

VERIFIED FIXED
98 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox96 --- unaffected
firefox97 --- unaffected
firefox98 + verified

People

(Reporter: ctuns, Assigned: aryx)

References

(Depends on 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Flags: needinfo?(alwu)

Just want to make sure, this only happens when running on Beta?

Flags: needinfo?(alwu) → needinfo?(ctuns)

This should get fixed by bug 1748540. This is an existing issue but got workaround already for a while by adding fuzzy-if(winWidget&&swgl,31-38,8240-184080) and comparing webm&mp4 to av1.

Actually all types of videos (webm, mp4 and av1) are not able to display correctly in full range of color on SW WR on Windows. Bug 1713276 re-adjusts the comparison to make webm&mp4 compare to PNG, which expose the issue again. I think we should not add fuzzy range to workaround it again, it should get a proper fixed in bug 1748540.

Depends on: 1750858
Depends on: 1748540
No longer depends on: 1750858

(In reply to Alastor Wu [:alwu] from comment #1)

Just want to make sure, this only happens when running on Beta?

Afaik yes, I haven't notice to be present on central or autoland.

Flags: needinfo?(ctuns)
Has Regression Range: --- → yes

As I mentioned in comment2, this issue has been existing for a while, so we should not simply make it fuzzy to pass the test, and should fix the root cause completely.

Severity: -- → S3
Priority: -- → P3

Changing the priority to p2 as the bug is tracked by a release manager for the current nightly.
See What Do You Triage for more information

Priority: P3 → P2

This started with direct decoding of video frames from data (bug 1713276)
which also changed test expectations.

Assignee: nobody → aryx.bugmail
Status: NEW → ASSIGNED
Pushed by archaeopteryx@coole-files.de: https://hg.mozilla.org/integration/autoland/rev/1e8fbd2fd53d fuzz annotation for video playback with Windows and AddressSanitizer and beta or release. r=alwu DONTBUILD

The patch seems to be working in today's beta-sim

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch

Verified fixed as of today's beta-sim

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

Attachment

General

Created:
Updated:
Size: