diff options
| author |  Robert Phillips <robertphillips@google.com> | 2016-11-23 19:37:13 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-11-25 21:36:00 +0000 |
| commit | 93f1633abca95e302fdd31ece0f4d602b0b26708 (patch) | |
| tree | 6748ace1f1f7fc629e29511e22ffe96e023432b8 /src/gpu/GrBlurUtils.cpp | |
| parent | 58cda8fa495225bbcb3d93bdd7f81ec515402475 (diff) | |
Fix gpu blurring on platforms that "useDrawInsteadOfClear"
Platforms that "useDrawInsteadOfClear" take a different path in
GrRenderTargetContext::internalClear. The different path involves
a lot of comparisons of the drawn rect with the bounds of the RenderTarget. Since the RenderTargets are now deferred (and the instantiated version might be larger than the proxied size) case must be taken in prematurely optimizing away clears and draws.
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=5148
TBR=bsalomon@google.com
Change-Id: If63b6393fb66328175bc92de211e7242e903f25e
Reviewed-on: https://skia-review.googlesource.com/5148
Reviewed-by: Robert Phillips <robertphillips@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
Diffstat (limited to 'src/gpu/GrBlurUtils.cpp')
| -rw-r--r-- | src/gpu/GrBlurUtils.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/gpu/GrBlurUtils.cpp b/src/gpu/GrBlurUtils.cpp index 97fa623e2f..8f14bf75e5 100644 --- a/src/gpu/GrBlurUtils.cpp +++ b/src/gpu/GrBlurUtils.cpp @@ -141,7 +141,8 @@ static void draw_path_with_mask_filter(GrContext* context, SkASSERT(maskFilter); SkIRect clipBounds; - clip.getConservativeBounds(renderTargetContext->width(), renderTargetContext->height(), + clip.getConservativeBounds(renderTargetContext->worstCaseWidth(), + renderTargetContext->worstCaseHeight(), &clipBounds); SkTLazy<SkPath> tmpPath; SkStrokeRec::InitStyle fillOrHairline; |