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

Side by Side Diff: media/base/audio_decoder_config.cc

Issue 743483002: Emit Media Source codec names as media_log events. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix missed call to ChunkDemuxer constructor. Created 6 years, 1 month 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
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/base/audio_decoder_config.h" 5 #include "media/base/audio_decoder_config.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/metrics/histogram.h" 8 #include "base/metrics/histogram.h"
9 #include "base/time/time.h" 9 #include "base/time/time.h"
10 #include "media/audio/sample_rates.h" 10 #include "media/audio/sample_rates.h"
11 #include "media/base/limits.h" 11 #include "media/base/limits.h"
12 #include "media/base/sample_format.h" 12 #include "media/base/sample_format.h"
13 13
14 namespace media { 14 namespace media {
15 15
16 // The names come from src/third_party/ffmpeg/libavcodec/codec_desc.c
17 const char* AudioCodecName(AudioCodec audio_codec, SampleFormat sample_format) {
18 switch (audio_codec) {
19 case kCodecAAC:
20 return "aac";
21 case kCodecMP3:
22 return "mp3";
23 case kCodecPCM:
24 switch (sample_format) {
DaleCurtis 2014/11/19 00:01:56 I wouldn't really worry about differentiating thes
watk 2014/11/19 19:18:09 Done.
25 case kUnknownSampleFormat:
26 return "pcm";
27 case kSampleFormatU8:
28 return "pcm_u8";
29 case kSampleFormatS16:
30 return "pcm_s16le";
31 case kSampleFormatS32:
32 return "pcm_s24le";
33 case kSampleFormatF32:
34 return "pcm_f32le";
35 case kSampleFormatPlanarS16:
36 return "pcm_s16le_planar";
37 case kSampleFormatPlanarF32:
38 return "pcm_f32le_planar";
39 default:
40 NOTREACHED() << "Invalid SampleFormat: " << sample_format;
41 return "pcm";
42 }
43 case kCodecPCM_S16BE:
44 return "pcm_s16be";
45 case kCodecPCM_S24BE:
46 return "pcm_s24be";
47 case kCodecVorbis:
48 return "vorbis";
49 case kCodecFLAC:
50 return "flac";
51 case kCodecAMR_NB:
52 return "amr_nb";
53 case kCodecAMR_WB:
54 return "amr_wb";
55 case kCodecGSM_MS:
56 return "gsm_ms";
57 case kCodecPCM_ALAW:
58 return "pcm_alaw";
59 case kCodecPCM_MULAW:
60 return "pcm_mulaw";
61 case kCodecOpus:
62 return "opus";
63 default:
64 NOTREACHED() << "Invalid AudioCodec: " << audio_codec;
65 return "unknown";
66 }
67 }
68
16 AudioDecoderConfig::AudioDecoderConfig() 69 AudioDecoderConfig::AudioDecoderConfig()
17 : codec_(kUnknownAudioCodec), 70 : codec_(kUnknownAudioCodec),
18 sample_format_(kUnknownSampleFormat), 71 sample_format_(kUnknownSampleFormat),
19 bytes_per_channel_(0), 72 bytes_per_channel_(0),
20 channel_layout_(CHANNEL_LAYOUT_UNSUPPORTED), 73 channel_layout_(CHANNEL_LAYOUT_UNSUPPORTED),
21 samples_per_second_(0), 74 samples_per_second_(0),
22 bytes_per_frame_(0), 75 bytes_per_frame_(0),
23 is_encrypted_(false), 76 is_encrypted_(false),
24 codec_delay_(0) { 77 codec_delay_(0) {
25 } 78 }
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 << " sample_format: " << sample_format() 168 << " sample_format: " << sample_format()
116 << " bytes_per_frame: " << bytes_per_frame() 169 << " bytes_per_frame: " << bytes_per_frame()
117 << " seek_preroll: " << seek_preroll().InMilliseconds() << "ms" 170 << " seek_preroll: " << seek_preroll().InMilliseconds() << "ms"
118 << " codec_delay: " << codec_delay() 171 << " codec_delay: " << codec_delay()
119 << " has extra data? " << (extra_data() ? "true" : "false") 172 << " has extra data? " << (extra_data() ? "true" : "false")
120 << " encrypted? " << (is_encrypted() ? "true" : "false"); 173 << " encrypted? " << (is_encrypted() ? "true" : "false");
121 return s.str(); 174 return s.str();
122 } 175 }
123 176
124 } // namespace media 177 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698