[SVG] SVGAnimatedBoolean migration to new SVG property impl.

Source/core/svg/SVGFEConvolveMatrixElement.h

Index: Source/core/svg/SVGFEConvolveMatrixElement.h
diff --git a/Source/core/svg/SVGFEConvolveMatrixElement.h b/Source/core/svg/SVGFEConvolveMatrixElement.h
index 4bc21ab5fc47f7cf97e3e59a2e761787028bd71f..391b57222d4bf947dc55373f8c2eef0caccc1695 100644
--- a/Source/core/svg/SVGFEConvolveMatrixElement.h
+++ b/Source/core/svg/SVGFEConvolveMatrixElement.h
@@ -70,6 +70,8 @@ public:
     void setOrder(float orderX, float orderY);
     void setKernelUnitLength(float kernelUnitLengthX, float kernelUnitLengthY);
 
+    SVGAnimatedBoolean* preserveAlpha() const { return m_preserveAlpha.get(); }

[pdr., 2014/01/10 18:57:32]

Can we refptr-ize this return?

[haraken, 2014/01/14 03:32:47]

I don't have a strong opinion here, but as far as I see other code in Blink, we
return raw pointers from getters when it is clear that raw pointers are OK.

+
 private:
     explicit SVGFEConvolveMatrixElement(Document&);
 
@@ -84,6 +86,7 @@ private:
     static const AtomicString& kernelUnitLengthXIdentifier();
     static const AtomicString& kernelUnitLengthYIdentifier();
 
+    RefPtr<SVGAnimatedBoolean> m_preserveAlpha;
     BEGIN_DECLARE_ANIMATED_PROPERTIES(SVGFEConvolveMatrixElement)
         DECLARE_ANIMATED_STRING(In1, in1)
         DECLARE_ANIMATED_INTEGER(OrderX, orderX)
@@ -96,7 +99,6 @@ private:
         DECLARE_ANIMATED_ENUMERATION(EdgeMode, edgeMode, EdgeModeType)
         DECLARE_ANIMATED_NUMBER(KernelUnitLengthX, kernelUnitLengthX)
         DECLARE_ANIMATED_NUMBER(KernelUnitLengthY, kernelUnitLengthY)
-        DECLARE_ANIMATED_BOOLEAN(PreserveAlpha, preserveAlpha)
     END_DECLARE_ANIMATED_PROPERTIES
 };