diff options
author | 2016-07-06 09:59:43 -0700 | |
---|---|---|
committer | 2016-07-06 09:59:43 -0700 | |
commit | 0d28e574ac73fef8bf75cab083ffe23f2d8860a1 (patch) | |
tree | 0831082f5905199c8c5deab72d63b109b960d817 /src/gpu/gl/GrGLGpu.h | |
parent | 01a19504c4dbd980e96b1c5b9fe8c8779432488d (diff) |
Fix caching of sample locations
The original caching logic for sample locations wishfully assumed that
the GPU would always use the same sample pattern for render targets
that had the same number of samples. It turns out we can't rely on
that. This change improves the caching logic to handle mismatched
simple patterns with the same count, and adds a unit test that
emulates different sample patterns observed on real hardware.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2111423002
Committed: https://skia.googlesource.com/skia/+/09d49a3bfe2d1e652a648ce1ea0962b38d10d166
Review-Url: https://codereview.chromium.org/2111423002
Diffstat (limited to 'src/gpu/gl/GrGLGpu.h')
-rw-r--r-- | src/gpu/gl/GrGLGpu.h | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/gpu/gl/GrGLGpu.h b/src/gpu/gl/GrGLGpu.h index 013613ad27..c8edbb9c28 100644 --- a/src/gpu/gl/GrGLGpu.h +++ b/src/gpu/gl/GrGLGpu.h @@ -216,10 +216,8 @@ private: const SkIRect& srcRect, const SkIPoint& dstPoint) override; - void onGetMultisampleSpecs(GrRenderTarget*, - const GrStencilSettings&, - int* effectiveSampleCnt, - SkAutoTDeleteArray<SkPoint>* sampleLocations) override; + void onGetMultisampleSpecs(GrRenderTarget*, const GrStencilSettings&, + int* effectiveSampleCnt, SamplePattern*) override; // binds texture unit in GL void setTextureUnit(int unitIdx); |