Video Color Managament

cc/ipc/cc_param_traits.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 "cc/ipc/cc_param_traits.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/numerics/safe_conversions.h"
@@ skipping 867 matching lines @@
 }
 
 void ParamTraits<cc::YUVVideoDrawQuad>::GetSize(base::PickleSizer* s,
                                                 const param_type& p) {
   ParamTraits<cc::DrawQuad>::GetSize(s, p);
   GetParamSize(s, p.ya_tex_coord_rect);
   GetParamSize(s, p.uv_tex_coord_rect);
   GetParamSize(s, p.ya_tex_size);
   GetParamSize(s, p.uv_tex_size);
   GetParamSize(s, p.color_space);
+  GetParamSize(s, p.video_color_space);
   GetParamSize(s, p.resource_offset);
   GetParamSize(s, p.resource_multiplier);
   GetParamSize(s, p.bits_per_channel);
 }
 
 void ParamTraits<cc::YUVVideoDrawQuad>::Write(base::Pickle* m,
                                               const param_type& p) {
   ParamTraits<cc::DrawQuad>::Write(m, p);
   WriteParam(m, p.ya_tex_coord_rect);
   WriteParam(m, p.uv_tex_coord_rect);
   WriteParam(m, p.ya_tex_size);
   WriteParam(m, p.uv_tex_size);
   WriteParam(m, p.color_space);
+  WriteParam(m, p.video_color_space);
   WriteParam(m, p.resource_offset);
   WriteParam(m, p.resource_multiplier);
   WriteParam(m, p.bits_per_channel);
 }
 
 bool ParamTraits<cc::YUVVideoDrawQuad>::Read(const base::Pickle* m,
                                              base::PickleIterator* iter,
                                              param_type* p) {
   return ParamTraits<cc::DrawQuad>::Read(m, iter, p) &&
          ReadParam(m, iter, &p->ya_tex_coord_rect) &&
          ReadParam(m, iter, &p->uv_tex_coord_rect) &&
          ReadParam(m, iter, &p->ya_tex_size) &&
          ReadParam(m, iter, &p->uv_tex_size) &&
          ReadParam(m, iter, &p->color_space) &&
+         ReadParam(m, iter, &p->video_color_space) &&
          ReadParam(m, iter, &p->resource_offset) &&
          ReadParam(m, iter, &p->resource_multiplier) &&
          ReadParam(m, iter, &p->bits_per_channel) &&
          p->bits_per_channel >= cc::YUVVideoDrawQuad::kMinBitsPerChannel &&
          p->bits_per_channel <= cc::YUVVideoDrawQuad::kMaxBitsPerChannel;
 }
 
 void ParamTraits<cc::YUVVideoDrawQuad>::Log(const param_type& p,
                                             std::string* l) {
   l->append("(");
   ParamTraits<cc::DrawQuad>::Log(p, l);
   l->append(", ");
   LogParam(p.ya_tex_coord_rect, l);
   l->append(", ");
   LogParam(p.uv_tex_coord_rect, l);
   l->append(", ");
   LogParam(p.ya_tex_size, l);
   l->append(", ");
   LogParam(p.uv_tex_size, l);
   l->append(", ");
   LogParam(p.color_space, l);
   l->append(", ");

[nasko, 2016/08/08 21:15:21]

Shouldn't we also be logging the new member here?

[hubbe, 2016/08/10 17:03:25]

Done.

   LogParam(p.resource_offset, l);
   l->append(", ");
   LogParam(p.resource_multiplier, l);
   l->append(", ");
   LogParam(p.bits_per_channel, l);
   l->append("])");
 }
 
 }  // namespace IPC
 
@@ skipping 17 matching lines @@
 #undef CC_IPC_CC_PARAM_TRAITS_MACROS_H_
 #include "cc/ipc/cc_param_traits_macros.h"
 }  // namespace IPC
 
 // Generate param traits log methods.
 #include "ipc/param_traits_log_macros.h"
 namespace IPC {
 #undef CC_IPC_CC_PARAM_TRAITS_MACROS_H_
 #include "cc/ipc/cc_param_traits_macros.h"
 }  // namespace IPC