[gstreamer-bugs] [Bug 612154] New: rtph263depay removes all timestamps from outgoing buffers

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=612154
  GStreamer | gst-plugins-good | 0.10.19

           Summary: rtph263depay removes all timestamps from outgoing
                    buffers
    Classification: Desktop
           Product: GStreamer
           Version: 0.10.19
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: blocker
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: tester at tester.ca
         QAContact: gstreamer-bugs at lists.sourceforge.net
      GNOME target: ---
     GNOME version: ---


The commit 564581e1 "rtph263depay: baseclass handles timestamps for us" removes
the part that sets the timestamp. The baseclass actually does not set the
timestamp for us unless gst_base_rtp_depayload_push_ts() is called. Imho, this
patch should just be reverted (and 0.10.20 released). This will break video
calls between Empathy and the N8x0 tablets for one. Or is the right option to
do GST_BUF_TS(outbuf) = GST_BUF_TS(inbuf);

-- 
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.