Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(67)

Unified Diff: media/filters/ffmpeg_demuxer.h

Issue 334163002: Revert of Fix seeking when the start time is non-zero. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/filters/ffmpeg_audio_decoder.cc ('k') | media/filters/ffmpeg_demuxer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/ffmpeg_demuxer.h
diff --git a/media/filters/ffmpeg_demuxer.h b/media/filters/ffmpeg_demuxer.h
index 373ca32f18c5005a78c6e4075903eb56402be4c7..02682bbc8b2c38c4b3a07771a7c86558c595bfb9 100644
--- a/media/filters/ffmpeg_demuxer.h
+++ b/media/filters/ffmpeg_demuxer.h
@@ -55,14 +55,9 @@
class FFmpegDemuxerStream : public DemuxerStream {
public:
- // Keeps a copy of |demuxer| and initializes itself using information inside
- // |stream|. Both parameters must outlive |this|.
- // |discard_negative_timestamps| tells the DemuxerStream that all packets with
- // negative timestamps should be marked for post-decode discard. All decoded
- // data before time zero will be discarded.
- FFmpegDemuxerStream(FFmpegDemuxer* demuxer,
- AVStream* stream,
- bool discard_negative_timestamps);
+ // Keeps a copy of |demuxer| and initializes itself using information
+ // inside |stream|. Both parameters must outlive |this|.
+ FFmpegDemuxerStream(FFmpegDemuxer* demuxer, AVStream* stream);
virtual ~FFmpegDemuxerStream();
// Enqueues the given AVPacket. It is invalid to queue a |packet| after
@@ -79,7 +74,8 @@
// Empties the queues and ignores any additional calls to Read().
void Stop();
- base::TimeDelta duration() const { return duration_; }
+ // Returns the duration of this stream.
+ base::TimeDelta duration();
// DemuxerStream implementation.
virtual Type type() OVERRIDE;
@@ -140,7 +136,6 @@
bool bitstream_converter_enabled_;
std::string encryption_key_id_;
- const bool discard_negative_timestamps_;
DISALLOW_COPY_AND_ASSIGN(FFmpegDemuxerStream);
};
@@ -160,6 +155,7 @@
virtual void Stop(const base::Closure& callback) OVERRIDE;
virtual void Seek(base::TimeDelta time, const PipelineStatusCB& cb) OVERRIDE;
virtual DemuxerStream* GetStream(DemuxerStream::Type type) OVERRIDE;
+ virtual base::TimeDelta GetStartTime() const OVERRIDE;
virtual base::Time GetTimelineOffset() const OVERRIDE;
virtual Liveness GetLiveness() const OVERRIDE;
@@ -171,10 +167,6 @@
// about capacity and what buffered data is available.
void NotifyCapacityAvailable();
void NotifyBufferingChanged();
-
- // The lowest demuxed timestamp. DemuxerStream's must use this to adjust
- // packet timestamps such that external clients see a zero-based timeline.
- base::TimeDelta start_time() const { return start_time_; }
private:
// To allow tests access to privates.
@@ -253,9 +245,9 @@
// Derived bitrate after initialization has completed.
int bitrate_;
- // The first timestamp of the audio or video stream, whichever is lower. This
- // is used to adjust timestamps so that external consumers always see a zero
- // based timeline.
+ // The first timestamp of the opened media file. This is used to set the
+ // starting clock value to match the timestamps in the media file. Default
+ // is 0.
base::TimeDelta start_time_;
// The Time associated with timestamp 0. Set to a null
@@ -278,10 +270,6 @@
const NeedKeyCB need_key_cb_;
- // The index of the stream in |streams_| to use for seeking. Chosen by the
- // stream with the lowest starting timestamp.
- int stream_index_for_seeking_;
-
// NOTE: Weak pointers must be invalidated before all other member variables.
base::WeakPtrFactory<FFmpegDemuxer> weak_factory_;
« no previous file with comments | « media/filters/ffmpeg_audio_decoder.cc ('k') | media/filters/ffmpeg_demuxer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698