If the <use> is hidden and the child of it is visible, should clip the non-resource

third_party/WebKit/Source/core/svg/SVGUseElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007 Rob Buis <buis@kde.org>
  * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
  * Copyright (C) 2011 Torch Mobile (Beijing) Co. Ltd. All rights reserved.
  * Copyright (C) 2012 University of Szeged
  * Copyright (C) 2012 Renata Hodovan <reni@webkit.org>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 464 matching lines @@
         || isSVGEllipseElement(element)
         || isSVGPolygonElement(element)
         || isSVGPolylineElement(element)
         || isSVGTextElement(element);
 }
 
 void SVGUseElement::toClipPath(Path& path) const
 {
     ASSERT(path.isEmpty());
 
-    const SVGGraphicsElement* element = targetGraphicsElementForClipping();
+    const SVGGraphicsElement* element = visibleTargetGraphicsElementForClipping();
 
     if (!element || !element->layoutObject())

[fs, 2016/04/27 14:14:40]

You should move the second part of this condition into
visibleTargetGraphicsElementForClipping too - it's already there, so this will
be redundant.

[hyunjunekim2, 2016/04/29 12:59:09]

Done. Moved to visibleTargetGraphicsElementForClipping function.

         return;
 
     if (element->isSVGGeometryElement()) {
         toSVGGeometryElement(*element).toClipPath(path);
         // FIXME: Avoid manual resolution of x/y here. Its potentially harmful.
         SVGLengthContext lengthContext(this);
         path.translate(FloatSize(m_x->currentValue()->value(lengthContext), m_y->currentValue()->value(lengthContext)));
         path.transform(calculateAnimatedLocalTransform());
     }
 }
 
-SVGGraphicsElement* SVGUseElement::targetGraphicsElementForClipping() const
+SVGGraphicsElement* SVGUseElement::visibleTargetGraphicsElementForClipping() const
 {
     Node* n = userAgentShadowRoot()->firstChild();
     if (!n || !n->isSVGElement())
         return nullptr;
 
     SVGElement& element = toSVGElement(*n);
 
     if (!element.isSVGGraphicsElement())
         return nullptr;
 
+    const ComputedStyle* style = element.layoutObject() ? element.layoutObject()->style() : nullptr;
+    if (!style || style->visibility() != VISIBLE)
+        return nullptr;
+
     // Spec: "If a <use> element is a child of a clipPath element, it must directly
     // reference <path>, <text> or basic shapes elements. Indirect references are an
     // error and the clipPath element must be ignored."
     // http://dev.w3.org/fxtf/css-masking-1/#the-clip-path
     if (!isDirectReference(element)) {
         // Spec: Indirect references are an error (14.3.5)
         document().accessSVGExtensions().reportError("Not allowed to use indirect reference in <clip-path>");
         return nullptr;
     }
 
@@ skipping 222 matching lines @@
 
     if (m_resource)
         m_resource->removeClient(this);
 
     m_resource = resource;
     if (m_resource)
         m_resource->addClient(this);
 }
 
 } // namespace blink