Use a table-based implementation of SkDefaultXform

src/opts/SkColorXform_opts.h

Index: src/opts/SkColorXform_opts.h
diff --git a/src/opts/SkColorXform_opts.h b/src/opts/SkColorXform_opts.h
index 2c14c802831e51950b799054ee73e67b5021a3a0..c19a7fa545a62fb3e90176347eb37fbd1a15a406 100644
--- a/src/opts/SkColorXform_opts.h
+++ b/src/opts/SkColorXform_opts.h
@@ -260,22 +260,22 @@ static void color_xform_RGB1(uint32_t* dst, const uint32_t* src, int len,
     }
 }
 
-static void color_xform_RGB1_srgb_to_2dot2(uint32_t* dst, const uint32_t* src, int len,
+inline void color_xform_RGB1_srgb_to_2dot2(uint32_t* dst, const uint32_t* src, int len,

[mtklein_C, 2016/06/21 20:32:52]

?

These should probably be best as static inline.

[msarett, 2016/06/21 21:24:47]

Think we can just go back to static now that I'm not including this file
anymore?

                                            const float matrix[16]) {
     color_xform_RGB1<linear_from_srgb, linear_to_2dot2>(dst, src, len, matrix);
 }
 
-static void color_xform_RGB1_2dot2_to_2dot2(uint32_t* dst, const uint32_t* src, int len,
+inline void color_xform_RGB1_2dot2_to_2dot2(uint32_t* dst, const uint32_t* src, int len,
                                            const float matrix[16]) {
     color_xform_RGB1<linear_from_2dot2, linear_to_2dot2>(dst, src, len, matrix);
 }
 
-static void color_xform_RGB1_srgb_to_srgb(uint32_t* dst, const uint32_t* src, int len,
+inline void color_xform_RGB1_srgb_to_srgb(uint32_t* dst, const uint32_t* src, int len,
                                            const float matrix[16]) {
     color_xform_RGB1<linear_from_srgb, linear_to_srgb>(dst, src, len, matrix);
 }
 
-static void color_xform_RGB1_2dot2_to_srgb(uint32_t* dst, const uint32_t* src, int len,
+inline void color_xform_RGB1_2dot2_to_srgb(uint32_t* dst, const uint32_t* src, int len,
                                            const float matrix[16]) {
     color_xform_RGB1<linear_from_2dot2, linear_to_srgb>(dst, src, len, matrix);
 }