Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(602)

Unified Diff: media/base/video_decoder_config.cc

Issue 10910293: Add is_encrypted() in VideoDecoderConfig. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move is_encrypted into VideoDecoderConfig. Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/video_decoder_config.cc
diff --git a/media/base/video_decoder_config.cc b/media/base/video_decoder_config.cc
index 1cbf5f81fbfdaf88cfaa6767ee320ee0dab374d1..9bf3543bb6fe7aa99772a6c765e2a66a1922d5b1 100644
--- a/media/base/video_decoder_config.cc
+++ b/media/base/video_decoder_config.cc
@@ -23,9 +23,10 @@ VideoDecoderConfig::VideoDecoderConfig(VideoCodec codec,
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() {}
@@ -60,6 +61,7 @@ void VideoDecoderConfig::Initialize(VideoCodec codec,
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));
@@ -90,6 +92,8 @@ void VideoDecoderConfig::Initialize(VideoCodec codec,
} else {
extra_data_.reset();
}
+
+ is_encrypted_ = is_encrypted;
}
void VideoDecoderConfig::CopyFrom(const VideoDecoderConfig& video_config) {
@@ -101,6 +105,7 @@ void VideoDecoderConfig::CopyFrom(const VideoDecoderConfig& video_config) {
video_config.natural_size(),
video_config.extra_data(),
video_config.extra_data_size(),
+ video_config.is_encrypted(),
false);
}
@@ -120,7 +125,8 @@ bool VideoDecoderConfig::Matches(const VideoDecoderConfig& config) const {
(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 {
@@ -134,7 +140,8 @@ std::string VideoDecoderConfig::AsHumanReadableString() const {
<< "," << 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();
}
@@ -170,4 +177,12 @@ 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

Powered by Google App Engine
This is Rietveld 408576698