Use consistent types for ICC profiles

third_party/WebKit/Source/platform/graphics/ColorBehavior.cpp

Index: third_party/WebKit/Source/platform/graphics/ColorBehavior.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/ColorBehavior.cpp b/third_party/WebKit/Source/platform/graphics/ColorBehavior.cpp
index 67635875686d8640ce9b87e9d641c11c609650ef..2e61629145bddeb2feadc0ddfe0283c5a132fe8c 100644
--- a/third_party/WebKit/Source/platform/graphics/ColorBehavior.cpp
+++ b/third_party/WebKit/Source/platform/graphics/ColorBehavior.cpp
@@ -18,9 +18,9 @@ SkColorSpace* gTargetColorSpace = nullptr;
 }  // namespace
 
 // static
-void ColorBehavior::setGlobalTargetColorProfile(
-    const WebVector<char>& profile) {
-  if (profile.isEmpty())
+void ColorBehavior::setGlobalTargetColorSpace(
+    const sk_sp<SkColorSpace>& colorSpace) {
+  if (!colorSpace)
     return;
 
   // Take a lock around initializing and accessing the global device color
@@ -31,8 +31,9 @@ void ColorBehavior::setGlobalTargetColorProfile(
   if (gTargetColorSpace)
     return;
 
-  gTargetColorSpace =
-      SkColorSpace::MakeICC(profile.data(), profile.size()).release();
+  // Ensure that the color space object be leaked.

[jbroman, 2016/12/28 17:08:20]

nit: "is leaked"?

[ccameron, 2016/12/28 21:52:56]

It was intended to be the subjunctive ... but I'm not 100% sure that this is the
right place to use subjunctive.

[jbroman, 2016/12/29 00:09:28]

Indicative sounds better to me, but after searching the web it seems that there
are at least some people who think the subjunctive is fine here. So feel free to
disregard my comment. (I'd assumed a typo -- I don't mean to debate the finer
points of English grammar.)

+  gTargetColorSpace = colorSpace.get();
+  SkSafeRef(gTargetColorSpace);
 
   // UMA statistics.
   BitmapImageMetrics::countOutputGammaAndGamut(gTargetColorSpace);