Closed Bug 1530977 Opened 1 year ago Closed 1 year ago

Video hardware decoder no longer work with e10s off

Categories

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

x86
Windows
defect

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox-esr60 --- unaffected
firefox65 --- unaffected
firefox66 + wontfix
firefox67 --- fixed

People

(Reporter: jya, Assigned: jya)

References

(Blocks 1 open bug)

Details

(Keywords: regression)

Attachments

(1 file)

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.

Rank: 15
Priority: -- → P2
Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f9e25c03e77a
Initialize D3D devices immediately when e10s is not active. r=mattwoodrow
Blocks: 1531207
Assignee: nobody → jyavenard
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67

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

Flags: needinfo?(jyavenard)

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?

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.

Flags: needinfo?(jyavenard)
QA Whiteboard: [qa-67b-p2]
You need to log in before you can comment on or make changes to this bug.