Update animation when changes in animation keyframes are detected.

Source/core/css/resolver/ScopedStyleResolver.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2012 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:
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 72 matching lines @@
             features.add(contents->ruleSet().features());
     }
 }
 
 void ScopedStyleResolver::resetAuthorStyle()
 {
     m_authorStyleSheets.clear();
     m_keyframesRuleMap.clear();
 }
 
-const StyleRuleKeyframes* ScopedStyleResolver::keyframeStylesForAnimation(const StringImpl* animationName)
+const StyleRuleKeyframes* ScopedStyleResolver::keyframeStylesForAnimation(const StringImpl* animationName) const
 {
     if (m_keyframesRuleMap.isEmpty())
         return 0;
 
-    KeyframesRuleMap::iterator it = m_keyframesRuleMap.find(animationName);
+    KeyframesRuleMap::const_iterator it = m_keyframesRuleMap.find(animationName);
     if (it == m_keyframesRuleMap.end())
         return 0;
 
     return it->value.get();
 }
 
+PassRefPtrWillBeRawPtr<StyleRuleKeyframes> ScopedStyleResolver::keyframeStylesForAnimation(const StringImpl* animationName)
+{
+    if (m_keyframesRuleMap.isEmpty())
+        return nullptr;
+
+    KeyframesRuleMap::iterator it = m_keyframesRuleMap.find(animationName);
+    if (it == m_keyframesRuleMap.end())
+        return nullptr;
+
+    return it->value;
+}
+
 void ScopedStyleResolver::addKeyframeStyle(PassRefPtrWillBeRawPtr<StyleRuleKeyframes> rule)
 {
     AtomicString s(rule->name());
+
+    rule->styleChanged();

[dstockwell, 2015/01/05 07:31:18]

I think on Friday we discussed that we need to call this in one more place: when
we're mutating a keyframe already added to a keyframes rule.

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

Modifying the style of an existing keyframe seems to be handled by this (see
test), but setting the key text has to be separately handled.

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

Hmm, but how does that work?

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

I'm not exactly sure, but if you follow the call tree back up, it looks like
this method is called every time a new StyleResolver is created
(StyleEngine::ensureResolver), and a new StyleResolver seems to be created every
time I change the style of a keyframe. Is this the expected behaviour?

[dstockwell, 2015/01/08 05:27:24]

Ahh, I see! I think I misunderstood the purpose of this method. I think this
means that any time the style sheet is mutated (and the style resolver cleared)
we will bump the counts on all keyframes. @shans, wdyt?

+
     if (rule->isVendorPrefixed()) {
-        KeyframesRuleMap::iterator it = m_keyframesRuleMap.find(rule->name().impl());
+        KeyframesRuleMap::iterator it = m_keyframesRuleMap.find(s.impl());
         if (it == m_keyframesRuleMap.end())
             m_keyframesRuleMap.set(s.impl(), rule);
         else if (it->value->isVendorPrefixed())
             m_keyframesRuleMap.set(s.impl(), rule);
     } else {
         m_keyframesRuleMap.set(s.impl(), rule);
     }
 }
 
 void ScopedStyleResolver::collectMatchingAuthorRules(ElementRuleCollector& collector, bool includeEmptyRules, CascadeScope cascadeScope, CascadeOrder cascadeOrder)
@@ skipping 39 matching lines @@
 void ScopedStyleResolver::trace(Visitor* visitor)
 {
 #if ENABLE(OILPAN)
     visitor->trace(m_scope);
     visitor->trace(m_authorStyleSheets);
     visitor->trace(m_keyframesRuleMap);
 #endif
 }
 
 } // namespace blink