Seeking should be unified in that way we are always sure negative playback rate is always taken into account.
Already working on solution.
Created attachment 219411 [details] Patch
Comment on attachment 219411 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=219411&action=review > Source/WebCore/ChangeLog:11 > + Test: media/video-seek-with-negative-playback.html This is skipped in EFL but not in GTK, it seems. I guess it can now be unskipped? > Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:614 > +bool MediaPlayerPrivateGStreamer::seekIncludingRate(gint64 position, float rate, GstSeekFlags seekType) Hum what about doSeek() ? Having only the rate mentioned in the name is a bit odd.
Created attachment 219904 [details] patch
(In reply to comment #3) > (From update of attachment 219411 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=219411&action=review > > > Source/WebCore/ChangeLog:11 > > + Test: media/video-seek-with-negative-playback.html > > This is skipped in EFL but not in GTK, it seems. I guess it can now be unskipped? > > > Source/WebCore/platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:614 > > +bool MediaPlayerPrivateGStreamer::seekIncludingRate(gint64 position, float rate, GstSeekFlags seekType) > > Hum what about doSeek() ? Having only the rate mentioned in the name is a bit odd. Unfortunatelly this test is still flaky for EFL becouse of some race condition.
Comment on attachment 219904 [details] patch Clearing flags on attachment: 219904 Committed r160992: <http://trac.webkit.org/changeset/160992>
All reviewed patches have been landed. Closing bug.