Introduce VideoConfig/AudioConfig class for CMA backend

chromecast/media/cma/base/decoder_config_adapter.cc

+// Copyright 2015 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 "chromecast/media/cma/base/decoder_config_adapter.h"
+
+#include "chromecast/media/cma/public/audio_config.h"
+#include "chromecast/media/cma/public/video_config.h"
+#include "media/base/audio_decoder_config.h"
+#include "media/base/channel_layout.h"
+#include "media/base/video_decoder_config.h"
+
+namespace chromecast {
+namespace media {
+
+AudioConfig DecoderConfigAdapter::ToAudioConfig(
+    const ::media::AudioDecoderConfig& config) {
+  if (!config.IsValidConfig()) {
+    return AudioConfig();
+  }
+
+  return AudioConfig(
+      static_cast<AudioConfig::Codec>(config.codec()),

[lcwu1, 2015/04/13 23:55:42]

It's actually dangerous doing this. I am not sure if we want to always keep an
exact same enum with ::media's. I would prefer creating a separate util function
to convert (using a switch statement).

[erickung1, 2015/04/15 22:13:18]

Done. Create util function to do the conversion using switch 

+      static_cast<AudioConfig::ChannelLayout>(config.channel_layout()),
+      ::media::ChannelLayoutToChannelCount(config.channel_layout()),
+      config.bytes_per_channel(),
+      config.samples_per_second(),
+      config.extra_data(),
+      config.extra_data_size(),
+      config.is_encrypted());
+}
+
+VideoConfig DecoderConfigAdapter::ToVideoConfig(
+    const ::media::VideoDecoderConfig& config) {
+  if (!config.IsValidConfig()) {
+    return VideoConfig();
+  }
+
+  return VideoConfig(
+      static_cast<VideoConfig::Codec>(config.codec()),
+      static_cast<VideoConfig::CodecProfile>(config.profile()),
+      config.extra_data(),
+      config.extra_data_size(),
+      config.is_encrypted());
+}
+
+}  // namespace media
+}  // namespace chromecast