Switch to AVIO instead of a custom FFmpeg URLProtocol handler.

media/filters/ffmpeg_audio_decoder.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/filters/ffmpeg_audio_decoder.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/location.h"
 #include "base/message_loop_proxy.h"
@@ skipping 36 matching lines @@
       last_input_timestamp_(kNoTimestamp()),
       output_bytes_to_drop_(0),
       av_frame_(NULL) {
 }
 
 void FFmpegAudioDecoder::Initialize(
     const scoped_refptr<DemuxerStream>& stream,
     const PipelineStatusCB& status_cb,
     const StatisticsCB& statistics_cb) {
   // Ensure FFmpeg has been initialized
-  FFmpegGlue::GetInstance();
+  FFmpegGlue::InitializeFFmpeg();

[scherkus (not reviewing), 2012/10/02 00:20:56]

do you know why we still need this?

[DaleCurtis, 2012/10/02 01:24:23]

I think it's just a safety check. The unittests need to call it at least once,
but both the audio and video decoder unittests do so.  We can probably remove
it.

 
   if (!message_loop_) {
     message_loop_ = base::ResetAndReturn(&message_loop_factory_cb_).Run();
   } else {
     // TODO(scherkus): initialization currently happens more than once in
     // PipelineIntegrationTest.BasicPlayback.
     LOG(ERROR) << "Initialize has already been called.";
   }
   message_loop_->PostTask(
       FROM_HERE,
@@ skipping 308 matching lines @@
       &FFmpegAudioDecoder::DoDecodeBuffer, this, status, buffer));
 }
 
 base::TimeDelta FFmpegAudioDecoder::GetNextOutputTimestamp() const {
   DCHECK(output_timestamp_base_ != kNoTimestamp());
   double decoded_us = (total_frames_decoded_ / samples_per_second_) *
       base::Time::kMicrosecondsPerSecond;
   return output_timestamp_base_ + base::TimeDelta::FromMicroseconds(decoded_us);
 }
 }  // namespace media