diff options
| author |  brianosman <brianosman@google.com> | 2016-12-02 06:43:32 -0800 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-12-02 06:43:32 -0800 |
| commit | 20471894eaa441193d5ae8f2395e8244c91c55af (patch) | |
| tree | cc003f32bc938c79dcc8ea6bd477193924b235b5 /tools/gpu/GrContextFactory.cpp | |
| parent | 4b7b6f0229fa51f5beb71f92cb77ba84d39b41e1 (diff) | |
Two (related) changes here:
1) Our older iOS devices failed our sRGB tests, due to precision issues
with alpha. At this point, we only test on iPadMini 4, and that appears
not to have any problems.
2) iOS devices still don't have the sRGB texture decode extension. But,
some clients have no interest in mixing legacy/color-correct rendering,
and would like to use sRGB on these devices. This GrContextOptions flag
enables sRGB support in those cases.
Adjust the test code to produce sRGB capable contexts on these devices,
but only for configs that have a color space. (See comment).
BUG=skia:4148
Committed: https://skia.googlesource.com/skia/+/9db12d2341f3f8722c8b90b11dd4cce138a8a64e
Committed: https://skia.googlesource.com/skia/+/1aeb78c5d978b35b256525b711edd942bce01444
Review-Url: https://codereview.chromium.org/2539993002
Diffstat (limited to 'tools/gpu/GrContextFactory.cpp')
| -rw-r--r-- | tools/gpu/GrContextFactory.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tools/gpu/GrContextFactory.cpp b/tools/gpu/GrContextFactory.cpp index ffc5295ab2..8d9bd41245 100644 --- a/tools/gpu/GrContextFactory.cpp +++ b/tools/gpu/GrContextFactory.cpp @@ -198,6 +198,8 @@ ContextInfo GrContextFactory::getContextInfo(ContextType type, ContextOptions op if (ContextOptions::kUseInstanced & options) { grOptions.fEnableInstancedRendering = true; } + grOptions.fRequireDecodeDisableForSRGB = + SkToBool(ContextOptions::kRequireSRGBDecodeDisableSupport & options); grCtx.reset(GrContext::Create(backend, backendContext, grOptions)); if (!grCtx.get()) { return ContextInfo(); |