update callers to not use SkColorProfileType

src/core/SkColorSpace.cpp

 /*
  * Copyright 2016 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkColorSpace.h"
+#include "SkColorSpacePriv.h"
 #include "SkColorSpace_Base.h"
 #include "SkEndian.h"
 #include "SkOnce.h"
 
 #define SkColorSpacePrintf(...)
 
 static bool color_space_almost_equal(float a, float b) {
     return SkTAbs(a - b) < 0.01f;
 }
 
@@ skipping 1182 matching lines @@
     ptr32[4] = SkEndian_SwapBE32(0x000116cc);
     ptr += kTAG_XYZ_Bytes;
 
     // Write copyright tag
     memcpy(ptr, gEmptyTextTag, sizeof(gEmptyTextTag));
 
     // TODO (msarett): Should we try to hold onto the data so we can return immediately if
     //                 the client calls again?
     return SkData::MakeFromMalloc(profile.release(), kICCProfileSize);
 }
+
+///////////////////////////////////////////////////////////////////////////////////////////////////
+
+bool SkColorSpacePriv::EQ(SkColorSpace* a, SkColorSpace* b) {
+    if (!a) {
+        return !b;
+    }
+    if (!b) {
+        return false;
+    }
+    // TODO: could have an approx EQ where we compare gamma and xyz for a near-match, since this
+    //       operator is called to detect when they're close enough for a "fast-case"
+    return a == b;

[msarett, 2016/06/20 19:41:05]

So this will only work when both are sRGB or Adobe RGB (because we have once
ptrs).

Other profiles could be equivalent (or nearly equivalent) which we could
determine with more work.

I think a TODO is fine.  We may want to consider FastEq, SlowEq, ApproxEq, etc.

[reed1, 2016/06/20 21:05:53]

Removed

+}