[Bug 739990] basesink: set the last position to minimum value after rewind

[GStreamer (GNOME Bugzilla)]

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

Myoungsun Lee <ohmygod0327 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ohmygod0327 at gmail.com

--- Comment #7 from Myoungsun Lee <ohmygod0327 at gmail.com> ---
Created attachment 304123
  --> https://bugzilla.gnome.org/attachment.cgi?id=304123&action=edit
compare basesink timestamp log

I attached the log file about basesink:6 level.

[Summary]
I pushed REWIND command and wait until the video reached start point(0 sec).
Then I can see the progress bar pop-up to any time position, not 0 sec.
Because the last time is set "stop" value in gst_base_sink_get_position(), the
progress bar is abnormal.

-before
basesink gstbasesink.c:4627:default_element_query:<videosink> position query in
format time
basesink gstbasesink.c:4416:gst_base_sink_get_position:<videosink> using clock
and base time 0:37:31.851788888
basesink gstbasesink.c:4443:gst_base_sink_get_position:<videosink> in PAUSED
using last 0:00:05.000000000
basesink gstbasesink.c:4477:gst_base_sink_get_position:<videosink> using last
seen timestamp 0:00:05.000000000
basesink gstbasesink.c:4549:gst_base_sink_get_position:<videosink> res: 1,
POSITION: 0:00:05.000000000
basesink gstbasesink.c:4746:default_element_query:<videosink> query position
returns 1
======> The progress bar is set by the last time value. It is set "5" sec, so
the progress bar is pop-up this position. It need to start from "0" sec.



-after patch
basesink gstbasesink.c:4627:default_element_query:<videosink> position query in
format time
basesink gstbasesink.c:4416:gst_base_sink_get_position:<videosink> using clock
and base time 0:46:34.971955555
basesink gstbasesink.c:4443:gst_base_sink_get_position:<videosink> in PAUSED
using last 0:00:00.000000000
basesink gstbasesink.c:4477:gst_base_sink_get_position:<videosink> using last
seen timestamp 0:00:00.000000000
basesink gstbasesink.c:4549:gst_base_sink_get_position:<videosink> res: 1,
POSITION: 0:00:00.000000000
basesink gstbasesink.c:4746:default_element_query:<videosink> query position
returns 1
===> Then the last position is set the smallest time value. In REWIND case, the
last time has to mean the smallest one that the basesink remembered.(In FORWARD
case, it means the largest one.)

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