Add the 'lighter' composite operation to feComposite.

Source/core/svg/SVGFECompositeElement.cpp

Index: Source/core/svg/SVGFECompositeElement.cpp
diff --git a/Source/core/svg/SVGFECompositeElement.cpp b/Source/core/svg/SVGFECompositeElement.cpp
index 6b8a569c53a717adbd706536b9f02e9945277d0a..4eae0695f438d36ee202dd3ad9feb9735cd4ec23 100644
--- a/Source/core/svg/SVGFECompositeElement.cpp
+++ b/Source/core/svg/SVGFECompositeElement.cpp
@@ -38,10 +38,16 @@ template<> const SVGEnumerationStringEntries& getStaticStringEntries<CompositeOp
         entries.append(std::make_pair(FECOMPOSITE_OPERATOR_ATOP, "atop"));
         entries.append(std::make_pair(FECOMPOSITE_OPERATOR_XOR, "xor"));
         entries.append(std::make_pair(FECOMPOSITE_OPERATOR_ARITHMETIC, "arithmetic"));
+        entries.append(std::make_pair(FECOMPOSITE_OPERATOR_LIGHTER, "lighter"));
     }
     return entries;
 }
 
+template<> unsigned short getMaxExposedEnumValue<CompositeOperationType>()

[pdr., 2014/12/10 20:58:08]

Did you mean to leave this in the patch? This is deadcode.

[fs, 2014/12/11 09:30:20]

I don't think it's dead code - it's just really non-obvious code [1]. Might be
worth adding a test just to keep the assumption visible.

[1] SVGAnimatedEnumeration<CompositeOperationType>::create (m_svgOperator below)
which calls SVGEnumeration<Enum>::create, which in turn calls the constructor,
which calls this method (and the one above it.) Don't you just love templates?
=)
I have some ideas for how to make this more obvious, but still need to put those
to the test...

+{
+    return FECOMPOSITE_OPERATOR_ARITHMETIC;
+}
+
 inline SVGFECompositeElement::SVGFECompositeElement(Document& document)
     : SVGFilterPrimitiveStandardAttributes(SVGNames::feCompositeTag, document)
     , m_k1(SVGAnimatedNumber::create(this, SVGNames::k1Attr, SVGNumber::create()))