| commit | f1e59edef784dc1516e4699d2be2125ce0159904 | |
| author | tkent@chromium.org <tkent@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Mon, 16 Jun 2014 06:07:44 +0000 (16 06:07 +0000) | ||
| committer | tkent@chromium.org <tkent@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | |
| Mon, 16 Jun 2014 06:07:44 +0000 (16 06:07 +0000) | ||
| tree | 47a57bda6f927c332ee9e0da500b90a5125f16da | treesnapshot (tar.gz zip) |
| parent | 0e6c1615e51ba91534549f65a3c5663600b792cc | commitdiff |
Revert of Fix seeking when the start time is non-zero. (https://codereview.chromium.org/325503003/)
Reason for revert:
Broke the following layout tests:
- media/track/track-cue-rendering-horizontal.html
- media/track/track-cue-rendering-vertical.html
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20Blink&tests=media/track/track-cue-rendering-horizontal.html,media/track/track-cue-rendering-vertical.html
Original issue's description:
> Fix seeking when the start time is non-zero.
>
> The seek timestamp should always be adjusted by the start time,
> though this manipulation should not be visible to external (web)
> clients.
>
> Since this is an FFmpeg only problem, I've removed the concept
> of start time from the Demuxer interface in favor of local method
> only for FFmpegDemuxer. FFmpegDemuxerStream's will now use this
> value to adjust timestamps such that external clients always see
> a zero based timeline.
>
> Doing so required moving some of our ogg vorbis discard code into
> the FFmpegDemuxerStream, which actually makes it more accurate and
> more narrowly scoped to ogg w/ vorbis instead of all vorbis.
>
> These changes subtly change how we handle seeking. Previously we
> would let FFmpeg choose the stream to perform seeking within. Now
> we always use the stream with the lowest order timestamp. This
> means we will sometime use the audio stream where previously we
> used the video stream.
>
> I've extended the tests around non-zero start times to verify the
> new behavior.
>
> An FFmpeg DEPS roll is required for the new tests to pass:
> 9c12290 Revert "avformat/mp3dec: fix start time in light of initial skip samples"
> 1e661a6 avcodec/vorbisdec: Reset first_frame
> 7d88be4 avformat/oggparsevorbis: Dont attempt to calculate timestamps from gp=0
>
> BUG=377295
> TEST=new unittests. demo page from bug works.
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=277348
TBR=acolwell@chromium.org,dalecurtis@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=377295
Review URL: https://codereview.chromium.org/334163002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277361 0039d316-1c4b-4281-b951-d872f2087c98
Reason for revert:
Broke the following layout tests:
- media/track/track-cue-rendering-horizontal.html
- media/track/track-cue-rendering-vertical.html
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20Blink&tests=media/track/track-cue-rendering-horizontal.html,media/track/track-cue-rendering-vertical.html
Original issue's description:
> Fix seeking when the start time is non-zero.
>
> The seek timestamp should always be adjusted by the start time,
> though this manipulation should not be visible to external (web)
> clients.
>
> Since this is an FFmpeg only problem, I've removed the concept
> of start time from the Demuxer interface in favor of local method
> only for FFmpegDemuxer. FFmpegDemuxerStream's will now use this
> value to adjust timestamps such that external clients always see
> a zero based timeline.
>
> Doing so required moving some of our ogg vorbis discard code into
> the FFmpegDemuxerStream, which actually makes it more accurate and
> more narrowly scoped to ogg w/ vorbis instead of all vorbis.
>
> These changes subtly change how we handle seeking. Previously we
> would let FFmpeg choose the stream to perform seeking within. Now
> we always use the stream with the lowest order timestamp. This
> means we will sometime use the audio stream where previously we
> used the video stream.
>
> I've extended the tests around non-zero start times to verify the
> new behavior.
>
> An FFmpeg DEPS roll is required for the new tests to pass:
> 9c12290 Revert "avformat/mp3dec: fix start time in light of initial skip samples"
> 1e661a6 avcodec/vorbisdec: Reset first_frame
> 7d88be4 avformat/oggparsevorbis: Dont attempt to calculate timestamps from gp=0
>
> BUG=377295
> TEST=new unittests. demo page from bug works.
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=277348
TBR=acolwell@chromium.org,dalecurtis@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=377295
Review URL: https://codereview.chromium.org/334163002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277361 0039d316-1c4b-4281-b951-d872f2087c98