Add is_encrypted() in VideoDecoderConfig.

media/base/video_decoder_config.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/base/video_decoder_config.h"
 
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 
 namespace media {
 
 VideoDecoderConfig::VideoDecoderConfig()
     : codec_(kUnknownVideoCodec),
       profile_(VIDEO_CODEC_PROFILE_UNKNOWN),
       format_(VideoFrame::INVALID),
       extra_data_size_(0) {
 }
 
 VideoDecoderConfig::VideoDecoderConfig(VideoCodec codec,
                                        VideoCodecProfile profile,
                                        VideoFrame::Format format,
                                        const gfx::Size& coded_size,
                                        const gfx::Rect& visible_rect,
                                        const gfx::Size& natural_size,
                                        const uint8* extra_data,
-                                       size_t extra_data_size) {
+                                       size_t extra_data_size,
+                                       bool is_encrypted) {
   Initialize(codec, profile, format, coded_size, visible_rect, natural_size,
-             extra_data, extra_data_size, true);
+             extra_data, extra_data_size, is_encrypted, true);
 }
 
 VideoDecoderConfig::~VideoDecoderConfig() {}
 
 // Some videos just want to watch the world burn, with a height of 0; cap the
 // "infinite" aspect ratio resulting.
 static const int kInfiniteRatio = 99999;
 
 // Common aspect ratios (multiplied by 100 and truncated) used for histogramming
 // video sizes.  These were taken on 20111103 from
@@ skipping 14 matching lines @@
 }
 
 void VideoDecoderConfig::Initialize(VideoCodec codec,
                                     VideoCodecProfile profile,
                                     VideoFrame::Format format,
                                     const gfx::Size& coded_size,
                                     const gfx::Rect& visible_rect,
                                     const gfx::Size& natural_size,
                                     const uint8* extra_data,
                                     size_t extra_data_size,
+                                    bool is_encrypted,
                                     bool record_stats) {
   CHECK((extra_data_size != 0) == (extra_data != NULL));
 
   if (record_stats) {
     UMA_HISTOGRAM_ENUMERATION("Media.VideoCodec", codec, kVideoCodecMax + 1);
     // Drop UNKNOWN because U_H_E() uses one bucket for all values less than 1.
     if (profile >= 0) {
       UMA_HISTOGRAM_ENUMERATION("Media.VideoCodecProfile", profile,
                                 VIDEO_CODEC_PROFILE_MAX + 1);
     }
     UMA_HISTOGRAM_COUNTS_10000("Media.VideoCodedWidth", coded_size.width());
     UmaHistogramAspectRatio("Media.VideoCodedAspectRatio", coded_size);
     UMA_HISTOGRAM_COUNTS_10000("Media.VideoVisibleWidth", visible_rect.width());
     UmaHistogramAspectRatio("Media.VideoVisibleAspectRatio", visible_rect);

[scherkus (not reviewing), 2012/09/17 15:54:10]

do we want a histogram for is_encrypted?

[xhwang, 2012/09/17 19:42:25]

Interesting point. But why not? The encryption is only on frame level and what
we are reporting here are not encrypted.

   }
 
   codec_ = codec;
   profile_ = profile;
   format_ = format;
   coded_size_ = coded_size;
   visible_rect_ = visible_rect;
   natural_size_ = natural_size;
   extra_data_size_ = extra_data_size;
 
   if (extra_data_size_ > 0) {
     extra_data_.reset(new uint8[extra_data_size_]);
     memcpy(extra_data_.get(), extra_data, extra_data_size_);
   } else {
     extra_data_.reset();
   }
+
+  is_encrypted_ = is_encrypted;
 }
 
 void VideoDecoderConfig::CopyFrom(const VideoDecoderConfig& video_config) {
   Initialize(video_config.codec(),
              video_config.profile(),
              video_config.format(),
              video_config.coded_size(),
              video_config.visible_rect(),
              video_config.natural_size(),
              video_config.extra_data(),
              video_config.extra_data_size(),
+             video_config.is_encrypted(),
              false);
 }
 
 bool VideoDecoderConfig::IsValidConfig() const {
   return codec_ != kUnknownVideoCodec &&
       natural_size_.width() > 0 &&
       natural_size_.height() > 0 &&
       VideoFrame::IsValidConfig(format_, visible_rect().size(), natural_size_);
 }
 
 bool VideoDecoderConfig::Matches(const VideoDecoderConfig& config) const {
   return ((codec() == config.codec()) &&
           (format() == config.format()) &&
           (profile() == config.profile()) &&
           (coded_size() == config.coded_size()) &&
           (visible_rect() == config.visible_rect()) &&
           (natural_size() == config.natural_size()) &&
           (extra_data_size() == config.extra_data_size()) &&
           (!extra_data() || !memcmp(extra_data(), config.extra_data(),
-                                    extra_data_size())));
+                                    extra_data_size())) &&
+          (is_encrypted() == config.is_encrypted()));
 }
 
 std::string VideoDecoderConfig::AsHumanReadableString() const {
   std::ostringstream s;
   s << "codec: " << codec()
     << " format: " << format()
     << " coded size: [" << coded_size().width()
     << "," << coded_size().height() << "]"
     << " visible rect: [" << visible_rect().x()
     << "," << visible_rect().y()
     << "," << visible_rect().width()
     << "," << visible_rect().height() << "]"
     << " natural size: [" << natural_size().width()
-    << "," << natural_size().height() << "]";
+    << "," << natural_size().height() << "]"
+    << " encryption: [" << (is_encrypted() ? "true" : "false") << "]";
   return s.str();
 }
 
 VideoCodec VideoDecoderConfig::codec() const {
   return codec_;
 }
 
 VideoCodecProfile VideoDecoderConfig::profile() const {
   return profile_;
 }
@@ skipping 15 matching lines @@
 }
 
 uint8* VideoDecoderConfig::extra_data() const {
   return extra_data_.get();
 }
 
 size_t VideoDecoderConfig::extra_data_size() const {
   return extra_data_size_;
 }
 
+bool VideoDecoderConfig::is_encrypted() const {
+  return is_encrypted_;
+}
+
+void VideoDecoderConfig::set_is_encrypted(bool is_encrypted) {
+  is_encrypted_ = is_encrypted;
+}
+
 }  // namespace media