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
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #include "core/layout/svg/SVGResourcesCache.h"
 
 #include "core/HTMLNames.h"
+#include "core/dom/ElementTraversal.h"
 #include "core/layout/svg/LayoutSVGResourceContainer.h"
 #include "core/layout/svg/SVGResources.h"
 #include "core/layout/svg/SVGResourcesCycleSolver.h"
 #include "core/svg/SVGDocumentExtensions.h"
 #include <memory>
 
 namespace blink {
 
 SVGResourcesCache::SVGResourcesCache() {}
 
@@ skipping 77 matching lines @@
 
   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;
 
+  LayoutObject* current = layoutObject->parent();
+  while (current) {
+    if (current->isSVGResourceContainer()) {
+      if (newStyle.visibility() == EVisibility::kVisible) {
+        LayoutSVGResourceContainer* container =
+          toLayoutSVGResourceContainer(current);
+        for (auto client : container->clients()) {
+          if (client) {
+            LayoutRect boundingBox = client->visualRect();
+            if (client->style()->visibility() == EVisibility::Visible &&
+                !boundingBox.width() && !boundingBox.height())
+              LayoutSVGResourceContainer::markForLayoutAndParentResourceInvalidation(client, true);

[fs, 2017/01/16 10:03:10]

Why isn't this handled by the call to markForLayoutAndParentResourceInvalidation
at the end of the function? (Which should already walk the ancestor chain and
"discover" a potential LayoutSVGResourceContainer ancestor if there is one.)

+          }
+        }
+      }
+      break;
+    }
+    current = current->parent();
+  }
+
   // 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);
   }
@@ skipping 32 matching lines @@
   ASSERT(layoutObject);
 
   SVGResources* resources = cachedResourcesForLayoutObject(layoutObject);
   if (resources)
     resources->removeClientFromCache(layoutObject);
   SVGResourcesCache& cache = resourcesCache(layoutObject->document());
   cache.removeResourcesFromLayoutObject(layoutObject);
 }
 
 }  // namespace blink