From 61201641383dada029f79be4df98d925efc1a827 Mon Sep 17 00:00:00 2001 From: Mike Klein Date: Thu, 7 Jun 2018 14:46:07 -0400 Subject: remove "srgb" config from DM,nanobench Now that "srgb" is broken I don't want to accidentally run it. "srgbnl" if of course identical, and not broken so much as simply not yet working. :) While here, simplify the configs we run in nanobench too, eliminating 565 and moving F16 to GCE-only (i.e. fast, abundant machines). Similarly, remove "adobe" VIA that doesn't use Adobe RGB correctly... Change-Id: Ic295dec97a2caadadbe8500655243db36dd2c43d Reviewed-on: https://skia-review.googlesource.com/132932 Commit-Queue: Mike Klein Commit-Queue: Brian Osman Auto-Submit: Mike Klein Reviewed-by: Brian Osman --- dm/DM.cpp | 9 --------- 1 file changed, 9 deletions(-) (limited to 'dm') diff --git a/dm/DM.cpp b/dm/DM.cpp index 69c2504191..34ffa541ca 100644 --- a/dm/DM.cpp +++ b/dm/DM.cpp @@ -916,26 +916,17 @@ static Sink* create_sink(const GrContextOptions& grCtxOptions, const SkCommandLi SINK("pdfa", PDFSink, true, SK_ScalarDefaultRasterDPI); SINK("pdf300", PDFSink, false, 300); SINK("jsdebug", DebugSink); - - // TODO: kRGBA_sRGB_SkColorType / srgbLinearColorSpace? - SINK("srgb", RasterSink, kN32_SkColorType, srgbColorSpace); } #undef SINK return nullptr; } -static sk_sp adobe_rgb() { - return SkColorSpace::MakeRGB(SkColorSpace::kSRGB_RenderTargetGamma, - SkColorSpace::kAdobeRGB_Gamut); -} - static sk_sp rgb_to_gbr() { return SkColorSpace::MakeSRGB()->makeColorSpin(); } static Sink* create_via(const SkString& tag, Sink* wrapped) { #define VIA(t, via, ...) if (tag.equals(t)) { return new via(__VA_ARGS__); } - VIA("adobe", ViaCSXform, wrapped, adobe_rgb(), false); VIA("gbr", ViaCSXform, wrapped, rgb_to_gbr(), true); VIA("lite", ViaLite, wrapped); VIA("pipe", ViaPipe, wrapped); -- cgit v1.2.3