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

Source/platform/graphics/filters/FilterEffect.h

Index: Source/platform/graphics/filters/FilterEffect.h
diff --git a/Source/platform/graphics/filters/FilterEffect.h b/Source/platform/graphics/filters/FilterEffect.h
index 8e4c351797b89e23242f9b4739f65183519eb87e..bd02fc01ca9aa81c5530bed974c54528ba486559 100644
--- a/Source/platform/graphics/filters/FilterEffect.h
+++ b/Source/platform/graphics/filters/FilterEffect.h
@@ -37,8 +37,6 @@
 #include "wtf/Uint8ClampedArray.h"
 #include "wtf/Vector.h"
 
-static const float kMaxFilterSize = 5000.0f;
-
 namespace WebCore {
 
 class Filter;
@@ -69,6 +67,10 @@ class PLATFORM_EXPORT FilterEffect : public RefCounted<FilterEffect> {
 public:
     virtual ~FilterEffect();
 
+    static bool isFilterSizeValid(IntRect);
+    static bool isFilterSizeValid(FloatRect);
+    static float maxFilterArea();
+
     void clearResult();
     void clearResultsRecursive();
 
@@ -178,8 +180,9 @@ public:
     virtual bool affectsTransparentPixels() { return false; }
 
 protected:
-    FilterEffect(Filter*);
+    static const float kMaxFilterArea;

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

Nit: Any reason to have this in class scope? (I.e., why not just leave as static
in the .cpp file)

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

In platform graphics it is typical to put these in the .cpp file of the class
that uses them, in file scope. So remove this declaration here and remove the
FilterEffect scope on the definition.

 
+    FilterEffect(Filter*);
     ImageBuffer* createImageBufferResult();
     Uint8ClampedArray* createUnmultipliedImageResult();
     Uint8ClampedArray* createPremultipliedImageResult();