Closed
Bug 1200775
Opened 10 years ago
Closed 10 years ago
Check intel specific h264 decoder when checking for DXVA support
Categories
(Core :: Audio/Video: Playback, defect, P1)
Tracking
()
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: mattwoodrow, Assigned: mattwoodrow)
References
Details
Attachments
(1 file)
9.00 KB,
patch
|
cpearce
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
Seems like some of the older intel cards only support this one.
Attachment #8655595 -
Flags: review?(jyavenard)
Updated•10 years ago
|
Priority: -- → P1
Comment 2•10 years ago
|
||
Comment on attachment 8655595 [details] [diff] [review]
intel-decoder
Review of attachment 8655595 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM, but I have no idea about why this is required nor how it would actually work.
"Seems like some of the older intel cards only support this one." what is "this one" ?
302 to cpearce
Attachment #8655595 -
Flags: review?(jyavenard) → review?(cpearce)
Assignee | ||
Comment 3•10 years ago
|
||
It looks like older intel cards have a different GUID for their h264 decoder, but it otherwise functions identically. The MFTransform we are using to access the decoder will pick this one if the 'normal' one isn't available.
Our code for determining DXVA support was only checking the normal GUID, so we think DXVA isn't available when it would otherwise work through the MFTransform. This patch fixes that.
Updated•10 years ago
|
Attachment #8655595 -
Flags: review?(cpearce) → review+
Comment 6•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/aa8cbb652d86
https://hg.mozilla.org/mozilla-central/rev/6f234301d1b8
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in
before you can comment on or make changes to this bug.
Description
•