color: Fix accidental partial color conversion

ui/gfx/color_transform.cc

 // Copyright (c) 2016 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_transform.h"
 
 #include <algorithm>
 #include <cmath>
 #include <list>
 #include <memory>
@@ skipping 600 matching lines @@
     // TODO(hubbe): shrink gamuts here (never stretch gamuts)
   }
 
   steps_.push_back(
       base::MakeUnique<ColorTransformMatrix>(GetRangeAdjustMatrix(from)));
 
   steps_.push_back(
       base::MakeUnique<ColorTransformMatrix>(Invert(GetTransferMatrix(from))));
 
   // If the target color space is not defined, just apply the adjust and
-  // tranfer matrices.
+  // tranfer matrices. This path is used by YUV to RGB color conversion
+  // when full color conversion is not enabled.
   if (!to.IsValid())
     return;
 
   SkColorSpaceTransferFn to_linear_fn;
   bool to_linear_fn_valid = from.GetTransferFunction(&to_linear_fn);
   steps_.push_back(base::MakeUnique<ColorTransformToLinear>(
       from.transfer_, to_linear_fn, to_linear_fn_valid));
 
   if (from.matrix_ == ColorSpace::MatrixID::BT2020_CL) {
     // BT2020 CL is a special case.
@@ skipping 93 matching lines @@
 
 ColorTransformInternal::ColorTransformInternal(const ColorSpace& src,
                                                const ColorSpace& dst,
                                                Intent intent)
     : src_(src), dst_(dst) {
   // If no source color space is specified, do no transformation.
   // TODO(ccameron): We may want dst assume sRGB at some point in the future.
   if (!src_.IsValid())
     return;
 
-  ScopedQcmsProfile src_profile = GetQCMSProfileIfNecessary(src_);
-  ScopedQcmsProfile dst_profile = GetQCMSProfileIfNecessary(dst_);
+  // If the target color space is not defined, just apply the adjust and
+  // tranfer matrices. This path is used by YUV to RGB color conversion
+  // when full color conversion is not enabled.
+  ScopedQcmsProfile src_profile;
+  ScopedQcmsProfile dst_profile;
+  if (dst.IsValid()) {
+    src_profile = GetQCMSProfileIfNecessary(src_);
+    dst_profile = GetQCMSProfileIfNecessary(dst_);
+  }
   bool has_src_profile = !!src_profile;
   bool has_dst_profile = !!dst_profile;
 
   if (src_profile) {
     steps_.push_back(base::MakeUnique<QCMSColorTransform>(
         std::move(src_profile), GetXYZD50Profile()));
   }
 
   AppendColorSpaceToColorSpaceTransform(
       has_src_profile ? ColorSpace::CreateXYZD50() : src_,
@@ skipping 65 matching lines @@
     const ColorSpace& to,
     Intent intent) {
   return std::unique_ptr<ColorTransform>(
       new ColorTransformInternal(from, to, intent));
 }
 
 ColorTransform::ColorTransform() {}
 ColorTransform::~ColorTransform() {}
 
 }  // namespace gfx