Bug 1230265 added a large, non-const static array representing a small webm file. This data isn't shareable between processes, and is responsible for a noticeable jump in Fennec's libxul size. Making it const is trivial; do we even need to enable this code for Fennec?

Anthony: when we uplift bug 1230265 to Aurora 47, we will want this non-const data fix.

We can probably remove this in Fennec as many android phone now have hardware decoder. I will open a bug to ensure that VP9 is always enabled on platforms with vp9 hardware decoding.

Review commit: https://reviewboard.mozilla.org/r/43679/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/43679/

Comment on attachment 8737042 [details] MozReview Request: Bug 1261129: Make VP9 sample data const. r?kentuckyfriedtakahe https://reviewboard.mozilla.org/r/43679/#review41217

Comment on attachment 8737042 [details] MozReview Request: Bug 1261129: Make VP9 sample data const. r?kentuckyfriedtakahe Approval Request Comment [Feature/regressing bug #]: 1261129 [User impact if declined]: blocking bug 1263839 uplift [Describe test coverage new/current, TreeHerder]: in central for a while, and there's nothing to test really [Risks and why]: none, just making the code const [String/UUID change made/needed]: none

Comment on attachment 8737042 [details] MozReview Request: Bug 1261129: Make VP9 sample data const. r?kentuckyfriedtakahe Recent issue due to VP9 benchmarking fix uplifted a few weeks back, this has baked in Nightly for a bit, Aurora47+