commit 670a290163a7de3766a7fab9c3b2dc21b3ee143e
author Brian Osman <brianosman@google.com> Mon Apr 23 13:31:56 2018 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Apr 23 17:55:10 2018 +0000
tree 022a6f81d3be59ca1d33014004d37988e549e73e
parent 773a639d431a14295ac7ed02389d217733a7edcd

Enable skcms by default

This effectively switches Flutter and Android Framework to use skcms for
color space transformation.

Change-Id: I254f6a8614d6976ff13979ae0e745c49c334e5c6
Reviewed-on: https://skia-review.googlesource.com/123080
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Mike Klein <mtklein@chromium.org>

BUILD.gn

diff --git a/BUILD.gn b/BUILD.gn
index 99d0d57..3eb5c54 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -30,7 +30,7 @@
   skia_use_zlib = true
   skia_use_metal = false
   skia_use_libheif = is_skia_dev_build
-  skia_use_skcms = is_skia_dev_build
+  skia_use_skcms = true
 
   skia_android_serial = ""
   skia_enable_discrete_gpu = true

gn/gn_to_bp.py

diff --git a/gn/gn_to_bp.py b/gn/gn_to_bp.py
index d8ec9e3..3e25ad5 100644
--- a/gn/gn_to_bp.py
+++ b/gn/gn_to_bp.py
@@ -233,6 +233,9 @@
 export_includes = strip_slashes(js['targets']['//:public']['include_dirs'])
 defines      = [str(d) for d in js['targets']['//:skia']['defines']]
 
+# We need to add skcms.c, which lives in third_party
+srcs.add("third_party/skcms/skcms.c")
+
 dm_srcs         = strip_slashes(js['targets']['//:dm']['sources'])
 dm_includes     = strip_slashes(js['targets']['//:dm']['include_dirs'])