Re-enable using Adobe GMP for decoding when system H.264/AAC codecs not available

VERIFIED FIXED in Firefox 46

Status

()

defect
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: cpearce, Assigned: cpearce)

Tracking

({feature})

unspecified
mozilla47
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox44 wontfix, firefox45 ?, firefox46+ verified, firefox47+ verified, relnote-firefox 46+)

Details

Attachments

(1 attachment)

Now that Bug 1236756 has been fixed by Adobe, we can re-enable using the Adobe GMP for decoding when system H.264/AAC codecs are unavailable.

This means Windows XP and Windows Vista and N/KN Windows variants without the media feature packs installed will be able to decode and play MP4/H.264/AAC video.
Release Note Request (optional, but appreciated)
[Why is this notable]: Enabling Adobe's H.264/AAC decoder means Windows XP and Windows Vista and N/KN Windows variants without the media feature packs installed will be able to decode and play MP4/H.264/AAC video. This change increases the percentage of Windows Firefox users that can play H.264 from about 85% to almost 100%.
[Suggested wording]: "Use Adobe's H.264 and AAC decoders on Windows machines without Microsoft's decoders, such as Windows XP and Vista and N/KN editions without the media feature packs installed."
[Links (documentation, blog post, etc)]:
Comment on attachment 8722801 [details]
MozReview Request: Bug 1250766 - Re-enable using Adobe GMP for decoding when system H.264/AAC codecs not available. r?jya

https://reviewboard.mozilla.org/r/36247/#review32837
Attachment #8722801 - Flags: review?(jyavenard) → review+

Comment 5

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/5043859cda39
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Comment on attachment 8722801 [details]
MozReview Request: Bug 1250766 - Re-enable using Adobe GMP for decoding when system H.264/AAC codecs not available. r?jya

Requesting uplift to Firefox 46.

Approval Request Comment
[Feature/regressing bug #]: MP4/H.264/AAC decoding on Windows XP.

[User impact if declined]: Firefox users on Windows XP won't get MP4/H.264/AAC playback. This patch enables playback of MP4 videos for Windows XP users using the Adobe Gecko Media Plugin. We had this in 45, but had to turn it off due to bugs in the Adobe GMP. Those have been fixed, so we should get this uplifted so that WinXP users can finally get MP4 playback.

[Describe test coverage new/current, TreeHerder]: We have mochitests for this feature.
[Risks and why]: Low; we've been running Adobe's decoder for EME content for a while, so we're unlikely to find errors.
[String/UUID change made/needed]: None.
Attachment #8722801 - Flags: approval-mozilla-aurora?
OK, let's see how this does on aurora. Wasn't there an e10s issue? Or am I thinking of something else? 
Tracking since this is enabling a new feature. 

Want this to go into the release notes for beta 46?
Comment on attachment 8722801 [details]
MozReview Request: Bug 1250766 - Re-enable using Adobe GMP for decoding when system H.264/AAC codecs not available. r?jya

OK for aurora, has test coverage, and the blocking bugs have been fixed in 46.
Attachment #8722801 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Flags: qe-verify+
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #7)
> OK, let's see how this does on aurora. Wasn't there an e10s issue? Or am I
> thinking of something else?

Possibly you're thinking of Bug 1250745?

> Tracking since this is enabling a new feature. 
> 
> Want this to go into the release notes for beta 46?

Yes, this should be in the release notes.
Adding the relnote for 46 as suggested in comment 2. The wording is a bit long. We can work that out in the draft which I'll send round soon.
I tried some h264/AAC samples using Firefox 46 beta 8 and latest Developer Edition 47.0a2 and videos played as expected, also h.264 and MSE & H.264 are 'checked' in youtube/html5. Testing was done on Windows XP 64-bit and Windows Vista 32-bit.
Status: RESOLVED → VERIFIED
Flags: qe-verify+

Updated

3 years ago
See Also: → 1234100
Depends on: 1265815
You need to log in before you can comment on or make changes to this bug.