[gstreamer-bugs] [Bug 629839] New: [qtdemux] Update xmp tags parsing

[GStreamer (bugzilla.gnome.org)]

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

           Summary: [qtdemux] Update xmp tags parsing
    Classification: Desktop
           Product: GStreamer
           Version: git
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: thiago.sousa.santos at collabora.co.uk
         QAContact: gstreamer-bugs at lists.sourceforge.net
      GNOME target: ---
     GNOME version: ---


New adobe xmp specs have different placements for qt/mp4 variants. We are only
handling the qt one.

These 2 patches make it work for the mp4 one, too.

The first patch is for pull mode and makes qtdemux parse all top-level atoms
before start pushing the streams' data. This is because for mp4 the xmp is
placed on a top-level uuid atom.

The second one delays the tags posting to immediately before starting to push
the streams' data. This is needed for merging xmp and native tags before
posting them.

Not sure how we can handle nicely this for push mode, maybe use best-effort and
only parse it if we find it before we get a moov+mdat.

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