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

Side by Side Diff: media/filters/ffmpeg_demuxer.cc

Issue 10690140: Reorganize bitstream converter classes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase again Created 8 years, 5 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « media/filters/ffmpeg_demuxer.h ('k') | media/filters/ffmpeg_h264_bitstream_converter.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "media/filters/ffmpeg_demuxer.h" 5 #include "media/filters/ffmpeg_demuxer.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <string> 8 #include <string>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
11 #include "base/callback.h" 11 #include "base/callback.h"
12 #include "base/command_line.h" 12 #include "base/command_line.h"
13 #include "base/memory/scoped_ptr.h" 13 #include "base/memory/scoped_ptr.h"
14 #include "base/message_loop.h" 14 #include "base/message_loop.h"
15 #include "base/stl_util.h" 15 #include "base/stl_util.h"
16 #include "base/string_util.h" 16 #include "base/string_util.h"
17 #include "base/time.h" 17 #include "base/time.h"
18 #include "media/base/audio_decoder_config.h" 18 #include "media/base/audio_decoder_config.h"
19 #include "media/base/decoder_buffer.h" 19 #include "media/base/decoder_buffer.h"
20 #include "media/base/limits.h" 20 #include "media/base/limits.h"
21 #include "media/base/media_switches.h" 21 #include "media/base/media_switches.h"
22 #include "media/base/video_decoder_config.h" 22 #include "media/base/video_decoder_config.h"
23 #include "media/ffmpeg/ffmpeg_common.h" 23 #include "media/ffmpeg/ffmpeg_common.h"
24 #include "media/filters/bitstream_converter.h"
25 #include "media/filters/ffmpeg_glue.h" 24 #include "media/filters/ffmpeg_glue.h"
26 #include "media/filters/ffmpeg_h264_bitstream_converter.h" 25 #include "media/filters/ffmpeg_h264_to_annex_b_bitstream_converter.h"
27 26
28 namespace media { 27 namespace media {
29 28
30 // 29 //
31 // FFmpegDemuxerStream 30 // FFmpegDemuxerStream
32 // 31 //
33 FFmpegDemuxerStream::FFmpegDemuxerStream( 32 FFmpegDemuxerStream::FFmpegDemuxerStream(
34 FFmpegDemuxer* demuxer, 33 FFmpegDemuxer* demuxer,
35 AVStream* stream) 34 AVStream* stream)
36 : demuxer_(demuxer), 35 : demuxer_(demuxer),
(...skipping 166 matching lines...) Expand 10 before | Expand all | Expand 10 after
203 read_queue_.pop_front(); 202 read_queue_.pop_front();
204 203
205 // Execute the callback. 204 // Execute the callback.
206 read_cb.Run(buffer); 205 read_cb.Run(buffer);
207 } 206 }
208 207
209 void FFmpegDemuxerStream::EnableBitstreamConverter() { 208 void FFmpegDemuxerStream::EnableBitstreamConverter() {
210 // Called by hardware decoder to require different bitstream converter. 209 // Called by hardware decoder to require different bitstream converter.
211 // Currently we assume that converter is determined by codec_id; 210 // Currently we assume that converter is determined by codec_id;
212 DCHECK(stream_); 211 DCHECK(stream_);
213 212 DCHECK_EQ(stream_->codec->codec_id, CODEC_ID_H264);
214 if (stream_->codec->codec_id == CODEC_ID_H264) { 213 bitstream_converter_.reset(
215 // Use Chromium bitstream converter in case of H.264 214 new FFmpegH264ToAnnexBBitstreamConverter(stream_->codec));
216 bitstream_converter_.reset(
217 new FFmpegH264BitstreamConverter(stream_->codec));
218 CHECK(bitstream_converter_->Initialize());
219 } else if (stream_->codec->codec_id == CODEC_ID_MPEG4) {
220 bitstream_converter_.reset(
221 new FFmpegBitstreamConverter("mpeg4video_es", stream_->codec));
222 CHECK(bitstream_converter_->Initialize());
223 } else {
224 NOTREACHED() << "Unsupported bitstream format.";
225 }
226 } 215 }
227 216
228 const AudioDecoderConfig& FFmpegDemuxerStream::audio_decoder_config() { 217 const AudioDecoderConfig& FFmpegDemuxerStream::audio_decoder_config() {
229 CHECK_EQ(type_, AUDIO); 218 CHECK_EQ(type_, AUDIO);
230 return audio_config_; 219 return audio_config_;
231 } 220 }
232 221
233 const VideoDecoderConfig& FFmpegDemuxerStream::video_decoder_config() { 222 const VideoDecoderConfig& FFmpegDemuxerStream::video_decoder_config() {
234 CHECK_EQ(type_, VIDEO); 223 CHECK_EQ(type_, VIDEO);
235 return video_config_; 224 return video_config_;
(...skipping 499 matching lines...) Expand 10 before | Expand all | Expand 10 after
735 } else if (audio) { 724 } else if (audio) {
736 buffered = audio->GetBufferedRanges(); 725 buffered = audio->GetBufferedRanges();
737 } else if (video) { 726 } else if (video) {
738 buffered = video->GetBufferedRanges(); 727 buffered = video->GetBufferedRanges();
739 } 728 }
740 for (size_t i = 0; i < buffered.size(); ++i) 729 for (size_t i = 0; i < buffered.size(); ++i)
741 host_->AddBufferedTimeRange(buffered.start(i), buffered.end(i)); 730 host_->AddBufferedTimeRange(buffered.start(i), buffered.end(i));
742 } 731 }
743 732
744 } // namespace media 733 } // namespace media
OLDNEW
« no previous file with comments | « media/filters/ffmpeg_demuxer.h ('k') | media/filters/ffmpeg_h264_bitstream_converter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698