Applying the "all" property needs to check all properties against the whitelist.

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/)
  * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. All rights reserved.
@@ skipping 1192 matching lines @@
     // Properties that we currently support outside of spec.
     case CSSPropertyWebkitLineBoxContain:
     case CSSPropertyVisibility:
         return true;
 
     default:
         return false;
     }
 }
 
-static bool shouldIgnoreTextTrackAuthorStyle(Document& document)
+static bool shouldIgnoreTextTrackAuthorStyle(const Document& document)
 {
     Settings* settings = document.settings();
     if (!settings)
         return false;
     // Ignore author specified settings for text tracks when any of the user settings are present.
     if (!settings->textTrackBackgroundColor().isEmpty()
         || !settings->textTrackFontFamily().isEmpty()
         || !settings->textTrackFontStyle().isEmpty()
         || !settings->textTrackFontVariant().isEmpty()
         || !settings->textTrackTextColor().isEmpty()
         || !settings->textTrackTextShadow().isEmpty()
         || !settings->textTrackTextSize().isEmpty())
         return true;
     return false;
 }
 
+static inline bool isPropertyInWhitelist(PropertyWhitelistType propertyWhitelistType, CSSPropertyID property, const Document& document)
+{
+    if (propertyWhitelistType == PropertyWhitelistNone)
+        return true; // Early bail for the by far most common case.
+
+    if (propertyWhitelistType == PropertyWhitelistFirstLetter)
+        return isValidFirstLetterStyleProperty(property);
+
+    if (propertyWhitelistType == PropertyWhitelistCue)
+        return isValidCueStyleProperty(property) && !shouldIgnoreTextTrackAuthorStyle(document);
+
+    ASSERT_NOT_REACHED();
+    return true;
+}
+
 // This method expands the 'all' shorthand property to longhand properties
 // and applies the expanded longhand properties.
 template <CSSPropertyPriority priority>
-void StyleResolver::applyAllProperty(StyleResolverState& state, CSSValue* allValue, bool inheritedOnly)
+void StyleResolver::applyAllProperty(StyleResolverState& state, CSSValue* allValue, bool inheritedOnly, PropertyWhitelistType propertyWhitelistType)
 {
     unsigned startCSSProperty = CSSPropertyPriorityData<priority>::first();
     unsigned endCSSProperty = CSSPropertyPriorityData<priority>::last();
 
     for (unsigned i = startCSSProperty; i <= endCSSProperty; ++i) {
         CSSPropertyID propertyId = static_cast<CSSPropertyID>(i);
 
         // StyleBuilder does not allow any expanded shorthands.
         if (isShorthandProperty(propertyId))
             continue;
 
         // all shorthand spec says:
         // The all property is a shorthand that resets all CSS properties
         // except direction and unicode-bidi.
         // c.f. http://dev.w3.org/csswg/css-cascade/#all-shorthand
         // We skip applyProperty when a given property is unicode-bidi or
         // direction.
         if (!CSSProperty::isAffectedByAllProperty(propertyId))
             continue;
 
+        if (!isPropertyInWhitelist(propertyWhitelistType, propertyId, document()))
+            continue;
+
         // When hitting matched properties' cache, only inherited properties will be
         // applied.
         if (inheritedOnly && !CSSPropertyMetadata::isInheritedProperty(propertyId))
             continue;
 
         StyleBuilder::applyProperty(propertyId, state, allValue);
     }
 }
 
 template <CSSPropertyPriority priority>
 void StyleResolver::applyProperties(StyleResolverState& state, const StylePropertySet* properties, bool isImportant, bool inheritedOnly, PropertyWhitelistType propertyWhitelistType)
 {
     unsigned propertyCount = properties->propertyCount();
     for (unsigned i = 0; i < propertyCount; ++i) {
         StylePropertySet::PropertyReference current = properties->propertyAt(i);
         if (isImportant != current.isImportant())
             continue;
 
         CSSPropertyID property = current.id();
         if (property == CSSPropertyAll) {
-            applyAllProperty<priority>(state, current.value(), inheritedOnly);
+            applyAllProperty<priority>(state, current.value(), inheritedOnly, propertyWhitelistType);
             continue;
         }
 
-        if (propertyWhitelistType == PropertyWhitelistCue && (!isValidCueStyleProperty(property) || shouldIgnoreTextTrackAuthorStyle(document())))
-            continue;
-
-        if (propertyWhitelistType == PropertyWhitelistFirstLetter && !isValidFirstLetterStyleProperty(property))
+        if (!isPropertyInWhitelist(propertyWhitelistType, property, document()))
             continue;
 
         if (inheritedOnly && !current.isInherited()) {
             // If the property value is explicitly inherited, we need to apply further non-inherited properties
             // as they might override the value inherited here. For this reason we don't allow declarations with
             // explicitly inherited properties to be cached.
             ASSERT(!current.value()->isInheritedValue());
             continue;
         }
 
@@ skipping 216 matching lines @@
     visitor->trace(m_watchedSelectorsRules);
     visitor->trace(m_treeBoundaryCrossingScopes);
     visitor->trace(m_styleResourceLoader);
     visitor->trace(m_styleSharingLists);
     visitor->trace(m_pendingStyleSheets);
     visitor->trace(m_document);
 #endif
 }
 
 } // namespace blink