MSE needs to expose the layer manager backend type to MP4Reader

RESOLVED FIXED in mozilla31

Status

()

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: cpearce, Assigned: kinetik)

Tracking

(Blocks: 1 bug)

unspecified
mozilla31
x86_64
Windows Vista
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

The BufferDecoder that owns the MP4Reader doesn't return a MediaDecoderOwner in its GetOwner() implementation, so MP4Reader::InitLayersBackendType() fails to determine the backend type, so it can't initialize DXVA.

The same thing happens when WMFReader is used in MSE.

This means we don't get DXVA decoding, which makes the perf fairy is sad.

We need a way to expose to the decoder whether it's safe to use hardware accelerated decoding or not.

Testcase: http://people.mozilla.org/~ydelendik/mse/demo-mse4.html
(Assignee)

Comment 2

5 years ago
I haven't actually tested this, but it's a simple change.  There's probably other stuff on the SubBufferDecoders that need to be forwarded too; they'll be fixed as necessary.
(Reporter)

Updated

5 years ago
Attachment #8413463 - Flags: review?(cpearce) → review+
https://hg.mozilla.org/mozilla-central/rev/dcbcab7596ae
Assignee: nobody → kinetik
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Target Milestone: mozilla32 → mozilla31
You need to log in before you can comment on or make changes to this bug.