Push hasValidAttributeName/Type down into SVGAnimateElement

third_party/WebKit/Source/core/svg/animation/SMILTimeContainer.cpp

 /*
  * Copyright (C) 2008 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 53 matching lines @@
   cancelAnimationPolicyTimer();
   ASSERT(!m_wakeupTimer.isActive());
 #if ENABLE(ASSERT)
   ASSERT(!m_preventScheduledAnimationsChanges);
 #endif
 }
 
 void SMILTimeContainer::schedule(SVGSMILElement* animation,
                                  SVGElement* target,
                                  const QualifiedName& attributeName) {
-  ASSERT(animation->timeContainer() == this);
-  ASSERT(target);
-  ASSERT(animation->hasValidAttributeName());
-  ASSERT(animation->hasValidAttributeType());
-  ASSERT(animation->inActiveDocument());
+  DCHECK_EQ(animation->timeContainer(), this);
+  DCHECK(target);
+  DCHECK(animation->hasValidTarget());
 
 #if ENABLE(ASSERT)
   ASSERT(!m_preventScheduledAnimationsChanges);
 #endif
 
   ElementAttributePair key(target, attributeName);
   Member<AnimationsLinkedHashSet>& scheduled =
       m_scheduledAnimations.add(key, nullptr).storedValue->value;
   if (!scheduled)
     scheduled = new AnimationsLinkedHashSet;
@@ skipping 362 matching lines @@
     // priority.  In case of a tie, document order decides.
     // FIXME: This should also consider timing relationships between the
     // elements. Dependents have higher priority.
     copyToVector(*entry.value, scheduledAnimationsInSameGroup);
     std::sort(scheduledAnimationsInSameGroup.begin(),
               scheduledAnimationsInSameGroup.end(), PriorityCompare(elapsed));
 
     AnimationsVector sandwich;
     for (const auto& itAnimation : scheduledAnimationsInSameGroup) {
       SVGSMILElement* animation = itAnimation.get();
-      ASSERT(animation->timeContainer() == this);
-      ASSERT(animation->targetElement());
-      ASSERT(animation->hasValidAttributeName());
-      ASSERT(animation->hasValidAttributeType());
+      DCHECK_EQ(animation->timeContainer(), this);
+      DCHECK(animation->hasValidTarget());
 
       // This will calculate the contribution from the animation and update
       // timing.
       if (animation->progress(elapsed, seekToTime)) {
         sandwich.append(animation);
       } else {
         animation->clearAnimatedType();
       }
 
       SMILTime nextFireTime = animation->nextProgressTime();
@@ skipping 60 matching lines @@
 void SMILTimeContainer::advanceFrameForTesting() {
   setElapsed(elapsed() + initialFrameDelay);
 }
 
 DEFINE_TRACE(SMILTimeContainer) {
   visitor->trace(m_scheduledAnimations);
   visitor->trace(m_ownerSVGElement);
 }
 
 }  // namespace blink