Towards deleting YUV to RGB computation redundancy

ui/gfx/color_space.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 "ui/gfx/color_space.h"
 
 #include <map>
 
 #include "base/lazy_instance.h"
 #include "base/synchronization/lock.h"
 #include "third_party/skia/include/core/SkColorSpace.h"
 #include "ui/gfx/icc_profile.h"
+#include "ui/gfx/transform.h"
 
 namespace gfx {
 
 namespace {
 
 SkColorSpaceTransferFn InvertTransferFn(SkColorSpaceTransferFn fn) {
   SkColorSpaceTransferFn fn_inv = {0};
   if (fn.fA > 0 && fn.fG > 0) {
     double a_to_the_g = pow(fn.fA, fn.fG);
     fn_inv.fA = 1.f / a_to_the_g;
@@ skipping 86 matching lines @@
 }
 
 // Static
 ColorSpace ColorSpace::CreateXYZD50() {
   return ColorSpace(PrimaryID::XYZ_D50, TransferID::LINEAR, MatrixID::RGB,
                     RangeID::FULL);
 }
 
 // static
 ColorSpace ColorSpace::CreateJpeg() {
-  return ColorSpace(PrimaryID::BT709, TransferID::IEC61966_2_1, MatrixID::BT709,
-                    RangeID::FULL);
+  return ColorSpace(PrimaryID::SMPTE170M, TransferID::IEC61966_2_1,

[hubbe, 2017/02/02 04:51:37]

I thought JPEG used srgb primaries by default?

[ccameron, 2017/02/02 17:33:52]

It's not clear to me what exact spec "YUVVideoDrawQuad::JPEG" was meant to refer
to. I've changed to BT709 primaries and added a "let's find out what was
intended here" comment.

[hubbe, 2017/02/02 18:35:25]

Yeah, YUVVideoDrawQuad::JPEG was normally just used to mean "video with values
from 0-255" it really didn't have an opinion on the primaries I think.  Either
way, that enum is too coarse-grained to accurately reflect the color space of
the video frame...

+                    MatrixID::SMPTE170M, RangeID::FULL);
 }
 
 // static
 ColorSpace ColorSpace::CreateREC601() {
   return ColorSpace(PrimaryID::SMPTE170M, TransferID::SMPTE170M,
                     MatrixID::SMPTE170M, RangeID::LIMITED);
 }
 
 // static
 ColorSpace ColorSpace::CreateREC709() {
@@ skipping 342 matching lines @@
   return false;
 }
 
 bool ColorSpace::GetInverseTransferFunction(SkColorSpaceTransferFn* fn) const {
   if (!GetTransferFunction(fn))
     return false;
   *fn = InvertTransferFn(*fn);
   return true;
 }
 
+void ColorSpace::GetTransferMatrix(SkMatrix44* matrix) const {
+  float Kr = 0;
+  float Kb = 0;
+  switch (matrix_) {
+    case ColorSpace::MatrixID::RGB:
+      matrix->setIdentity();
+      return;
+
+    case ColorSpace::MatrixID::BT709:
+    case ColorSpace::MatrixID::UNSPECIFIED:
+    case ColorSpace::MatrixID::RESERVED:
+    case ColorSpace::MatrixID::UNKNOWN:
+      Kr = 0.2126f;
+      Kb = 0.0722f;
+      break;
+
+    case ColorSpace::MatrixID::FCC:
+      Kr = 0.30f;
+      Kb = 0.11f;
+      break;
+
+    case ColorSpace::MatrixID::BT470BG:
+    case ColorSpace::MatrixID::SMPTE170M:
+      Kr = 0.299f;
+      Kb = 0.114f;
+      break;
+
+    case ColorSpace::MatrixID::SMPTE240M:
+      Kr = 0.212f;
+      Kb = 0.087f;
+      break;
+
+    case ColorSpace::MatrixID::YCOCG: {
+      float data[16] = {
+           0.25f, 0.5f,  0.25f, 0.5f,  // Y
+          -0.25f, 0.5f, -0.25f, 0.5f,  // Cg
+            0.5f, 0.0f,  -0.5f, 0.0f,  // Co
+            0.0f, 0.0f,   0.0f, 1.0f
+      };
+      matrix->setRowMajorf(data);
+      return;
+    }
+
+    // BT2020_CL is a special case.
+    // Basically we return a matrix that transforms RGB values
+    // to RYB values. (We replace the green component with the
+    // the luminance.) Later steps will compute the Cb & Cr values.
+    case ColorSpace::MatrixID::BT2020_CL: {
+      Kr = 0.2627f;
+      Kb = 0.0593f;
+      float data[16] = {
+          1.0f,           0.0f, 0.0f, 0.0f,  // R
+            Kr, 1.0f - Kr - Kb,   Kb, 0.0f,  // Y
+          0.0f,           0.0f, 1.0f, 0.0f,  // B
+          0.0f,           0.0f, 0.0f, 1.0f
+      };
+      matrix->setRowMajorf(data);
+      return;
+    }
+
+    case ColorSpace::MatrixID::BT2020_NCL:
+      Kr = 0.2627f;
+      Kb = 0.0593f;
+      break;
+
+    case ColorSpace::MatrixID::YDZDX: {
+      float data[16] = {
+          0.0f,              1.0f,             0.0f, 0.0f,  // Y
+          0.0f,             -0.5f, 0.986566f / 2.0f, 0.5f,  // DX or DZ
+          0.5f, -0.991902f / 2.0f,             0.0f, 0.5f,  // DZ or DX
+          0.0f,              0.0f,             0.0f, 1.0f,
+      };
+      matrix->setRowMajorf(data);
+      return;
+    }
+  }
+  float Kg = 1.0f - Kr - Kb;
+  float u_m = 0.5f / (1.0f - Kb);
+  float v_m = 0.5f / (1.0f - Kr);
+  float data[16] = {
+                     Kr,        Kg,                Kb, 0.0f,  // Y
+              u_m * -Kr, u_m * -Kg, u_m * (1.0f - Kb), 0.5f,  // U
+      v_m * (1.0f - Kr), v_m * -Kg,         v_m * -Kb, 0.5f,  // V
+                   0.0f,      0.0f,              0.0f, 1.0f,
+  };
+  matrix->setRowMajorf(data);
+}
+
+void ColorSpace::GetRangeAdjustMatrix(SkMatrix44* matrix) const {
+  switch (range_) {
+    case RangeID::FULL:
+    case RangeID::UNSPECIFIED:
+      matrix->setIdentity();
+      return;
+
+    case RangeID::DERIVED:
+    case RangeID::LIMITED:
+      break;
+  }
+  switch (matrix_) {
+    case MatrixID::RGB:
+    case MatrixID::YCOCG:
+      matrix->setScale(255.0f/219.0f, 255.0f/219.0f, 255.0f/219.0f);
+      matrix->postTranslate(-16.0f/219.0f, -16.0f/219.0f, -16.0f/219.0f);
+      break;
+
+    case MatrixID::BT709:
+    case MatrixID::UNSPECIFIED:
+    case MatrixID::RESERVED:
+    case MatrixID::FCC:
+    case MatrixID::BT470BG:
+    case MatrixID::SMPTE170M:
+    case MatrixID::SMPTE240M:
+    case MatrixID::BT2020_NCL:
+    case MatrixID::BT2020_CL:
+    case MatrixID::YDZDX:
+    case MatrixID::UNKNOWN:
+      matrix->setScale(255.0f/219.0f, 255.0f/224.0f, 255.0f/224.0f);
+      matrix->postTranslate(-16.0f/219.0f, -15.5f/224.0f, -15.5f/224.0f);
+      break;
+  }
+}
+
 }  // namespace gfx