Last Comment Bug 772341 - Enable Opus support by default
: Enable Opus support by default
: relnote
Product: Core
Classification: Components
Component: Audio/Video (show other bugs)
: 15 Branch
: All All
-- normal (vote)
: mozilla16
Assigned To: Timothy B. Terriberry (:derf)
: Maire Reavy [:mreavy] Please needinfo me
Depends on: 1129355 935617 976488
Blocks: 674225
  Show dependency treegraph
Reported: 2012-07-09 21:40 PDT by Timothy B. Terriberry (:derf)
Modified: 2016-11-30 05:06 PST (History)
11 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Enable Opus support by default (774 bytes, patch)
2012-07-09 21:42 PDT, Timothy B. Terriberry (:derf)
roc: review+
akeybl: approval‑mozilla‑aurora+
Details | Diff | Splinter Review

Description User image Timothy B. Terriberry (:derf) 2012-07-09 21:40:10 PDT
The support for the Opus codec added in bug 674225 was hidden behind a preference in case there were any last-minute changes to the format during the final phases of standardization at the IETF. It has now been approved by the IESG, so this is no longer a danger. We should flip the preference to enable support by default.
Comment 1 User image Timothy B. Terriberry (:derf) 2012-07-09 21:42:10 PDT
Created attachment 640501 [details] [diff] [review]
Enable Opus support by default
Comment 2 User image j.j. 2012-07-09 22:19:30 PDT
Is this intended to land on Fx 15? Otherwise might need documentation changes.
Comment 3 User image Timothy B. Terriberry (:derf) 2012-07-09 22:21:58 PDT
Yes, after getting review I was planning to ask for aurora approval.
Comment 4 User image Timothy B. Terriberry (:derf) 2012-07-10 06:16:28 PDT
Comment on attachment 640501 [details] [diff] [review]
Enable Opus support by default

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Bug 674225
User impact if declined: Opus support will remain behind a pref and thus be unavailable to most users (hindering adoption by websites). Flipping this pref now means we will be shipping it to users right about the same time the final RFC comes out, so we should be able to build on momentum around that event. Holding off until the next release means missing out on that.
Testing completed (on m-c, etc.): The codec library itself has undergone extensive testing (see for just a brief summary). The version we imported and the code we wrote to hook it into Firefox has been fuzz-tested (bug 750714), and has been shipping in Nightlies and on Aurora (behind this pref) since May 4th. Bug 674225 added a set of media tests for Opus, and a try run with this pref flipped (enabling them) is relatively greenish:
Risk to taking this patch (and alternatives if risky): Low. No code changes. This should not impact any existing features.
String or UUID changes made by this patch: None.
Comment 5 User image Timothy B. Terriberry (:derf) 2012-07-10 10:53:19 PDT
Comment 6 User image Ryan VanderMeulen [:RyanVM] 2012-07-10 20:33:09 PDT
Comment 7 User image Alex Keybl [:akeybl] 2012-07-12 15:37:01 PDT
Asa - any non-engineering reasons to hold off on uplift of having Opus enabled by default? I can't think of any, but want to be sure.
Comment 8 User image Alex Keybl [:akeybl] 2012-07-13 13:47:47 PDT
Comment on attachment 640501 [details] [diff] [review]
Enable Opus support by default

[Triage Comment]
Spoke with Asa. This has the green light. Please land on mozilla-aurora ASAP to make the next merge on Monday.
Comment 9 User image Timothy B. Terriberry (:derf) 2012-07-13 14:39:53 PDT
Comment 10 User image j.j. 2012-07-13 15:03:09 PDT
removing dev-doc-needed, no changes required after landing on aurora
Comment 11 User image Ralph Giles (:rillian) | needinfo me 2012-07-13 17:46:45 PDT
Restoring dev-doc-needed. Opus is mentioned on the codec support page and in the release notes. However, the line,

", although support is only available if you set the media.opus.enabled preference to true"

should be removed from now that this bug has landed. Wiki edits are currently disabled for the kuma migration, so we need a reminder to do this later.
Comment 12 User image j.j. 2012-07-13 20:31:34 PDT
oops, looked only at the browser compatibility table.

Also: Heading structure should change to:
== Ogg
=== Ogg Ogg Theora Vorbis
=== Ogg Opus

could link into #Ogg_Opus
Comment 14 User image Ralph Giles (:rillian) | needinfo me 2012-08-03 11:59:09 PDT
I've removed the user pref reference now that the wiki is open for editing again, but left the section headers as is. Either way, linking to

will work fine.

Note You need to log in before you can comment on or make changes to this bug.