SkAlphaThresholdFilter allow flattening

src/effects/SkAlphaThresholdFilter.cpp

Index: src/effects/SkAlphaThresholdFilter.cpp
diff --git a/src/effects/SkAlphaThresholdFilter.cpp b/src/effects/SkAlphaThresholdFilter.cpp
index 661d0d362ad2eed7ca3ce72b991d8f34d7ff6a19..31ca19025991ba4c4338a14ef0ae30ce8385e313 100644
--- a/src/effects/SkAlphaThresholdFilter.cpp
+++ b/src/effects/SkAlphaThresholdFilter.cpp
@@ -6,6 +6,7 @@
  */
 
 #include "SkAlphaThresholdFilter.h"
+#include "SkAlphaThresholdFilterImpl.h"
 #include "SkBitmap.h"
 #include "SkDevice.h"
 #include "SkReadBuffer.h"
@@ -15,31 +16,6 @@
 #include "GrDrawContext.h"
 #endif
 
-class SK_API SkAlphaThresholdFilterImpl : public SkImageFilter {

[mtklein, 2015/12/07 16:47:10]

Why can't this stay here in the .cpp file?  Seems easier to read as it is.

-public:
-    SkAlphaThresholdFilterImpl(const SkRegion& region, SkScalar innerThreshold,
-                               SkScalar outerThreshold, SkImageFilter* input);
-
-    SK_TO_STRING_OVERRIDE()
-    SK_DECLARE_PUBLIC_FLATTENABLE_DESERIALIZATION_PROCS(SkAlphaThresholdFilterImpl)
-
-protected:
-    void flatten(SkWriteBuffer&) const override;
-
-    bool onFilterImage(Proxy*, const SkBitmap& src, const Context&,
-                       SkBitmap* result, SkIPoint* offset) const override;
-#if SK_SUPPORT_GPU
-    bool asFragmentProcessor(GrFragmentProcessor**, GrTexture*, const SkMatrix&,
-                             const SkIRect& bounds) const override;
-#endif
-
-private:
-    SkRegion fRegion;
-    SkScalar fInnerThreshold;
-    SkScalar fOuterThreshold;
-    typedef SkImageFilter INHERITED;
-};
-
 SkImageFilter* SkAlphaThresholdFilter::Create(const SkRegion& region,
                                               SkScalar innerThreshold,
                                               SkScalar outerThreshold,