[gstreamer-bugs] [Bug 640327] New: Add VP8 RTP payloaders and depayloaders

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=640327
  GStreamer | gst-plugins-bad | git

           Summary: Add VP8 RTP payloaders and depayloaders
    Classification: Desktop
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: sjoerd at luon.net
         QAContact: gstreamer-bugs at lists.sourceforge.net
      GNOME target: ---
     GNOME version: ---


Created an attachment (id=179082)
 --> (https://bugzilla.gnome.org/attachment.cgi?id=179082)
Proposed patch, also as git branch:
http://cgit.freedesktop.org/~sjoerd/gst-plugins-bad/log/?h=vp8

http://www.webmproject.org/code/specs/rtp/ has a DRAFT rtp spec for VP8. I've
implemented this in the attached patch. The implementation is quite mimimal and
has the following todo items:

* It does never set the N bit. To do this correctly it's necessary to parse
even more of the bitstream 
   header, which is a pain. Also what the conditions are to set the bit aren't
really properly defined. Not
   ever setting it should never be an issue (we're not telling the receiver
that the packet is a 
   non-reference frame that's safe to drop).
* We can't set the PictureID in the payloader (not mandatory), but do correctly
cope in the depayloader
   when it's set
* the SDP encoding name is VP8-DRAFT-0-3-2 instead of VP8 as i'm not looking
forward to versions in 
   the wild that are incompatible with whatever the final draft is :)
* The depayloader is minimal and isn't able to handle frames with incomplete
partitions (due to 
   packet loss). Otoh that's impossible to do correctly with the current spec
and i'm not sure how to
   feed it into the vpx decoder (it's not documented how this could be done)

None of these items are big enough to not merge the patches though (imho) :)

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.