[Bug 764370] New: codecparser: vp9: Wrong default values in non-intraonly/non-keyframe header fields

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=764370

            Bug ID: 764370
           Summary: codecparser: vp9: Wrong default values in
                    non-intraonly/non-keyframe header fields
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: Linux
            Status: NEW
          Severity: major
          Priority: Normal
         Component: gst-plugins-bad
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: bsreerenj at gmail.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

The bit_depth, color_space, color_range, subsampling_x and subsampling_y fields
in frame_headers will be present only when the frames are either keyframe or
intra-only frame.

The current code just assigning default values for these fields when the frame
is nonkey/non-intraonly which is wrong. For eg: this will assign bitdepth=8
even fro 10-bit frames.

Either we have to move these fields from FrameHdr structure to global
GstVp9Parser structure. Which is no extra copy but with ABI change.

OR

We should keep the copies of all these fields in Private structure
(GstVp9ParserPrivate), and copy them
back to FrameHdr for each non-keyframe/non-intraonly frame.

Not sure which one is preferred :)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.