[Webkit-unassigned] [Bug 174817] New: [GStreamer] Refactor media player to use MediaTime consistently

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jul 25 06:43:20 PDT 2017


https://bugs.webkit.org/show_bug.cgi?id=174817

            Bug ID: 174817
           Summary: [GStreamer] Refactor media player to use MediaTime
                    consistently
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit Gtk
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: cturner at igalia.com
                CC: bugs-noreply at webkitgtk.org

WebCore moved to using a rational class for working with media positions called MediaTime a while back, I suspect because Apple's media backend uses a rational class internally. GStreamer just uses a 64 bit integer for timestamps so we don't benefit from much of the information/safety MediaTime supports. However, the existing code was a bit weird in that there were many places of curiosity that were converting between floats and doubles into MediaTimes just to support the new APIs, which was making parts of the code quite hard to read.

This patch makes more consistent use of MediaTime and cleans up these areas.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170725/43715bea/attachment-0001.html>


More information about the webkit-unassigned mailing list