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

Unified Diff: media/base/audio_decoder_config.h

Issue 7867051: Introduce AudioDecoderConfig to migrate away from GetAVStream(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: blam Created 9 years, 3 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 | « no previous file | media/base/audio_decoder_config.cc » ('j') | media/base/audio_decoder_config.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/audio_decoder_config.h
diff --git a/media/base/audio_decoder_config.h b/media/base/audio_decoder_config.h
new file mode 100644
index 0000000000000000000000000000000000000000..95625ef497f5fa7eb6dd6dc4c6663099353a0a8a
--- /dev/null
+++ b/media/base/audio_decoder_config.h
@@ -0,0 +1,75 @@
+// Copyright (c) 2011 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.
+
+#ifndef MEDIA_BASE_AUDIO_DECODER_CONFIG_H_
+#define MEDIA_BASE_AUDIO_DECODER_CONFIG_H_
+
+#include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"
+#include "media/base/channel_layout.h"
+#include "media/base/media_export.h"
+
+namespace media {
+
+enum AudioCodec {
+ kUnknownAudioCodec,
+ kCodecAAC,
+ kCodecMP3,
+ kCodecPCM,
+ kCodecVorbis,
+
+ // DO NOT ADD RANDOM AUDIO CODECS!
+ //
+ // The only acceptable time to add a new codec is if there is production code
+ // that uses said codec in the same CL.
+};
+
+class MEDIA_EXPORT AudioDecoderConfig {
+ public:
+ // Constructs an uninitialized object. Clients should call Initialize() with
+ // appropriate values before using.
+ AudioDecoderConfig();
+
+ // Constructs an initialized object. It is acceptable to pass in NULL for
+ // |extra_data|, otherwise the memory is copied.
+ AudioDecoderConfig(AudioCodec codec, int bits_per_channel,
+ ChannelLayout channel_layout, int sample_rate,
+ const uint8* extra_data, size_t extra_data_size);
+
+ ~AudioDecoderConfig();
+
+ // Resets the internal state of this object.
+ void Initialize(AudioCodec codec, int bits_per_channel,
+ ChannelLayout channel_layout, int sample_rate,
+ const uint8* extra_data, size_t extra_data_size);
+
+ // Returns true if this object has appropriate configuration values, false
+ // otherwise.
+ bool IsValidConfig() const;
+
+ AudioCodec codec() const;
+ int bits_per_channel() const;
+ ChannelLayout channel_layout() const;
+ int sample_rate() const;
+
+ // Optional byte data required to initialize audio decoders such as Vorbis
+ // codebooks.
+ uint8* extra_data() const;
+ size_t extra_data_size() const;
+
+ private:
+ AudioCodec codec_;
+ int bits_per_channel_;
+ ChannelLayout channel_layout_;
+ int sample_rate_;
+
+ scoped_array<uint8> extra_data_;
+ size_t extra_data_size_;
+
+ DISALLOW_COPY_AND_ASSIGN(AudioDecoderConfig);
+};
+
+} // namespace media
+
+#endif // MEDIA_BASE_AUDIO_DECODER_CONFIG_H_
« no previous file with comments | « no previous file | media/base/audio_decoder_config.cc » ('j') | media/base/audio_decoder_config.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698