Factor gfx::Transform and skia_util into their own component.

ui/gfx/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/ui.gni")
 import("//testing/test.gni")
 import("//testing/libfuzzer/fuzzer_test.gni")
 
 if (is_android) {
   import("//build/config/android/config.gni")
   import("//build/config/android/rules.gni")
 }
 
 # Several targets want to include this header file, and some of them are
 # child dependencies of "gfx". Therefore, we separate it out here so multiple
 # targets can all have a dependency for header checking purposes without
 # creating circular dependencies.
 source_set("gfx_export") {
   sources = [
     "gfx_export.h",
   ]
 }
 
+component("transform") {

[danakj, 2017/03/14 21:25:00]

Can you put these in geometry?

[danakj, 2017/03/14 21:36:05]

OK this is fine then but I just don't think skia_util inside "transform" makes
sense. Uhhhhmm.. how about skia_geometry?

[danakj, 2017/03/14 21:36:52]

Or geometry_skia.. maybe? Less namespacey?

+  sources = [
+    "skia_util.cc",
+    "skia_util.h",
+    "transform.cc",
+    "transform.h",
+    "transform_util.cc",
+    "transform_util.h",
+  ]
+  public_deps = [
+    ":gfx_export",
+  ]
+  deps = [
+    "//base",
+    "//skia",
+    "//ui/gfx/geometry",
+  ]
+  defines = [ "GFX_IMPLEMENTATION" ]
+}
+
 component("gfx") {
   sources = [
     "android/gfx_jni_registrar.cc",
     "android/gfx_jni_registrar.h",
     "android/java_bitmap.cc",
     "android/java_bitmap.h",
     "android/view_configuration.cc",
     "android/view_configuration.h",
     "animation/animation.cc",
     "animation/animation.h",
@@ skipping 131 matching lines @@
     "selection_model.cc",
     "selection_model.h",
     "sequential_id_generator.cc",
     "sequential_id_generator.h",
     "shadow_value.cc",
     "shadow_value.h",
     "skbitmap_operations.cc",
     "skbitmap_operations.h",
     "skia_color_space_util.cc",
     "skia_color_space_util.h",
-    "skia_util.cc",
-    "skia_util.h",
     "switches.cc",
     "switches.h",
     "sys_color_change_listener.cc",
     "sys_color_change_listener.h",
     "text_constants.h",
     "text_elider.cc",
     "text_elider.h",
     "text_utils.cc",
     "text_utils.h",
     "text_utils_android.cc",
     "text_utils_ios.mm",
-    "transform.cc",
-    "transform.h",
-    "transform_util.cc",
-    "transform_util.h",
     "ui_gfx_exports.cc",
     "utf16_indexing.cc",
     "utf16_indexing.h",
     "vsync_provider.cc",
     "vsync_provider.h",
     "win/direct_manipulation.cc",
     "win/direct_manipulation.h",
     "win/direct_write.cc",
     "win/direct_write.h",
     "win/hwnd_util.cc",
@@ skipping 46 matching lines @@
   ]
 
   # This is part of the gfx component in the component build.
   defines = [ "GFX_IMPLEMENTATION" ]
 
   public_deps = [
     ":color_space",
     ":memory_buffer_sources",
     ":native_widget_types",
     ":selection_bound_sources",
+    ":transform",
     "//base",
     "//skia",
     "//third_party/icu",
     "//ui/gfx/geometry",
     "//ui/gfx/range",
   ]
   deps = [
     ":gfx_export",
     "//base",
     "//base:base_static",
@@ skipping 503 matching lines @@
 
 fuzzer_test("color_transform_fuzztest") {
   sources = [
     "color_transform_fuzzer.cc",
   ]
   deps = [
     ":gfx",
   ]
   libfuzzer_options = [ "max_len=1024" ]
 }