Fix up that no render SVG shape when change clip-path to visible after hidden.

third_party/WebKit/Source/core/layout/svg/SVGResourcesCache.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 83 matching lines @@
   if (object->selfNeedsLayout() || resources->filter())
     resources->removeClientFromCache(object);
 }
 
 static inline bool layoutObjectCanHaveResources(LayoutObject* layoutObject) {
   ASSERT(layoutObject);
   return layoutObject->node() && layoutObject->node()->isSVGElement() &&
          !layoutObject->isSVGInlineText();
 }
 
+static inline bool isLayoutObjectOfResourceContainer(LayoutObject* layoutObject) {
+  LayoutObject* current = layoutObject;
+  while (current) {
+    if (current->isSVGResourceContainer()) {
+      return true;
+    }
+    current = current->parent();
+  }
+  return false;
+}
+
 void SVGResourcesCache::clientStyleChanged(LayoutObject* layoutObject,
                                            StyleDifference diff,
                                            const ComputedStyle& newStyle) {
   ASSERT(layoutObject);
   ASSERT(layoutObject->node());
   ASSERT(layoutObject->node()->isSVGElement());
 
   if (!diff.hasDifference() || !layoutObject->parent())
     return;
 
   // In this case the proper SVGFE*Element will decide whether the modified CSS
   // properties require
   // a relayout or paintInvalidation.
   if (layoutObject->isSVGResourceFilterPrimitive() && !diff.needsLayout())
     return;
 
   // Dynamic changes of CSS properties like 'clip-path' may require us to
   // recompute the associated resources for a LayoutObject.
   // TODO(fs): Avoid passing in a useless StyleDifference, but instead compare
   // oldStyle/newStyle to see which resources changed to be able to selectively
   // rebuild individual resources, instead of all of them.
   if (layoutObjectCanHaveResources(layoutObject)) {
     SVGResourcesCache& cache = resourcesCache(layoutObject->document());
     cache.removeResourcesFromLayoutObject(layoutObject);
     cache.addResourcesFromLayoutObject(layoutObject, newStyle);
   }
 
+  bool needsLayout = diff.needsPaintInvalidation() &&

[fs, 2017/02/20 09:07:43]

It would be good to add a comment here to describe what we're trying to catch
here by doing this.

[hyunjunekim2, 2017/03/13 10:40:11]

Done. Added a comment. Could you check it? Thank you.

+                     isLayoutObjectOfResourceContainer(layoutObject);
+
   LayoutSVGResourceContainer::markForLayoutAndParentResourceInvalidation(
-      layoutObject, false);
+      layoutObject, needsLayout);
 }
 
 void SVGResourcesCache::clientWasAddedToTree(LayoutObject* layoutObject,
                                              const ComputedStyle& newStyle) {
   if (!layoutObject->node())
     return;
   LayoutSVGResourceContainer::markForLayoutAndParentResourceInvalidation(
       layoutObject, false);
 
   if (!layoutObjectCanHaveResources(layoutObject))
@@ skipping 19 matching lines @@
   ASSERT(layoutObject);
 
   SVGResources* resources = cachedResourcesForLayoutObject(layoutObject);
   if (resources)
     resources->removeClientFromCache(layoutObject);
   SVGResourcesCache& cache = resourcesCache(layoutObject->document());
   cache.removeResourcesFromLayoutObject(layoutObject);
 }
 
 }  // namespace blink