Unprefix the CSS 'filter' property

third_party/WebKit/Source/core/layout/svg/LayoutSVGResourceContainer.cpp

 /*
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  *
  * 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,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 227 matching lines @@
         extensions.clearHasPendingResourcesIfPossible(pendingClient);
         LayoutObject* layoutObject = pendingClient->layoutObject();
         if (!layoutObject)
             continue;
 
         const ComputedStyle& style = layoutObject->styleRef();
 
         // If the client has a layer (is a non-SVGElement) we need to signal
         // invalidation in the same way as is done in markAllResourceClientsForInvalidation above.
         if (layoutObject->hasLayer() && resourceType() == FilterResourceType) {
-            if (style.hasFilter())
-                toLayoutBoxModelObject(layoutObject)->layer()->filterNeedsPaintInvalidation();
-            // If this is the SVG root, we could have both 'filter' and
-            // '-webkit-filter' applied, so we need to do the invalidation
-            // below as well, unless we can optimistically determine that
-            // 'filter' does not apply to the element in question.
-            if (!layoutObject->isSVGRoot() || !style.svgStyle().hasFilter())
+            if (!style.hasFilter())
                 continue;
+            toLayoutBoxModelObject(layoutObject)->layer()->filterNeedsPaintInvalidation();
+            if (!layoutObject->isSVGRoot())

[chrishtr, 2016/06/16 04:55:21]

DCHECK(layoutObject->isSVGRoot())

[fs, 2016/06/16 23:06:41]

|layoutObject| can be a non-SVG element here. Or did you mean after the
continue? 

[chrishtr, 2016/06/17 06:33:45]

Right, ok. Ignore my comment then.

+                continue;
+            // A root SVG element with a filter, however, still needs to run
+            // the full invalidation step below.
         }
 
         StyleDifference diff;
         diff.setNeedsFullLayout();
         SVGResourcesCache::clientStyleChanged(layoutObject, diff, style);
         layoutObject->setNeedsLayoutAndFullPaintInvalidation(LayoutInvalidationReason::SvgResourceInvalidated);
     }
 }
 
 static inline void removeFromCacheAndInvalidateDependencies(LayoutObject* object, bool needsLayout)
@@ skipping 55 matching lines @@
             // This will process the rest of the ancestors.
             toLayoutSVGResourceContainer(current)->removeAllClientsFromCache();
             break;
         }
 
         current = current->parent();
     }
 }
 
 } // namespace blink