diff options
Diffstat (limited to 'src/core/SkColorSpace.cpp')
-rw-r--r-- | src/core/SkColorSpace.cpp | 55 |
1 files changed, 33 insertions, 22 deletions
diff --git a/src/core/SkColorSpace.cpp b/src/core/SkColorSpace.cpp index 33f8402da3..06ec26d101 100644 --- a/src/core/SkColorSpace.cpp +++ b/src/core/SkColorSpace.cpp @@ -116,12 +116,17 @@ sk_sp<SkColorSpace> SkColorSpace_Base::MakeRGB(SkGammaNamed gammaNamed, const Sk switch (gammaNamed) { case kSRGB_SkGammaNamed: if (xyz_almost_equal(toXYZD50, gSRGB_toXYZD50)) { - return SkColorSpace::MakeSRGB(); + return SkColorSpace_Base::MakeNamed(kSRGB_Named); + } + break; + case k2Dot2Curve_SkGammaNamed: + if (xyz_almost_equal(toXYZD50, gAdobeRGB_toXYZD50)) { + return SkColorSpace_Base::MakeNamed(kAdobeRGB_Named); } break; case kLinear_SkGammaNamed: if (xyz_almost_equal(toXYZD50, gSRGB_toXYZD50)) { - return SkColorSpace::MakeSRGBLinear(); + return SkColorSpace_Base::MakeNamed(kSRGBLinear_Named); } break; case kNonStandard_SkGammaNamed: @@ -196,6 +201,10 @@ 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; @@ -205,12 +214,22 @@ 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 sk_ref_sp(srgb()); + return SkColorSpace_Base::MakeNamed(SkColorSpace_Base::kSRGB_Named); } sk_sp<SkColorSpace> SkColorSpace::MakeSRGBLinear() { - return sk_ref_sp(srgb_linear()); + return SkColorSpace_Base::MakeNamed(SkColorSpace_Base::kSRGBLinear_Named); } /////////////////////////////////////////////////////////////////////////////////////////////////// @@ -271,13 +290,6 @@ 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. @@ -310,7 +322,7 @@ struct ColorSpaceHeader { SkASSERT(k0_Version == version); header.fVersion = (uint8_t) version; - SkASSERT(named <= kSRGBLinear_NamedColorSpace); + SkASSERT(named <= SkColorSpace_Base::kSRGBLinear_Named); header.fNamed = (uint8_t) named; SkASSERT(gammaNamed <= kNonStandard_SkGammaNamed); @@ -339,13 +351,19 @@ size_t SkColorSpace::writeToMemory(void* memory) const { if (this == srgb()) { if (memory) { *((ColorSpaceHeader*) memory) = ColorSpaceHeader::Pack( - k0_Version, kSRGB_NamedColorSpace, gammaNamed, 0); + 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); } return sizeof(ColorSpaceHeader); } else if (this == srgb_linear()) { if (memory) { *((ColorSpaceHeader*) memory) = ColorSpaceHeader::Pack( - k0_Version, kSRGBLinear_NamedColorSpace, gammaNamed, 0); + k0_Version, SkColorSpace_Base::kSRGBLinear_Named, gammaNamed, 0); } return sizeof(ColorSpaceHeader); } @@ -437,14 +455,7 @@ 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) { - 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); - } + return SkColorSpace_Base::MakeNamed((SkColorSpace_Base::Named) header.fNamed); } switch ((SkGammaNamed) header.fGammaNamed) { |