Video Color Managament

cc/quads/yuv_video_draw_quad.h

 // Copyright 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.
 
 #ifndef CC_QUADS_YUV_VIDEO_DRAW_QUAD_H_
 #define CC_QUADS_YUV_VIDEO_DRAW_QUAD_H_
 
 #include <stddef.h>
 
 #include <memory>
 
 #include "cc/base/cc_export.h"
 #include "cc/quads/draw_quad.h"
+#include "ui/gfx/color_space.h"
 #include "ui/gfx/geometry/rect_f.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace cc {
 
 class CC_EXPORT YUVVideoDrawQuad : public DrawQuad {
  public:
   static const size_t kYPlaneResourceIdIndex = 0;
   static const size_t kUPlaneResourceIdIndex = 1;
   static const size_t kVPlaneResourceIdIndex = 2;
@@ skipping 22 matching lines @@
               // coordinates consistent across all quad types: crbug.com/487370
               const gfx::RectF& ya_tex_coord_rect,
               const gfx::RectF& uv_tex_coord_rect,
               const gfx::Size& ya_tex_size,
               const gfx::Size& uv_tex_size,
               unsigned y_plane_resource_id,
               unsigned u_plane_resource_id,
               unsigned v_plane_resource_id,
               unsigned a_plane_resource_id,
               ColorSpace color_space,
+              const gfx::ColorSpace& video_color_space,
               float offset,
               float multiplier,
               uint32_t bits_per_channel);
 
   void SetAll(const SharedQuadState* shared_quad_state,
               const gfx::Rect& rect,
               const gfx::Rect& opaque_rect,
               const gfx::Rect& visible_rect,
               bool needs_blending,
               // |*_tex_coord_rect| contains non-normalized coordinates.
               // TODO(reveman): Make the use of normalized vs non-normalized
               // coordinates consistent across all quad types: crbug.com/487370
               const gfx::RectF& ya_tex_coord_rect,
               const gfx::RectF& uv_tex_coord_rect,
               const gfx::Size& ya_tex_size,
               const gfx::Size& uv_tex_size,
               unsigned y_plane_resource_id,
               unsigned u_plane_resource_id,
               unsigned v_plane_resource_id,
               unsigned a_plane_resource_id,
               ColorSpace color_space,
+              const gfx::ColorSpace& video_color_space,
               float offset,
               float multiplier,
               uint32_t bits_per_channel);
 
   gfx::RectF ya_tex_coord_rect;
   gfx::RectF uv_tex_coord_rect;
   gfx::Size ya_tex_size;
   gfx::Size uv_tex_size;
   ColorSpace color_space;
   float resource_offset = 0.0f;
   float resource_multiplier = 1.0f;
   uint32_t bits_per_channel = 8;
+  gfx::ColorSpace video_color_space;

[ccameron, 2016/08/08 18:45:07]

I'd rather pull this out of the ResourceProvider::ScopedSamplerGL, since we'll
need to be setting the cc::ResourceProvider's parameters for all resources
anyway.

That plumbing isn't present, so I'm fine leaving this as-is, and unifying it
when the ResourceProvider work is in place.

[hubbe, 2016/08/08 19:45:38]

I'll add a TODO.

 
   static const YUVVideoDrawQuad* MaterialCast(const DrawQuad*);
 
   ResourceId y_plane_resource_id() const {
     return resources.ids[kYPlaneResourceIdIndex];
   }
   ResourceId u_plane_resource_id() const {
     return resources.ids[kUPlaneResourceIdIndex];
   }
   ResourceId v_plane_resource_id() const {
     return resources.ids[kVPlaneResourceIdIndex];
   }
   ResourceId a_plane_resource_id() const {
     return resources.ids[kAPlaneResourceIdIndex];
   }
 
  private:
   void ExtendValue(base::trace_event::TracedValue* value) const override;
 };
 
 }  // namespace cc
 
 #endif  // CC_QUADS_YUV_VIDEO_DRAW_QUAD_H_