Cascade declared property values instead of applying values on top of each other

Source/core/css/CSSProperty.cpp

 /**
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2005, 2006 Apple Computer, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 655 matching lines @@
     case CSSPropertyInvalid:
         ASSERT_NOT_REACHED();
         return false;
     }
     ASSERT_NOT_REACHED();
     return false;
 }
 
 bool CSSProperty::isAffectedByAllProperty(CSSPropertyID propertyID)
 {
-    if (propertyID == CSSPropertyAll)
-        return false;
-
     // all shorthand spec says:
     // The all property is a shorthand that resets all CSS properties except
     // direction and unicode-bidi. It only accepts the CSS-wide keywords.
     // c.f. http://dev.w3.org/csswg/css-cascade/#all-shorthand
-    // So CSSPropertyUnicodeBidi and CSSPropertyDirection are not
-    // affected by all property.
-    return propertyID != CSSPropertyUnicodeBidi && propertyID != CSSPropertyDirection;
+    return propertyID != CSSPropertyAll
+        && propertyID != CSSPropertyUnicodeBidi
+        && propertyID != CSSPropertyDirection
+        && propertyID != CSSPropertyInternalCallback
+        && !isExpandedShorthandForAll(propertyID);
 }
 
 } // namespace blink