Web Animations CSS: Snapshot missing start/end Keyframes and remove duplicates

Source/core/animation/css/CSSAnimations.cpp

 /*
  * Copyright (C) 2013 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 164 matching lines @@
                 existing->value->setPaused(paused);
                 inactive.remove(animationName.impl());
                 continue;
             }
 
             KeyframeAnimationEffect::KeyframeVector keyframes;
             element->document()->styleResolver()->resolveKeyframes(element, style, animationName.impl(), keyframes);
             if (!keyframes.isEmpty()) {
                 Timing timing;
                 timingFromAnimationData(animationData, timing);
+                // FIXME: Keyframes are already normalized, perhaps there should be a flag on KeyframeAnimationEffect to skip normalization.

[Steve Block, 2013/08/06 06:30:20]

Sounds reasonable. Filed crbug.com/268791

                 m_animations.set(animationName.impl(), element->document()->timeline()->play(
                     Animation::create(element, KeyframeAnimationEffect::create(keyframes), timing).get()).get());
             }
         }
     }
 
     for (HashSet<StringImpl*>::const_iterator iter = inactive.begin(); iter != inactive.end(); ++iter)
         m_animations.take(*iter)->cancel();
 }
 
 void CSSAnimations::cancel()
 {
     for (AnimationMap::iterator iter = m_animations.begin(); iter != m_animations.end(); ++iter)
         iter->value->cancel();
 
     m_animations.clear();
 }
 
 } // namespace WebCore