1530977 - Video hardware decoder no longer work with e10s off

Status: RESOLVED FIXED

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

Description

[Jean-Yves Avenard [:jya]]

With e10s off, HW decoding no longer works.

This is due to https://searchfox.org/mozilla-central/source/gfx/thebes/gfxWindowsPlatform.cpp#454
bool gfxWindowsPlatform::CanUseHardwareVideoDecoding() always return false as dm->TextureSharingWorks() always return false, and it returns false because DeviceManagerDx::TextureSharingWorks has DeviceManagerDx::mDeviceStatus not set at the time the test is done.

This was regressed in bug 1513308.

The ability to work with the hardware decoders functional while e10s is off is essential to my workflow as developing and debugging the various HW decoder over various process and IPC communication is much more complex.

Comment 1

[Jean-Yves Avenard [:jya]]

Attachment: Bug 1530977 - Initialize D3D devices immediately when e10s is not active. r?mattwoodrow

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f9e25c03e77a
Initialize D3D devices immediately when e10s is not active. r=mattwoodrow

Comment 3

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/f9e25c03e77a

Comment 4

[Ryan VanderMeulen [:RyanVM]]

I'm guessing this is something we want to backport to Beta to avoid shipping the regression? Please nominate if so :)

Comment 5

[Nils Ohlmeier [:drno]]

If I recall correctly from talking for jya 66 is not affected by this, because on 66 two other bugs are canceling each other out so that we don't have the same problem.

@jya is this right, or do we need to nominate this for uplift?

Comment 6

[Jean-Yves Avenard [:jya]]

If I recall correctly from talking for jya 66 is not affected by this, because on 66 two other bugs are canceling each other out so that we don't have the same problem.

@jya is this right, or do we need to nominate this for uplift?

this was in reference to another bug that got slightly better.

However HW decoding would still be disabled on 66 as a consequence.

It's not that urgent to uplift to 66 as I believe everyone should be with a GPU decoder enabled and e10s on these days.