media config: expand is_encrypted to a struct.

media/base/fake_demuxer_stream.cc

Index: media/base/fake_demuxer_stream.cc
diff --git a/media/base/fake_demuxer_stream.cc b/media/base/fake_demuxer_stream.cc
index 267b3e5d8d23a6f21f1c6a8c1987b5787b5de360..5bfb2872ed7033a94b4abed9ca29236c99ac2a94 100644
--- a/media/base/fake_demuxer_stream.cc
+++ b/media/base/fake_demuxer_stream.cc
@@ -17,6 +17,7 @@
 #include "base/thread_task_runner_handle.h"
 #include "media/base/bind_to_current_loop.h"
 #include "media/base/decoder_buffer.h"
+#include "media/base/encryption_scheme.h"
 #include "media/base/media_util.h"
 #include "media/base/test_helpers.h"
 #include "media/base/timestamp_constants.h"
@@ -147,11 +148,10 @@ void FakeDemuxerStream::SeekToStart() {
 
 void FakeDemuxerStream::UpdateVideoDecoderConfig() {
   const gfx::Rect kVisibleRect(kStartWidth, kStartHeight);
-  video_decoder_config_.Initialize(kCodecVP8, VIDEO_CODEC_PROFILE_UNKNOWN,
-                                   PIXEL_FORMAT_YV12, COLOR_SPACE_UNSPECIFIED,
-                                   next_coded_size_, kVisibleRect,
-                                   next_coded_size_, EmptyExtraData(),
-                                   is_encrypted_);
+  video_decoder_config_.Initialize(
+      kCodecVP8, VIDEO_CODEC_PROFILE_UNKNOWN, PIXEL_FORMAT_YV12,
+      COLOR_SPACE_UNSPECIFIED, next_coded_size_, kVisibleRect, next_coded_size_,
+      EmptyExtraData(), EncryptionScheme(is_encrypted_));

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

For example, in this case, I'd rather store the encryption scheme (or have a
function that converts the bool) than support an extra constructor, especially
since it hides an important detail that some callers should care about.

[dougsteed, 2016/03/02 18:07:53]

Done.

   next_coded_size_.Enlarge(kWidthDelta, kHeightDelta);
 }