diff options
author | Brian Osman <brianosman@google.com> | 2017-12-13 15:44:40 -0500 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-12-13 21:34:30 +0000 |
commit | 411b8ea74d66d08252d9b617d7e7d458604dbc2e (patch) | |
tree | dc0f3e55e2ccb8b0fa7148745b44dae8e251b746 /src/core/SkColorSpace.cpp | |
parent | b67821da87ffa711c07af75ee618b13dbccd6b7d (diff) |
Remove SkColorSpace_Base::MakeNamed, along with most uses of Adobe RGB
Bug: skia:
Change-Id: If5935eac48184bc8cbe4db21dac4d6033a8704e6
Reviewed-on: https://skia-review.googlesource.com/84200
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/core/SkColorSpace.cpp')
-rw-r--r-- | src/core/SkColorSpace.cpp | 55 |
1 files changed, 22 insertions, 33 deletions
diff --git a/src/core/SkColorSpace.cpp b/src/core/SkColorSpace.cpp index 06ec26d101..33f8402da3 100644 --- a/src/core/SkColorSpace.cpp +++ b/src/core/SkColorSpace.cpp @@ -116,17 +116,12 @@ sk_sp<SkColorSpace> SkColorSpace_Base::MakeRGB(SkGammaNamed gammaNamed, const Sk switch (gammaNamed) { case kSRGB_SkGammaNamed: if (xyz_almost_equal(toXYZD50, gSRGB_toXYZD50)) { - return SkColorSpace_Base::MakeNamed(kSRGB_Named); - } - break; - case k2Dot2Curve_SkGammaNamed: - if (xyz_almost_equal(toXYZD50, gAdobeRGB_toXYZD50)) { - return SkColorSpace_Base::MakeNamed(kAdobeRGB_Named); + return SkColorSpace::MakeSRGB(); } break; case kLinear_SkGammaNamed: if (xyz_almost_equal(toXYZD50, gSRGB_toXYZD50)) { - return SkColorSpace_Base::MakeNamed(kSRGBLinear_Named); + return SkColorSpace::MakeSRGBLinear(); } break; case kNonStandard_SkGammaNamed: @@ -201,10 +196,6 @@ static SkColorSpace* singleton_colorspace(SkGammaNamed gamma, const float to_xyz return new SkColorSpace_XYZ(gamma, m44); } -static SkColorSpace* adobe_rgb() { - static SkColorSpace* cs = singleton_colorspace(k2Dot2Curve_SkGammaNamed, gAdobeRGB_toXYZD50); - return cs; -} static SkColorSpace* srgb() { static SkColorSpace* cs = singleton_colorspace(kSRGB_SkGammaNamed, gSRGB_toXYZD50); return cs; @@ -214,22 +205,12 @@ static SkColorSpace* srgb_linear() { return cs; } -sk_sp<SkColorSpace> SkColorSpace_Base::MakeNamed(Named named) { - switch (named) { - case kSRGB_Named: return sk_ref_sp(srgb()); - case kAdobeRGB_Named: return sk_ref_sp(adobe_rgb()); - case kSRGBLinear_Named: return sk_ref_sp(srgb_linear()); - default: break; - } - return nullptr; -} - sk_sp<SkColorSpace> SkColorSpace::MakeSRGB() { - return SkColorSpace_Base::MakeNamed(SkColorSpace_Base::kSRGB_Named); + return sk_ref_sp(srgb()); } sk_sp<SkColorSpace> SkColorSpace::MakeSRGBLinear() { - return SkColorSpace_Base::MakeNamed(SkColorSpace_Base::kSRGBLinear_Named); + return sk_ref_sp(srgb_linear()); } /////////////////////////////////////////////////////////////////////////////////////////////////// @@ -290,6 +271,13 @@ enum Version { k0_Version, // Initial version, header + flags for matrix and profile }; +enum NamedColorSpace { + kSRGB_NamedColorSpace, + // No longer a singleton, preserved to support reading data from branches m65 and older + kAdobeRGB_NamedColorSpace, + kSRGBLinear_NamedColorSpace, +}; + struct ColorSpaceHeader { /** * It is only valid to set zero or one flags. @@ -322,7 +310,7 @@ struct ColorSpaceHeader { SkASSERT(k0_Version == version); header.fVersion = (uint8_t) version; - SkASSERT(named <= SkColorSpace_Base::kSRGBLinear_Named); + SkASSERT(named <= kSRGBLinear_NamedColorSpace); header.fNamed = (uint8_t) named; SkASSERT(gammaNamed <= kNonStandard_SkGammaNamed); @@ -351,19 +339,13 @@ size_t SkColorSpace::writeToMemory(void* memory) const { if (this == srgb()) { if (memory) { *((ColorSpaceHeader*) memory) = ColorSpaceHeader::Pack( - k0_Version, SkColorSpace_Base::kSRGB_Named, gammaNamed, 0); - } - return sizeof(ColorSpaceHeader); - } else if (this == adobe_rgb()) { - if (memory) { - *((ColorSpaceHeader*) memory) = ColorSpaceHeader::Pack( - k0_Version, SkColorSpace_Base::kAdobeRGB_Named, gammaNamed, 0); + k0_Version, kSRGB_NamedColorSpace, gammaNamed, 0); } return sizeof(ColorSpaceHeader); } else if (this == srgb_linear()) { if (memory) { *((ColorSpaceHeader*) memory) = ColorSpaceHeader::Pack( - k0_Version, SkColorSpace_Base::kSRGBLinear_Named, gammaNamed, 0); + k0_Version, kSRGBLinear_NamedColorSpace, gammaNamed, 0); } return sizeof(ColorSpaceHeader); } @@ -455,7 +437,14 @@ sk_sp<SkColorSpace> SkColorSpace::Deserialize(const void* data, size_t length) { data = SkTAddOffset<const void>(data, sizeof(ColorSpaceHeader)); length -= sizeof(ColorSpaceHeader); if (0 == header.fFlags) { - return SkColorSpace_Base::MakeNamed((SkColorSpace_Base::Named) header.fNamed); + switch ((NamedColorSpace)header.fNamed) { + case kSRGB_NamedColorSpace: + return SkColorSpace::MakeSRGB(); + case kSRGBLinear_NamedColorSpace: + return SkColorSpace::MakeSRGBLinear(); + case kAdobeRGB_NamedColorSpace: + return SkColorSpace::MakeRGB(g2Dot2_TransferFn, SkColorSpace::kAdobeRGB_Gamut); + } } switch ((SkGammaNamed) header.fGammaNamed) { |