Allow blacklisting of hardware accelerated video decoding on OSX

RESOLVED FIXED in Firefox 40

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: mattwoodrow, Assigned: jya)

Tracking

(Blocks: 1 bug)

29 Branch
mozilla40
x86
macOS
Points:
---

Firefox Tracking Flags

(firefox40 fixed)

Details

Attachments

(2 attachments)

Comment hidden (empty)
(Reporter)

Comment 1

4 years ago
The only functional change of this patch is that setting layers.acceleration.force-enabled will no longer override the DXVA blacklist. I think that's probably for the best.

The use-dxva pref should be hidden now, but users that have already set it manually should continue to not get dxva.
Attachment #8585800 - Flags: review?(cpearce)
(Reporter)

Comment 2

4 years ago
jya: Can you please do a followup patch that checks the value of gfxPlatform::CanUseHardwareVideoDecoding() within the mac decoders?
Flags: needinfo?(jyavenard)
Attachment #8585800 - Flags: review?(cpearce) → review+
(Assignee)

Comment 3

4 years ago
mac part
Attachment #8585856 - Flags: review?(matt.woodrow)
(Assignee)

Updated

4 years ago
Assignee: matt.woodrow → jyavenard
Status: NEW → ASSIGNED
(Assignee)

Updated

4 years ago
Flags: needinfo?(jyavenard)
(Reporter)

Updated

4 years ago
Attachment #8585856 - Flags: review?(matt.woodrow) → review+
https://hg.mozilla.org/mozilla-central/rev/f7bbc6c8f007
https://hg.mozilla.org/mozilla-central/rev/7779bff82d84
https://hg.mozilla.org/mozilla-central/rev/851707bd8630
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.