Fixes the rendering of a SVG filter (e.g. feFlood) by testing against total

Source/core/rendering/svg/RenderSVGResourceFilter.cpp

Index: Source/core/rendering/svg/RenderSVGResourceFilter.cpp
diff --git a/Source/core/rendering/svg/RenderSVGResourceFilter.cpp b/Source/core/rendering/svg/RenderSVGResourceFilter.cpp
index 1473c5f95392ca5e1cde99719bf8ebaf0536fd39..d6038848486b650aec993118d484a0a0a06055db 100644
--- a/Source/core/rendering/svg/RenderSVGResourceFilter.cpp
+++ b/Source/core/rendering/svg/RenderSVGResourceFilter.cpp
@@ -110,12 +110,12 @@ PassRefPtr<SVGFilterBuilder> RenderSVGResourceFilter::buildPrimitives(SVGFilter*
 bool RenderSVGResourceFilter::fitsInMaximumImageSize(const FloatSize& size, FloatSize& scale)

[Stephen Chennney, 2014/02/11 14:47:21]

The current implementation is wrong. It should compute the area, compare to the
max area, and reduce the scale by the sqrt of the ratio. (Preserve the aspect
ratio.)

 {
     bool matchesFilterSize = true;
-    if (size.width() * scale.width() > kMaxFilterSize) {
-        scale.setWidth(kMaxFilterSize / size.width());
+    if (size.width() * scale.width() > FilterEffect::maxFilterArea()) {

[fs, 2014/02/07 10:17:51]

This compares a distance to an area.

+        scale.setWidth(FilterEffect::maxFilterArea() / size.width());
         matchesFilterSize = false;
     }
-    if (size.height() * scale.height() > kMaxFilterSize) {
-        scale.setHeight(kMaxFilterSize / size.height());
+    if (size.height() * scale.height() > FilterEffect::maxFilterArea()) {

[fs, 2014/02/07 10:17:51]

Same as above.

+        scale.setHeight(FilterEffect::maxFilterArea() / size.height());
         matchesFilterSize = false;
     }