Update animation when changes in animation keyframes are detected.

Source/core/css/CSSKeyframeRule.cpp

 /*
  * Copyright (C) 2007, 2008, 2012 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 10 matching lines @@
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "core/css/CSSKeyframeRule.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "core/css/CSSKeyframesRule.h"
+#include "core/css/CSSStyleSheet.h"
 #include "core/css/PropertySetCSSStyleDeclaration.h"
 #include "core/css/StylePropertySet.h"
 #include "core/css/parser/CSSParser.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/frame/UseCounter.h"
 #include "wtf/text/StringBuilder.h"
 
 namespace blink {
 
 StyleKeyframe::StyleKeyframe()
@@ skipping 104 matching lines @@
 #if !ENABLE(OILPAN)
     if (m_propertiesCSSOMWrapper)
         m_propertiesCSSOMWrapper->clearParentRule();
 #endif
 }
 
 void CSSKeyframeRule::setKeyText(const String& keyText, ExceptionState& exceptionState)
 {
     if (!m_keyframe->setKeyText(keyText))
         exceptionState.throwDOMException(SyntaxError, "The key '" + keyText + "' is invalid and cannot be parsed");
+
+    CSSStyleSheet::RuleMutationScope mutationScope(this);

[dstockwell, 2015/01/07 00:18:33]

Is this needed? If it is, appears as though it should be called before the
mutation happens on line 153.

[shend, 2015/01/08 05:05:29]

Oops, you're right.

+
+    CSSKeyframesRule* parent = toCSSKeyframesRule(parentRule());
+    parent->m_keyframesRule->styleChanged();

[shend, 2015/01/06 00:36:18]

Should we move this logic into StyleKeyframe so that CSSKeyframeRule doesn't
know anything about style counters? If so, is it okay to keep a pointer in
StyleKeyframe to the parent StyleKeyframes?

 }
 
 CSSStyleDeclaration* CSSKeyframeRule::style() const
 {
     if (!m_propertiesCSSOMWrapper)
         m_propertiesCSSOMWrapper = StyleRuleCSSStyleDeclaration::create(m_keyframe->mutableProperties(), const_cast<CSSKeyframeRule*>(this));
     return m_propertiesCSSOMWrapper.get();
 }
 
 void CSSKeyframeRule::reattach(StyleRuleBase*)
 {
     // No need to reattach, the underlying data is shareable on mutation.
     ASSERT_NOT_REACHED();
 }
 
 void CSSKeyframeRule::trace(Visitor* visitor)
 {
     visitor->trace(m_keyframe);
     visitor->trace(m_propertiesCSSOMWrapper);
     CSSRule::trace(visitor);
 }
 
 } // namespace blink