media config: expand is_encrypted to a struct.

media/ffmpeg/ffmpeg_common.cc

 // Copyright (c) 2012 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 "media/ffmpeg/ffmpeg_common.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/sha1.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "build/build_config.h"
 #include "media/base/decoder_buffer.h"
+#include "media/base/encryption_scheme.h"
 #include "media/base/video_decoder_config.h"
 #include "media/base/video_util.h"
 #include "media/media_features.h"
 
 namespace media {
 
 // Why FF_INPUT_BUFFER_PADDING_SIZE? FFmpeg assumes all input buffers are
 // padded. Check here to ensure FFmpeg only receives data padded to its
 // specifications.
 static_assert(DecoderBuffer::kPaddingSize >= FF_INPUT_BUFFER_PADDING_SIZE,
@@ skipping 338 matching lines @@
                << " extra data cannot have size of "
                << codec_context->extradata_size << ".";
     return false;
   }
 
   std::vector<uint8_t> extra_data;
   if (codec_context->extradata_size > 0) {
     extra_data.assign(codec_context->extradata,
                       codec_context->extradata + codec_context->extradata_size);
   }
-  config->Initialize(codec,
-                     sample_format,
-                     channel_layout,
-                     sample_rate,
-                     extra_data,
-                     is_encrypted,
-                     seek_preroll,
+  config->Initialize(codec, sample_format, channel_layout, sample_rate,
+                     extra_data, EncryptionScheme(is_encrypted), seek_preroll,

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

FFmpeg only supports CTR without a pattern, so we should not hide that
information. Perhaps this should be pushed up the the call stack, but for now we
can pass the correct scheme.

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

Done.

                      codec_context->delay);
 
   // Verify that AudioConfig.bits_per_channel was calculated correctly for
   // codecs that have |sample_fmt| set by FFmpeg.
   switch (codec) {
     case kCodecOpus:
 #if BUILDFLAG(ENABLE_AC3_EAC3_AUDIO_DEMUXING)
     case kCodecAC3:
     case kCodecEAC3:
 #endif
@@ skipping 137 matching lines @@
                << stream->codec->extradata_size << ".";
     return false;
   }
 
   std::vector<uint8_t> extra_data;
   if (stream->codec->extradata_size > 0) {
     extra_data.assign(stream->codec->extradata,
                       stream->codec->extradata + stream->codec->extradata_size);
   }
   config->Initialize(codec, profile, format, color_space, coded_size,
-                     visible_rect, natural_size, extra_data, is_encrypted);
+                     visible_rect, natural_size, extra_data,
+                     EncryptionScheme(is_encrypted));

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

ditto

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

Done.

   return true;
 }
 
 void VideoDecoderConfigToAVCodecContext(
     const VideoDecoderConfig& config,
     AVCodecContext* codec_context) {
   codec_context->codec_type = AVMEDIA_TYPE_VIDEO;
   codec_context->codec_id = VideoCodecToCodecID(config.codec());
   codec_context->profile = VideoCodecProfileToProfileID(config.profile());
   codec_context->coded_width = config.coded_size().width();
@@ skipping 211 matching lines @@
 }
 
 int32_t HashCodecName(const char* codec_name) {
   // Use the first 32-bits from the SHA1 hash as the identifier.
   int32_t hash;
   memcpy(&hash, base::SHA1HashString(codec_name).substr(0, 4).c_str(), 4);
   return hash;
 }
 
 }  // namespace media