Add CSS property priority for CSS animation affecting properties

third_party/WebKit/Source/core/css/resolver/StyleResolver.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc.
  * All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
@@ skipping 1735 matching lines @@
     if (cacheCustomPropertiesForApplyAtRules(state,
                                              matchResult.authorRules())) {
       applyMatchedProperties<ResolveVariables>(state, matchResult.authorRules(),
                                                false, applyInheritedOnly);
       applyMatchedProperties<ResolveVariables>(state, matchResult.authorRules(),
                                                true, applyInheritedOnly);
       CSSVariableResolver::resolveVariableDefinitions(state);
     }
   }
 
+  // Apply animation properties.
+  applyMatchedProperties<AnimationPropertyPriority>(
+      state, matchResult.allRules(), false, applyInheritedOnly);

[Timothy Loh, 2016/10/21 05:42:55]

Don't we need to call this twice, once for not important and once for important?

[alancutter (OOO until 2018), 2016/10/24 04:08:27]

Oops, yes.

+
   // Now we have all of the matched rules in the appropriate order. Walk the
   // rules and apply high-priority properties first, i.e., those properties that
   // other properties depend on.  The order is (1) high-priority not important,
   // (2) high-priority important, (3) normal not important and (4) normal
   // important.
   applyMatchedProperties<HighPropertyPriority>(state, matchResult.allRules(),
                                                false, applyInheritedOnly);
   for (auto range : ImportantAuthorRanges(matchResult))
     applyMatchedProperties<HighPropertyPriority>(state, range, true,
                                                  applyInheritedOnly);
@@ skipping 178 matching lines @@
   visitor->trace(m_siblingRuleSet);
   visitor->trace(m_uncommonAttributeRuleSet);
   visitor->trace(m_watchedSelectorsRules);
   visitor->trace(m_treeBoundaryCrossingScopes);
   visitor->trace(m_styleSharingLists);
   visitor->trace(m_pendingStyleSheets);
   visitor->trace(m_document);
 }
 
 }  // namespace blink