diff options
| author |  brianosman <brianosman@google.com> | 2016-12-01 13:59:31 -0800 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-12-01 13:59:31 -0800 |
| commit | a6abb57b99ec273505dd7f0c72321c41f6e9a4ab (patch) | |
| tree | 048be347960f5f72bbfab13279c55057e7bf78c0 /tools/gpu/GrContextFactory.h | |
| parent | 271dabaeb246ffbad88c87072432a6670751c7a8 (diff) | |
Revert of Enable sRGB on iOS, make sRGB decode support optional (patchset #12 id:220001 of https://codereview.chromium.org/2539993002/ )
Reason for revert:
Command Buffer, too...
Original issue's description:
> 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
TBR=bsalomon@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=skia:4148
Review-Url: https://codereview.chromium.org/2546783005
Diffstat (limited to 'tools/gpu/GrContextFactory.h')
| -rw-r--r-- | tools/gpu/GrContextFactory.h | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/tools/gpu/GrContextFactory.h b/tools/gpu/GrContextFactory.h index a232112d75..54b6611e0f 100644 --- a/tools/gpu/GrContextFactory.h +++ b/tools/gpu/GrContextFactory.h @@ -93,11 +93,10 @@ public: * to not using GL_NV_path_rendering extension even when the driver supports it. */ enum class ContextOptions { - kNone = 0x0, - kEnableNVPR = 0x1, - kUseInstanced = 0x2, - kRequireSRGBSupport = 0x4, - kRequireSRGBDecodeDisableSupport = 0x8, + kNone = 0x0, + kEnableNVPR = 0x1, + kUseInstanced = 0x2, + kRequireSRGBSupport = 0x4, }; static ContextType NativeContextTypeForBackend(GrBackend backend) { |