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

Unified Diff: Source/platform/graphics/filters/FEGaussianBlur.cpp

Issue 793873002: Don't cap the FEGaussian max kernel size. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/platform/graphics/filters/FEGaussianBlur.cpp
diff --git a/Source/platform/graphics/filters/FEGaussianBlur.cpp b/Source/platform/graphics/filters/FEGaussianBlur.cpp
index f82096a47c9ac8a0f638f7937b1e2a9a0fc4a26d..d0482925d1d0b2cf77e80a66ef42d2e8108fca71 100644
--- a/Source/platform/graphics/filters/FEGaussianBlur.cpp
+++ b/Source/platform/graphics/filters/FEGaussianBlur.cpp
@@ -38,8 +38,6 @@ static inline float gaussianKernelFactor()
return 3 / 4.f * sqrtf(twoPiFloat);
}
-static const int gMaxKernelSize = 1000;
-
namespace blink {
FEGaussianBlur::FEGaussianBlur(Filter* filter, float x, float y)
@@ -83,12 +81,12 @@ IntSize FEGaussianBlur::calculateUnscaledKernelSize(const FloatPoint& std)
// inflates the absolute paint rect to much. This is compatible with Firefox' behavior.
if (std.x()) {
int size = std::max<unsigned>(2, static_cast<unsigned>(floorf(std.x() * gaussianKernelFactor() + 0.5f)));
- kernelSize.setWidth(std::min(size, gMaxKernelSize));
+ kernelSize.setWidth(size);
}
if (std.y()) {
int size = std::max<unsigned>(2, static_cast<unsigned>(floorf(std.y() * gaussianKernelFactor() + 0.5f)));
- kernelSize.setHeight(std::min(size, gMaxKernelSize));
+ kernelSize.setHeight(size);
}
return kernelSize;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698