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

Unified Diff: media/base/audio_decoder_config.cc

Issue 239423005: Wire up codec_delay() to MP3StreamParser and FFmpegAudioDecoder. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ResetTimestampState() on config change. 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/base/audio_decoder_config.h ('k') | media/ffmpeg/ffmpeg_common.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/audio_decoder_config.cc
diff --git a/media/base/audio_decoder_config.cc b/media/base/audio_decoder_config.cc
index 09910edbe40479f6af5daa7559967d8a9941a9c7..06a1643f0a1ae4c2cf1b869dbf87fd9a02c42024 100644
--- a/media/base/audio_decoder_config.cc
+++ b/media/base/audio_decoder_config.cc
@@ -20,7 +20,8 @@ AudioDecoderConfig::AudioDecoderConfig()
channel_layout_(CHANNEL_LAYOUT_UNSUPPORTED),
samples_per_second_(0),
bytes_per_frame_(0),
- is_encrypted_(false) {
+ is_encrypted_(false),
+ codec_delay_(0) {
}
AudioDecoderConfig::AudioDecoderConfig(AudioCodec codec,
@@ -32,7 +33,7 @@ AudioDecoderConfig::AudioDecoderConfig(AudioCodec codec,
bool is_encrypted) {
Initialize(codec, sample_format, channel_layout, samples_per_second,
extra_data, extra_data_size, is_encrypted, true,
- base::TimeDelta(), base::TimeDelta());
+ base::TimeDelta(), 0);
}
void AudioDecoderConfig::Initialize(AudioCodec codec,
@@ -44,7 +45,7 @@ void AudioDecoderConfig::Initialize(AudioCodec codec,
bool is_encrypted,
bool record_stats,
base::TimeDelta seek_preroll,
- base::TimeDelta codec_delay) {
+ int codec_delay) {
CHECK((extra_data_size != 0) == (extra_data != NULL));
if (record_stats) {
@@ -88,7 +89,7 @@ bool AudioDecoderConfig::IsValidConfig() const {
samples_per_second_ <= limits::kMaxSampleRate &&
sample_format_ != kUnknownSampleFormat &&
seek_preroll_ >= base::TimeDelta() &&
- codec_delay_ >= base::TimeDelta();
+ codec_delay_ >= 0;
}
bool AudioDecoderConfig::Matches(const AudioDecoderConfig& config) const {
@@ -114,7 +115,7 @@ std::string AudioDecoderConfig::AsHumanReadableString() const {
<< " sample_format: " << sample_format()
<< " bytes_per_frame: " << bytes_per_frame()
<< " seek_preroll: " << seek_preroll().InMilliseconds() << "ms"
- << " codec_delay: " << codec_delay().InMilliseconds() << "ms"
+ << " codec_delay: " << codec_delay()
<< " has extra data? " << (extra_data() ? "true" : "false")
<< " encrypted? " << (is_encrypted() ? "true" : "false");
return s.str();
« no previous file with comments | « media/base/audio_decoder_config.h ('k') | media/ffmpeg/ffmpeg_common.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698