Hide (mostly) unused SkColorSpace writeToICC() function

This originally was intended to be used for serialization, but
we've chosen another approach.

Includes a revert of:
https://codereview.chromium.org/2012133003/

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2193223002

Review-Url: https://codereview.chromium.org/2193223002
diff --git a/src/core/SkColorSpace_Base.h b/src/core/SkColorSpace_Base.h
index f450773..3a2518d 100644
--- a/src/core/SkColorSpace_Base.h
+++ b/src/core/SkColorSpace_Base.h
@@ -180,13 +180,17 @@
 
     const SkColorLookUpTable* colorLUT() const { return fColorLUT.get(); }
 
+private:
+
     /**
-     *  Writes this object as an ICC profile.
+     *  FIXME (msarett):
+     *  Hiding this function until we can determine if we need it.  Known issues include:
+     *  Only writes 3x3 matrices
+     *  Only writes float gammas
+     *  Rejected by some parsers because the "profile description" is empty
      */
     sk_sp<SkData> writeToICC() const;
 
-private:
-
     static sk_sp<SkColorSpace> NewRGB(GammaNamed gammaNamed, const SkMatrix44& toXYZD50);
 
     SkColorSpace_Base(GammaNamed gammaNamed, const SkMatrix44& toXYZ, Named named);
@@ -200,6 +204,7 @@
 
     friend class SkColorSpace;
     friend class ColorSpaceXformTest;
+    friend class ColorSpaceTest;
     typedef SkColorSpace INHERITED;
 };