When the SVG element is 'visible', the SVG element is not shown

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

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/paint/ReplacedPainter.h"
 
 #include "core/layout/LayoutReplaced.h"
 #include "core/layout/api/SelectionState.h"
 #include "core/layout/svg/LayoutSVGRoot.h"
 #include "core/paint/BoxPainter.h"
 #include "core/paint/LayoutObjectDrawingRecorder.h"
 #include "core/paint/ObjectPainter.h"
 #include "core/paint/PaintInfo.h"
 #include "core/paint/PaintLayer.h"
 #include "core/paint/RoundedInnerRectClipper.h"
 #include "wtf/Optional.h"
 
 namespace blink {
 
 static bool shouldApplyViewportClip(const LayoutReplaced& layoutReplaced)
 {
     return !layoutReplaced.isSVGRoot() || toLayoutSVGRoot(&layoutReplaced)->shouldApplyViewportClip();
 }
 
 void ReplacedPainter::paint(const PaintInfo& paintInfo, const LayoutPoint& paintOffset)
 {
     LayoutPoint adjustedPaintOffset = paintOffset + m_layoutReplaced.location();
-    if (!shouldPaint(paintInfo, adjustedPaintOffset))
+    if (!shouldPaint(paintInfo, adjustedPaintOffset)) {
+        if (m_layoutReplaced.isSVGRoot() && m_layoutReplaced.style()->visibility() != VISIBLE)

[fs, 2016/03/21 16:07:58]

I _think_ that clipping from 'overflow' should still be applied, i.e the
following should still clip:

https://jsfiddle.net/jzmdr360/1/
https://jsfiddle.net/vzn8r10o/

So maybe the check for visibility == hidden needs to be hoisted out of
shouldPaint(...).

[hyunjunekim2, 2016/03/23 14:19:33]

Could you checkPS13? Added more tests.
Thank you.

+            m_layoutReplaced.paintReplaced(paintInfo, adjustedPaintOffset);
         return;
+    }
 
     LayoutRect borderRect(adjustedPaintOffset, m_layoutReplaced.size());
 
     if (m_layoutReplaced.hasBoxDecorationBackground() && (paintInfo.phase == PaintPhaseForeground || paintInfo.phase == PaintPhaseSelection))
         m_layoutReplaced.paintBoxDecorationBackground(paintInfo, adjustedPaintOffset);
 
     if (paintInfo.phase == PaintPhaseMask) {
         m_layoutReplaced.paintMask(paintInfo, adjustedPaintOffset);
         return;
     }
@@ skipping 69 matching lines @@
     paintRect.unite(m_layoutReplaced.localSelectionRect());
     paintRect.moveBy(adjustedPaintOffset);
 
     if (!paintInfo.cullRect().intersectsCullRect(paintRect))
         return false;
 
     return true;
 }
 
 } // namespace blink