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

Unified Diff: src/core/SkMaskFilter.cpp

Issue 844673002: Revert of Cache blur mask for rects which can not break into nine-patch (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 11 months 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/core/SkMaskCache.cpp ('k') | src/effects/SkBlurMaskFilter.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/core/SkMaskFilter.cpp
diff --git a/src/core/SkMaskFilter.cpp b/src/core/SkMaskFilter.cpp
index 3a4e3a41e9075d5838fb87edb14478fc45d92c0d..c9783e76d6113e7bd9abb9a800341a1d83591c42 100644
--- a/src/core/SkMaskFilter.cpp
+++ b/src/core/SkMaskFilter.cpp
@@ -10,7 +10,6 @@
#include "SkMaskFilter.h"
#include "SkBlitter.h"
#include "SkDraw.h"
-#include "SkMaskCache.h"
#include "SkRasterClip.h"
#include "SkRRect.h"
#include "SkTypes.h"
@@ -264,41 +263,9 @@
}
SkAutoMaskFreeImage autoSrc(srcM.fImage);
- BlurRec rec;
- if (this->asABlur(&rec) && rectCount) {
- SkScalar scaledSigma = matrix.mapRadius(rec.fSigma);
- if (!SkMaskCache::FindAndCopy(scaledSigma, rec.fStyle, rec.fQuality,
- rects, rectCount, &dstM)) {
- if (!this->filterMask(&dstM, srcM, matrix, NULL)) {
- return false;
- }
- SkMaskCache::AddAndCopy(scaledSigma, rec.fStyle, rec.fQuality, rects, rectCount, dstM);
- } else {
- // Compute the correct bounds of dst mask if dst mask is got from cache.
- SkMask tmpSrc, tmpDst;
- tmpSrc = srcM;
- tmpSrc.fImage = NULL;
- if (!this->filterMask(&tmpDst, tmpSrc, matrix, NULL)) {
- return false;
- }
-
- // Fallback to original calculation if size of bounds is different with
- // size of the cached mask.
- if (dstM.fBounds.width() != tmpDst.fBounds.width() ||
- dstM.fBounds.height() != tmpDst.fBounds.height()) {
- if (!this->filterMask(&dstM, srcM, matrix, NULL)) {
- return false;
- }
- } else {
- dstM.fBounds = tmpDst.fBounds;
- }
- }
- } else {
- if (!this->filterMask(&dstM, srcM, matrix, NULL)) {
- return false;
- }
- }
-
+ if (!this->filterMask(&dstM, srcM, matrix, NULL)) {
+ return false;
+ }
SkAutoMaskFreeImage autoDst(dstM.fImage);
// if we get here, we need to (possibly) resolve the clip and blitter
« no previous file with comments | « src/core/SkMaskCache.cpp ('k') | src/effects/SkBlurMaskFilter.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698