[SPInvalidation] Update paint properties when SVG resource change

third_party/WebKit/Source/core/paint/PaintLayerResourceInfo.cpp

 /*
  * Copyright (C) 2012 Adobe Systems Incorporated. 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 33 matching lines @@
 TreeScope* PaintLayerResourceInfo::treeScope() {
   DCHECK(m_layer);
   Node* node = m_layer->layoutObject()->node();
   return node ? &node->treeScope() : nullptr;
 }
 
 void PaintLayerResourceInfo::resourceContentChanged() {
   DCHECK(m_layer);
   LayoutObject* layoutObject = m_layer->layoutObject();
   layoutObject->setShouldDoFullPaintInvalidation();
+  layoutObject->setNeedsPaintPropertyUpdate();

[pdr., 2016/12/16 21:13:37]

Can you add a comment here and below:
// The effect paint property nodes depend on SVG filters so we need
// to update these properties when filter resources change.

[Xianzhu, 2016/12/16 23:01:43]

Done.

   const ComputedStyle& style = layoutObject->styleRef();
   if (style.hasFilter() && style.filter().hasReferenceFilter())
     invalidateFilterChain();
 }
 
 void PaintLayerResourceInfo::resourceElementChanged() {
   DCHECK(m_layer);
   LayoutObject* layoutObject = m_layer->layoutObject();
   layoutObject->setShouldDoFullPaintInvalidation();
+  layoutObject->setNeedsPaintPropertyUpdate();
   const ComputedStyle& style = layoutObject->styleRef();
   if (style.hasFilter() && style.filter().hasReferenceFilter())
     invalidateFilterChain();
 }
 
 void PaintLayerResourceInfo::setLastEffect(FilterEffect* lastEffect) {
   m_lastEffect = lastEffect;
 }
 
 FilterEffect* PaintLayerResourceInfo::lastEffect() const {
   return m_lastEffect;
 }
 
 void PaintLayerResourceInfo::invalidateFilterChain() {
   m_lastEffect = nullptr;
 }
 
 DEFINE_TRACE(PaintLayerResourceInfo) {
   visitor->trace(m_lastEffect);
   SVGResourceClient::trace(visitor);
 }
 
 }  // namespace blink