media: Fix Opus support, and handle bad timestamps correctly in the Opus wrapper.

media/ffmpeg/ffmpeg_common.cc

Index: media/ffmpeg/ffmpeg_common.cc
diff --git a/media/ffmpeg/ffmpeg_common.cc b/media/ffmpeg/ffmpeg_common.cc
index 8a504d71393c92723ddad38eb682ac562b0e0277..2ae0b1b804631d5906cedc2d52f1bdd747682f33 100644
--- a/media/ffmpeg/ffmpeg_common.cc
+++ b/media/ffmpeg/ffmpeg_common.cc
@@ -289,8 +289,10 @@ void AVCodecContextToAudioDecoderConfig(
                      codec_context->extradata_size,
                      false,  // Not encrypted.
                      true);
-  DCHECK_EQ(av_get_bytes_per_sample(codec_context->sample_fmt) * 8,
-            config->bits_per_channel());
+  if (codec != kCodecOpus) {

[fbarchard1, 2013/01/14 20:42:30]

add a comment or todo to remove this.

[fbarchard1, 2013/01/15 02:26:50]

Isn't Opus already in current ffmpeg?  I ran some tests on ffmpeg with opus on
webm and it seemed ok?  Or do you mean opus is disabled in chromium's version of
ffmpeg?

[Tom Finegan, 2013/01/15 02:44:51]

Disabled in chromium's version.

+    DCHECK_EQ(av_get_bytes_per_sample(codec_context->sample_fmt) * 8,
+              config->bits_per_channel());
+  }
 }
 
 void AudioDecoderConfigToAVCodecContext(const AudioDecoderConfig& config,