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

Unified Diff: chromecast/common/media/cma_param_traits.cc

Issue 1786733004: Revert of media config: expand is_encrypted to a struct. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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
« no previous file with comments | « chromecast/common/media/cma_param_traits.h ('k') | chromecast/common/media/cma_param_traits_macros.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromecast/common/media/cma_param_traits.cc
diff --git a/chromecast/common/media/cma_param_traits.cc b/chromecast/common/media/cma_param_traits.cc
index f0ddc49a632252826ead493f139a5469e69b7639..f27043db3f96f2aeb1b54339db860c2577f00cad 100644
--- a/chromecast/common/media/cma_param_traits.cc
+++ b/chromecast/common/media/cma_param_traits.cc
@@ -12,7 +12,6 @@
#include "content/public/common/common_param_traits.h"
#include "ipc/ipc_message_macros.h"
#include "media/base/audio_decoder_config.h"
-#include "media/base/encryption_scheme.h"
#include "media/base/video_decoder_config.h"
#include "ui/gfx/ipc/gfx_param_traits.h"
@@ -30,16 +29,6 @@
namespace IPC {
-template <>
-struct ParamTraits<media::EncryptionScheme::Pattern> {
- typedef media::EncryptionScheme::Pattern param_type;
- static void Write(base::Pickle* m, const param_type& p);
- static bool Read(const base::Pickle* m, base::PickleIterator* iter,
- param_type* r);
- static void Log(const param_type& p, std::string* l);
-};
-
-
void ParamTraits<media::AudioDecoderConfig>::Write(
base::Pickle* m,
const media::AudioDecoderConfig& p) {
@@ -47,7 +36,7 @@
WriteParam(m, p.sample_format());
WriteParam(m, p.channel_layout());
WriteParam(m, p.samples_per_second());
- WriteParam(m, p.encryption_scheme());
+ WriteParam(m, p.is_encrypted());
WriteParam(m, p.extra_data());
}
@@ -59,17 +48,15 @@
media::SampleFormat sample_format;
media::ChannelLayout channel_layout;
int samples_per_second;
- media::EncryptionScheme encryption_scheme;
+ bool is_encrypted;
std::vector<uint8_t> extra_data;
if (!ReadParam(m, iter, &codec) || !ReadParam(m, iter, &sample_format) ||
!ReadParam(m, iter, &channel_layout) ||
!ReadParam(m, iter, &samples_per_second) ||
- !ReadParam(m, iter, &encryption_scheme) ||
- !ReadParam(m, iter, &extra_data))
+ !ReadParam(m, iter, &is_encrypted) || !ReadParam(m, iter, &extra_data))
return false;
*r = media::AudioDecoderConfig(codec, sample_format, channel_layout,
- samples_per_second, extra_data,
- encryption_scheme);
+ samples_per_second, extra_data, is_encrypted);
return true;
}
@@ -88,7 +75,7 @@
WriteParam(m, p.coded_size());
WriteParam(m, p.visible_rect());
WriteParam(m, p.natural_size());
- WriteParam(m, p.encryption_scheme());
+ WriteParam(m, p.is_encrypted());
WriteParam(m, p.extra_data());
}
@@ -103,18 +90,17 @@
gfx::Size coded_size;
gfx::Rect visible_rect;
gfx::Size natural_size;
- media::EncryptionScheme encryption_scheme;
+ bool is_encrypted;
std::vector<uint8_t> extra_data;
if (!ReadParam(m, iter, &codec) || !ReadParam(m, iter, &profile) ||
!ReadParam(m, iter, &format) || !ReadParam(m, iter, &color_space) ||
!ReadParam(m, iter, &coded_size) || !ReadParam(m, iter, &visible_rect) ||
!ReadParam(m, iter, &natural_size) ||
- !ReadParam(m, iter, &encryption_scheme) ||
- !ReadParam(m, iter, &extra_data))
+ !ReadParam(m, iter, &is_encrypted) || !ReadParam(m, iter, &extra_data))
return false;
*r = media::VideoDecoderConfig(codec, profile, format, color_space,
coded_size, visible_rect, natural_size,
- extra_data, encryption_scheme);
+ extra_data, is_encrypted);
return true;
}
@@ -123,45 +109,4 @@
l->append(base::StringPrintf("<VideoDecoderConfig>"));
}
-void ParamTraits<media::EncryptionScheme>::Write(
- base::Pickle* m, const param_type& p) {
- WriteParam(m, p.mode());
- WriteParam(m, p.pattern());
-}
-
-bool ParamTraits<media::EncryptionScheme>::Read(
- const base::Pickle* m, base::PickleIterator* iter, param_type* r) {
- media::EncryptionScheme::CipherMode mode;
- media::EncryptionScheme::Pattern pattern;
- if (!ReadParam(m, iter, &mode) || !ReadParam(m, iter, &pattern))
- return false;
- *r = media::EncryptionScheme(mode, pattern);
- return true;
-}
-
-void ParamTraits<media::EncryptionScheme>::Log(
- const param_type& p, std::string* l) {
- l->append(base::StringPrintf("<EncryptionScheme>"));
-}
-
-void ParamTraits<media::EncryptionScheme::Pattern>::Write(
- base::Pickle* m, const param_type& p) {
- WriteParam(m, p.encrypt_blocks());
- WriteParam(m, p.skip_blocks());
-}
-
-bool ParamTraits<media::EncryptionScheme::Pattern>::Read(
- const base::Pickle* m, base::PickleIterator* iter, param_type* r) {
- uint32_t encrypt_blocks, skip_blocks;
- if (!ReadParam(m, iter, &encrypt_blocks) || !ReadParam(m, iter, &skip_blocks))
- return false;
- *r = media::EncryptionScheme::Pattern(encrypt_blocks, skip_blocks);
- return true;
-}
-
-void ParamTraits<media::EncryptionScheme::Pattern>::Log(
- const param_type& p, std::string* l) {
- l->append(base::StringPrintf("<Pattern>"));
-}
-
} // namespace IPC
« no previous file with comments | « chromecast/common/media/cma_param_traits.h ('k') | chromecast/common/media/cma_param_traits_macros.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698