Fix assert failure in AnimatableStrokeDasharrayList::equalTo()

Source/core/animation/AnimatableRepeatable.h

Index: Source/core/animation/AnimatableRepeatable.h
diff --git a/Source/core/animation/AnimatableRepeatable.h b/Source/core/animation/AnimatableRepeatable.h
index c0418073d2dc5cb9d848563e5c8217478124d83d..6d1747116666aac96d03d536a293179739755e93 100644
--- a/Source/core/animation/AnimatableRepeatable.h
+++ b/Source/core/animation/AnimatableRepeatable.h
@@ -74,7 +74,8 @@ private:
 
 inline const AnimatableRepeatable* toAnimatableRepeatable(const AnimatableValue* value)
 {
-    ASSERT_WITH_SECURITY_IMPLICATION(value && value->isRepeatable());
+    // FIXME: Use a better typing system for AnimatableValues that doesn't require ugliness like this to support subclassing.

[dstockwell, 2013/10/20 23:46:27]

Why don't we do that now?

(like the toAnimatableRepeatableOverride that we discussed)

[alancutter (OOO until 2018), 2013/10/20 23:53:12]

I still consider that in the same league of ugliness and doesn't fix the fixme
but that's just my personal view. I'm fine with either approach though.

+    ASSERT_WITH_SECURITY_IMPLICATION(value && (value->isRepeatable() || value->isStrokeDasharrayList()));
     return static_cast<const AnimatableRepeatable*>(value);
 }