Invalid output format in D3D11SharedHandIemage::GetAsSourceFormat

RESOLVED FIXED in Firefox 55

Status

()

Core
Audio/Video: Playback
RESOLVED FIXED
11 months ago
11 months ago

People

(Reporter: jya, Assigned: jya)

Tracking

(Blocks: 1 bug)

unspecified
mozilla56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55- fixed, firefox56 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(2 attachments)

(Assignee)

Description

11 months ago
Reported by Matt in bug 1372874.

The transform use to convert NV12 to BGRA uses NV12 as output format, instead of BGRA.
(Assignee)

Comment 1

11 months ago
How come this wasn't causing problems with reftest or anything using canvas?
status-firefox55: --- → affected
tracking-firefox55: --- → ?
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 4

11 months ago
mozreview-review
Comment on attachment 8879835 [details]
Bug 1374936: P1. Properly output ARGB32 with transform.

https://reviewboard.mozilla.org/r/151232/#review156168
Attachment #8879835 - Flags: review?(matt.woodrow) → review+

Comment 5

11 months ago
mozreview-review
Comment on attachment 8879836 [details]
Bug 1374936: P2. Check for error.

https://reviewboard.mozilla.org/r/151234/#review156172
Attachment #8879836 - Flags: review?(matt.woodrow) → review+

Comment 6

11 months ago
Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/86510ad18b92
P1. Properly output ARGB32 with transform. r=mattwoodrow
https://hg.mozilla.org/integration/autoland/rev/3d2952de26c3
P2. Check for error. r=mattwoodrow

Comment 7

11 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/86510ad18b92
https://hg.mozilla.org/mozilla-central/rev/3d2952de26c3
Status: NEW → RESOLVED
Last Resolved: 11 months ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Jean-Yves, you requested tracking for 55, I assume you'll want to request uplift to beta?  Was this fix verified on nightly?
Flags: needinfo?(jyavenard)
(Assignee)

Comment 9

11 months ago
Comment on attachment 8879835 [details]
Bug 1374936: P1. Properly output ARGB32 with transform.

Approval Request Comment
[Feature/Bug causing the regression]: 1340398
[User impact if declined]: Bad content drawn on canvas.
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: Unsure
[Needs manual test from QE? If yes, steps to reproduce]: It needs to write some code to verify it.
[List of other uplifts needed for the feature/fix]: the two of this bug
[Is the change risky?]: No
[Why is the change risky/not risky?]: We set the proper data type on output
[String changes made/needed]: none
Flags: needinfo?(jyavenard)
Attachment #8879835 - Flags: approval-mozilla-beta?
So if we have no automated test and no manual test how can we be confident in this fix here?  "Unsure" isn't really a great answer...
Flags: needinfo?(jyavenard)
(Assignee)

Comment 11

11 months ago
because the bug in the code was so obvious, it jumped at you looking at the code.

Having said that, in bug 1372874 they may have the answer on how to test it.
Flags: needinfo?(jyavenard)
Comment on attachment 8879835 [details]
Bug 1374936: P1. Properly output ARGB32 with transform.

video format fix, beta55+
Attachment #8879835 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
tracking-firefox55: ? → -
You need to log in before you can comment on or make changes to this bug.