Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(40)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: src/effects/SkGpuBlurUtils.cpp

Issue 1407133019: Revert of Make SkBlurImageFilter capable of cropping during blur (GPU path). (Closed) Base URL: senorblanco-linux.mon:src/skia@blur-applyCropRect4-separate-loops
Patch Set: Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/effects/SkGpuBlurUtils.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/effects/SkGpuBlurUtils.cpp
diff --git a/src/effects/SkGpuBlurUtils.cpp b/src/effects/SkGpuBlurUtils.cpp
index e5236775e100429c7d30f5ee4dd3ee597736e170..5d6d014dfda5aec0c8191ce698e00fed30e0af99 100644
--- a/src/effects/SkGpuBlurUtils.cpp
+++ b/src/effects/SkGpuBlurUtils.cpp
@@ -71,20 +71,16 @@
int radiusY,
SkScalar sigmaX,
SkScalar sigmaY,
- const SkRect* srcBounds) {
+ bool useBounds,
+ SkIRect bounds) {
SkRect dstRect = SkRect::MakeWH(srcRect.width(), srcRect.height());
SkMatrix localMatrix = SkMatrix::MakeTrans(srcRect.x(), srcRect.y());
SkISize size = SkISize::Make(2 * radiusX + 1, 2 * radiusY + 1);
SkIPoint kernelOffset = SkIPoint::Make(radiusX, radiusY);
GrPaint paint;
- SkIRect bounds;
- if (srcBounds) {
- srcBounds->roundOut(&bounds);
- }
-
SkAutoTUnref<GrFragmentProcessor> conv(GrMatrixConvolutionEffect::CreateGaussian(
texture, bounds, size, 1.0, 0.0, kernelOffset,
- srcBounds ? GrTextureDomain::kDecal_Mode : GrTextureDomain::kIgnore_Mode,
+ useBounds ? GrTextureDomain::kClamp_Mode : GrTextureDomain::kIgnore_Mode,
true, sigmaX, sigmaY));
paint.addColorFragmentProcessor(conv);
drawContext->drawNonAARectWithLocalMatrix(clip, paint, SkMatrix::I(), dstRect, localMatrix);
@@ -97,62 +93,46 @@
Gr1DKernelEffect::Direction direction,
int radius,
float sigma,
- const SkRect* srcBounds,
- const SkPoint& srcOffset) {
+ bool cropToSrcRect) {
float bounds[2] = { 0.0f, 1.0f };
SkRect dstRect = SkRect::MakeWH(srcRect.width(), srcRect.height());
- if (!srcBounds) {
+ SkPoint srcOffset = SkPoint::Make(srcRect.x(), srcRect.y());
+ if (!cropToSrcRect) {
convolve_gaussian_1d(drawContext, clip, dstRect, srcOffset, texture,
direction, radius, sigma, false, bounds);
return;
}
- SkRect midRect = *srcBounds, leftRect, rightRect;
- midRect.offset(srcOffset);
- SkIRect topRect, bottomRect;
+ SkRect lowerDstRect = dstRect;
+ SkRect middleDstRect = dstRect;
+ SkRect upperDstRect = dstRect;
+ SkScalar size;
SkScalar rad = SkIntToScalar(radius);
if (direction == Gr1DKernelEffect::kX_Direction) {
- bounds[0] = SkScalarToFloat(srcBounds->left()) / texture->width();
- bounds[1] = SkScalarToFloat(srcBounds->right()) / texture->width();
- SkRect::MakeLTRB(0, 0, dstRect.right(), midRect.top()).roundOut(&topRect);
- SkRect::MakeLTRB(0, midRect.bottom(), dstRect.right(), dstRect.bottom())
- .roundOut(&bottomRect);
- midRect.inset(rad, 0);
- leftRect = SkRect::MakeLTRB(0, midRect.top(), midRect.left(), midRect.bottom());
- rightRect =
- SkRect::MakeLTRB(midRect.right(), midRect.top(), dstRect.width(), midRect.bottom());
- dstRect.fTop = midRect.top();
- dstRect.fBottom = midRect.bottom();
+ bounds[0] = SkScalarToFloat(srcRect.left()) / texture->width();
+ bounds[1] = SkScalarToFloat(srcRect.right()) / texture->width();
+ size = dstRect.width();
+ lowerDstRect.fRight = dstRect.left() + rad;
+ upperDstRect.fLeft = dstRect.right() - rad;
+ middleDstRect.inset(rad, 0);
} else {
- bounds[0] = SkScalarToFloat(srcBounds->top()) / texture->height();
- bounds[1] = SkScalarToFloat(srcBounds->bottom()) / texture->height();
- SkRect::MakeLTRB(0, 0, midRect.left(), dstRect.bottom()).roundOut(&topRect);
- SkRect::MakeLTRB(midRect.right(), 0, dstRect.right(), dstRect.bottom())
- .roundOut(&bottomRect);;
- midRect.inset(0, rad);
- leftRect = SkRect::MakeLTRB(midRect.left(), 0, midRect.right(), midRect.top());
- rightRect =
- SkRect::MakeLTRB(midRect.left(), midRect.bottom(), midRect.right(), dstRect.height());
- dstRect.fLeft = midRect.left();
- dstRect.fRight = midRect.right();
- }
- if (!topRect.isEmpty()) {
- drawContext->clear(&topRect, 0, false);
- }
-
- if (!bottomRect.isEmpty()) {
- drawContext->clear(&bottomRect, 0, false);
- }
- if (midRect.isEmpty()) {
- // Blur radius covers srcBounds; use bounds over entire draw
- convolve_gaussian_1d(drawContext, clip, dstRect, -srcOffset, texture,
+ bounds[0] = SkScalarToFloat(srcRect.top()) / texture->height();
+ bounds[1] = SkScalarToFloat(srcRect.bottom()) / texture->height();
+ size = dstRect.height();
+ lowerDstRect.fBottom = dstRect.top() + rad;
+ upperDstRect.fTop = dstRect.bottom() - rad;
+ middleDstRect.inset(0, rad);
+ }
+ if (radius >= size * SK_ScalarHalf) {
+ // Blur radius covers srcRect; use bounds over entire draw
+ convolve_gaussian_1d(drawContext, clip, dstRect, srcOffset, texture,
direction, radius, sigma, true, bounds);
} else {
- // Draw right and left margins with bounds; middle without.
- convolve_gaussian_1d(drawContext, clip, leftRect, -srcOffset, texture,
+ // Draw upper and lower margins with bounds; middle without.
+ convolve_gaussian_1d(drawContext, clip, lowerDstRect, srcOffset, texture,
direction, radius, sigma, true, bounds);
- convolve_gaussian_1d(drawContext, clip, rightRect, -srcOffset, texture,
+ convolve_gaussian_1d(drawContext, clip, upperDstRect, srcOffset, texture,
direction, radius, sigma, true, bounds);
- convolve_gaussian_1d(drawContext, clip, midRect, -srcOffset, texture,
+ convolve_gaussian_1d(drawContext, clip, middleDstRect, srcOffset, texture,
direction, radius, sigma, false, bounds);
}
}
@@ -160,12 +140,13 @@
GrTexture* GaussianBlur(GrContext* context,
GrTexture* srcTexture,
bool canClobberSrc,
- const SkRect& dstBounds,
- const SkRect* srcBounds,
+ const SkRect& rect,
+ bool cropToRect,
float sigmaX,
float sigmaY,
GrTextureProvider::SizeConstraint constraint) {
SkASSERT(context);
+
SkIRect clearRect;
int scaleFactorX, radiusX;
int scaleFactorY, radiusY;
@@ -173,25 +154,14 @@
sigmaX = adjust_sigma(sigmaX, maxTextureSize, &scaleFactorX, &radiusX);
sigmaY = adjust_sigma(sigmaY, maxTextureSize, &scaleFactorY, &radiusY);
- SkPoint srcOffset = SkPoint::Make(-dstBounds.x(), -dstBounds.y());
- SkRect localDstBounds = SkRect::MakeWH(dstBounds.width(), dstBounds.height());
- SkRect localSrcBounds;
- SkRect srcRect;
- if (srcBounds) {
- srcRect = localSrcBounds = *srcBounds;
- srcRect.offset(srcOffset);
- srcBounds = &localSrcBounds;
- } else {
- srcRect = localDstBounds;
- }
-
+ SkRect srcRect(rect);
scale_rect(&srcRect, 1.0f / scaleFactorX, 1.0f / scaleFactorY);
srcRect.roundOut(&srcRect);
scale_rect(&srcRect, static_cast<float>(scaleFactorX),
static_cast<float>(scaleFactorY));
// setup new clip
- GrClip clip(localDstBounds);
+ GrClip clip(SkRect::MakeWH(srcRect.width(), srcRect.height()));
SkASSERT(kBGRA_8888_GrPixelConfig == srcTexture->config() ||
kRGBA_8888_GrPixelConfig == srcTexture->config() ||
@@ -199,8 +169,8 @@
GrSurfaceDesc desc;
desc.fFlags = kRenderTarget_GrSurfaceFlag;
- desc.fWidth = SkScalarFloorToInt(dstBounds.width());
- desc.fHeight = SkScalarFloorToInt(dstBounds.height());
+ desc.fWidth = SkScalarFloorToInt(srcRect.width());
+ desc.fHeight = SkScalarFloorToInt(srcRect.height());
desc.fConfig = srcTexture->config();
GrTexture* dstTexture;
@@ -227,11 +197,12 @@
SkMatrix matrix;
matrix.setIDiv(srcTexture->width(), srcTexture->height());
SkRect dstRect(srcRect);
- if (srcBounds && i == 1) {
+ if (cropToRect && i == 1) {
+ dstRect.offset(-dstRect.fLeft, -dstRect.fTop);
SkRect domain;
- matrix.mapRect(&domain, *srcBounds);
- domain.inset((i < scaleFactorX) ? SK_ScalarHalf / srcTexture->width() : 0.0f,
- (i < scaleFactorY) ? SK_ScalarHalf / srcTexture->height() : 0.0f);
+ matrix.mapRect(&domain, rect);
+ domain.inset(i < scaleFactorX ? SK_ScalarHalf / srcTexture->width() : 0.0f,
+ i < scaleFactorY ? SK_ScalarHalf / srcTexture->height() : 0.0f);
SkAutoTUnref<const GrFragmentProcessor> fp(GrTextureDomainEffect::Create(
srcTexture,
matrix,
@@ -239,8 +210,6 @@
GrTextureDomain::kDecal_Mode,
GrTextureParams::kBilerp_FilterMode));
paint.addColorFragmentProcessor(fp);
- srcRect.offset(-srcOffset);
- srcOffset.set(0, 0);
} else {
GrTextureParams params(SkShader::kClamp_TileMode, GrTextureParams::kBilerp_FilterMode);
paint.addColorTextureProcessor(srcTexture, matrix, params);
@@ -259,8 +228,9 @@
srcRect = dstRect;
srcTexture = dstTexture;
SkTSwap(dstTexture, tempTexture);
- localSrcBounds = srcRect;
- }
+ }
+
+ const SkIRect srcIRect = srcRect.roundOut();
// For really small blurs (certainly no wider than 5x5 on desktop gpus) it is faster to just
// launch a single non separable kernel vs two launches
@@ -275,7 +245,7 @@
return nullptr;
}
convolve_gaussian_2d(dstDrawContext, clip, srcRect,
- srcTexture, radiusX, radiusY, sigmaX, sigmaY, srcBounds);
+ srcTexture, radiusX, radiusY, sigmaX, sigmaY, cropToRect, srcIRect);
srcDrawContext.swap(dstDrawContext);
srcRect.offsetTo(0, 0);
@@ -283,10 +253,6 @@
SkTSwap(dstTexture, tempTexture);
} else {
- srcRect = localDstBounds;
- scale_rect(&srcRect, 1.0f / scaleFactorX, 1.0f / scaleFactorY);
- srcRect.roundOut(&srcRect);
- const SkIRect srcIRect = srcRect.roundOut();
if (sigmaX > 0.0f) {
if (scaleFactorX > 1) {
// TODO: if we pass in the source draw context we don't need this here
@@ -311,13 +277,12 @@
}
convolve_gaussian(dstDrawContext, clip, srcRect,
srcTexture, Gr1DKernelEffect::kX_Direction, radiusX, sigmaX,
- srcBounds, srcOffset);
+ cropToRect);
+
srcDrawContext.swap(dstDrawContext);
srcTexture = dstTexture;
srcRect.offsetTo(0, 0);
SkTSwap(dstTexture, tempTexture);
- localSrcBounds = srcRect;
- srcOffset.set(0, 0);
}
if (sigmaY > 0.0f) {
@@ -344,7 +309,7 @@
}
convolve_gaussian(dstDrawContext, clip, srcRect,
srcTexture, Gr1DKernelEffect::kY_Direction, radiusY, sigmaY,
- srcBounds, srcOffset);
+ cropToRect);
srcDrawContext.swap(dstDrawContext);
srcTexture = dstTexture;
@@ -352,7 +317,6 @@
SkTSwap(dstTexture, tempTexture);
}
}
- const SkIRect srcIRect = srcRect.roundOut();
if (scaleFactorX > 1 || scaleFactorY > 1) {
SkASSERT(srcDrawContext);
« no previous file with comments | « src/effects/SkGpuBlurUtils.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698