media config: expand is_encrypted to a struct.

chromecast/media/audio/cast_audio_output_stream_unittest.cc

Index: chromecast/media/audio/cast_audio_output_stream_unittest.cc
diff --git a/chromecast/media/audio/cast_audio_output_stream_unittest.cc b/chromecast/media/audio/cast_audio_output_stream_unittest.cc
index 6d2f734c4b4007e3584e48c7fa50db075bdae1e0..7abea7e3878abb849536d99a21108046cd34333e 100644
--- a/chromecast/media/audio/cast_audio_output_stream_unittest.cc
+++ b/chromecast/media/audio/cast_audio_output_stream_unittest.cc
@@ -381,7 +381,7 @@ TEST_F(CastAudioOutputStreamTest, Format) {
     const AudioConfig& audio_config = audio_decoder->config();
     EXPECT_EQ(kCodecPCM, audio_config.codec);
     EXPECT_EQ(kSampleFormatS16, audio_config.sample_format);
-    EXPECT_FALSE(audio_config.is_encrypted);
+    EXPECT_FALSE(audio_config.encryption_scheme.is_encrypted);

[ddorwin, 2015/12/10 18:36:00]

Do we not have any similar tests for non-Chromecast code? Or perhaps those are
hidden by is_encrypted() calls. Please find and update any such tests.

[ddorwin, 2015/12/10 18:36:00]

Are there other members we can/should test?

[dougsteed, 2015/12/14 21:19:01]

Pretty sure I updated all the tests, as it would not compile otherwise.

[dougsteed, 2015/12/14 21:19:01]

Do you mean members of audio_config, or of audio_config.encryption_scheme ? In
any case, I am not familiar with the nuances of these tests, so all I am doing
is trying to preserve the semantics of the existing test.

[ddorwin, 2016/03/01 02:17:40]

I was referring to EncryptionScheme. The values should be whatever the defaults
are (no encryption) or the values for CENC v1.

[ddorwin, 2016/03/01 02:17:41]

This comment doesn't actually apply here. I moved it to audio_decoder_config.h.

 
     CloseStream(stream);
   }