[SVG] Remove existing event listeners when eventBase conditions are cleared

Source/core/svg/animation/SVGSMILElement.cpp

Index: Source/core/svg/animation/SVGSMILElement.cpp
diff --git a/Source/core/svg/animation/SVGSMILElement.cpp b/Source/core/svg/animation/SVGSMILElement.cpp
index cb08bd6bbbf467d531b0aa159e00a935a3284104..86ade8d0d271629fd81b27f4e30284e03fcbcfcb 100644
--- a/Source/core/svg/animation/SVGSMILElement.cpp
+++ b/Source/core/svg/animation/SVGSMILElement.cpp
@@ -506,6 +506,7 @@ void SVGSMILElement::parseAttribute(const QualifiedName& name, const AtomicStrin
     if (name == SVGNames::beginAttr) {
         if (!m_conditions.isEmpty()) {
             disconnectSyncBaseConditions();
+            disconnectEventBaseConditions();
             m_conditions.clear();

[f(malita), 2014/01/27 15:35:33]

Looks like m_conditions.clear() is always accompanied by
disconnectEventBaseConditions() & disconnectSyncBaseConditions() calls. How
about wrapping these 3 into a helper (clearConditions or such)?

[pavane, 2014/01/27 17:01:19]

Done.

             parseBeginOrEnd(fastGetAttribute(SVGNames::endAttr), End);
         }
@@ -515,6 +516,7 @@ void SVGSMILElement::parseAttribute(const QualifiedName& name, const AtomicStrin
     } else if (name == SVGNames::endAttr) {
         if (!m_conditions.isEmpty()) {
             disconnectSyncBaseConditions();
+            disconnectEventBaseConditions();
             m_conditions.clear();
             parseBeginOrEnd(fastGetAttribute(SVGNames::beginAttr), Begin);
         }