+++ This bug was initially created as a clone of Bug #1090742 +++ We need VP9 packetization code to interop with Chrome's impl of the draft VP9 packetization spec. Support for independent layers/SVC is not required at this time, though if we get it, great. Likely we can cherry-pick the code from the upstream webrtc.org VP9 support, ahead of importing stable branch 48 of webrtc.org.

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

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

Note that I decided to defined RTC_DCHECK/etc part way through, so some of the code I imported/merged has been modified, and some hasn't. I could go either way on this, probably it's simpler if the imported code keeps RTC_DCHECK

Comment on attachment 8711493 [details] MozReview Request: Bug 1237023: update of cherry-pick of VP9 - works with some chroma issues r?pkerr Review request updated; see interdiff: https://reviewboard.mozilla.org/r/32209/diff/1-2/

Comment on attachment 8711492 [details] MozReview Request: Bug 1237023: cherry-pick of VP9 packetization/jitter-buffer/encoder code from Webrtc.org 48 r?pkerr https://reviewboard.mozilla.org/r/32207/#review28879 ::: media/webrtc/trunk/webrtc/modules/interface/module_common_types.h:208 (Diff revision 1) > +#if 0 Why the #if 0? Is there a more informative ifdef label that could be used? Is this documentation?

Comment on attachment 8711493 [details] MozReview Request: Bug 1237023: update of cherry-pick of VP9 - works with some chroma issues r?pkerr https://reviewboard.mozilla.org/r/32209/#review28881

> ::: media/webrtc/trunk/webrtc/modules/interface/module_common_types.h:208 > (Diff revision 1) > > +#if 0 > > Why the #if 0? Is there a more informative ifdef label that could be used? > Is this documentation? Switched to more informative #if ... #else

https://hg.mozilla.org/mozilla-central/rev/f442638a278c https://hg.mozilla.org/mozilla-central/rev/4bd447a3d7da

Forgot build config review.

(In reply to Karl Tomlinson (ni?:karlt) from comment #11) > Forgot build config review. Ouch, sorry about that. Mostly this is just merging code from upstream but that bit wasn't (it was making it build with our libvpx in the non-system-libvpx case)