tidy up skcms_internal.h

Lots of things are now static in skcms.c.

I also removed the debug-mode profiling feature.

The only thing I really was on the fence about was removing
the unit test for isfinite_().  Seems nice, but on the other
hand, kind of hard for us to get isfinite_() wrong.  In C++11
I think we could make this constexpr and tested at compile time
with static_assert, so maybe just put off worrying until then?

Change-Id: I1ae326bad8fa6924245444521f8bd2d1e7d1bed1
Reviewed-on: https://skia-review.googlesource.com/138939
Commit-Queue: Mike Klein <mtklein@chromium.org>
Auto-Submit: Mike Klein <mtklein@chromium.org>
Reviewed-by: Brian Osman <brianosman@google.com>
5 files changed