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

Unified Diff: ppapi/proxy/ppapi_param_traits.cc

Issue 150403006: [PPAPI][MediaStream] Support configure for video input. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update Created 6 years, 10 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: ppapi/proxy/ppapi_param_traits.cc
diff --git a/ppapi/proxy/ppapi_param_traits.cc b/ppapi/proxy/ppapi_param_traits.cc
index 1e91f5cdf46bc5bfe5ecfda6e0421c1b003ed572..9b5e1078c4addaa8b7cc69e2a231194d45bab773 100644
--- a/ppapi/proxy/ppapi_param_traits.cc
+++ b/ppapi/proxy/ppapi_param_traits.cc
@@ -633,4 +633,42 @@ void ParamTraits<ppapi::SocketOptionData>::Log(const param_type& p,
std::string* l) {
}
+// ppapi::MediaStreamVideoTrack::Attributes ------------------------------------
+
+// static
+void ParamTraits<ppapi::MediaStreamVideoTrackShared::Attributes>::Write(
+ Message* m,
+ const param_type& p) {
+ ParamTraits<uint32_t>::Write(m, p.mask);
+ ParamTraits<int32_t>::Write(m, p.buffers);
+ ParamTraits<int32_t>::Write(m, p.width);
+ ParamTraits<int32_t>::Write(m, p.height);
+ ParamTraits<int32_t>::Write(m, static_cast<int32_t>(p.format));
+}
+
+// static
+bool ParamTraits<ppapi::MediaStreamVideoTrackShared::Attributes>::Read(
+ const Message* m,
+ PickleIterator* iter,
+ param_type* r) {
+ if (!ParamTraits<uint32_t>::Read(m, iter, &(r->mask)))
+ return false;
+ if (!ParamTraits<int32_t>::Read(m, iter, &(r->buffers)))
+ return false;
+ if (!ParamTraits<int32_t>::Read(m, iter, &(r->width)))
+ return false;
+ if (!ParamTraits<int32_t>::Read(m, iter, &(r->height)))
+ return false;
+ int32_t value = 0;
+ if (!ParamTraits<int32_t>::Read(m, iter, &value))
+ return false;
+ r->format = static_cast<PP_VideoFrame_Format>(value);
yzshen1 2014/02/13 19:23:26 This is not safe, you might get an invalid value i
Peng 2014/02/13 21:28:34 Done.
+ return true;
+}
+
+// static
+void ParamTraits<ppapi::MediaStreamVideoTrackShared::Attributes>::Log(
+ const param_type& p,
+ std::string* l) {
+}
} // namespace IPC

Powered by Google App Engine
This is Rietveld 408576698