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

Unified Diff: media/filters/opus_audio_decoder.h

Issue 259453003: Introduce AudioDiscardHelper. Refactor audio decoders to use it. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix x64 type. Created 6 years, 8 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/opus_audio_decoder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/filters/opus_audio_decoder.h
diff --git a/media/filters/opus_audio_decoder.h b/media/filters/opus_audio_decoder.h
index d0dfcf07b3ebdcabae02135d36b5ffcda9ef1e2b..f7a6161e6e03dac764efd439eeda7ba4af40da92 100644
--- a/media/filters/opus_audio_decoder.h
+++ b/media/filters/opus_audio_decoder.h
@@ -20,7 +20,7 @@ class SingleThreadTaskRunner;
namespace media {
class AudioBuffer;
-class AudioTimestampHelper;
+class AudioDiscardHelper;
class DecoderBuffer;
struct QueuedAudioBuffer;
@@ -55,21 +55,12 @@ class MEDIA_EXPORT OpusAudioDecoder : public AudioDecoder {
AudioDecoderConfig config_;
OpusMSDecoder* opus_decoder_;
- // Used for computing output timestamps.
- scoped_ptr<AudioTimestampHelper> output_timestamp_helper_;
- base::TimeDelta last_input_timestamp_;
-
- // Number of frames to be discarded from the start of the packet. This value
- // is respected for all packets except for the first one in the stream. For
- // the first packet in the stream, |frame_delay_at_start_| is used. This is
- // usually set to the SeekPreRoll value from the container whenever a seek
- // happens.
- int frames_to_discard_;
-
// When the input timestamp is |start_input_timestamp_| the decoder needs to
// drop |config_.codec_delay()| frames.
base::TimeDelta start_input_timestamp_;
+ scoped_ptr<AudioDiscardHelper> discard_helper_;
+
DISALLOW_IMPLICIT_CONSTRUCTORS(OpusAudioDecoder);
};
« no previous file with comments | « media/filters/ffmpeg_audio_decoder.cc ('k') | media/filters/opus_audio_decoder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698