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"
@@ skipping 12 matching lines @@
 }
 
 void ReplacedPainter::paint(const PaintInfo& paintInfo, const LayoutPoint& paintOffset)
 {
     LayoutPoint adjustedPaintOffset = paintOffset + m_layoutReplaced.location();
     if (!shouldPaint(paintInfo, adjustedPaintOffset))
         return;
 
     LayoutRect borderRect(adjustedPaintOffset, m_layoutReplaced.size());
 
-    if (m_layoutReplaced.hasBoxDecorationBackground() && (paintInfo.phase == PaintPhaseForeground || paintInfo.phase == PaintPhaseSelection))
+    if (m_layoutReplaced.style()->visibility() == VISIBLE && m_layoutReplaced.hasBoxDecorationBackground() && (paintInfo.phase == PaintPhaseForeground || paintInfo.phase == PaintPhaseSelection))
         m_layoutReplaced.paintBoxDecorationBackground(paintInfo, adjustedPaintOffset);
 
     if (paintInfo.phase == PaintPhaseMask) {
         m_layoutReplaced.paintMask(paintInfo, adjustedPaintOffset);
         return;
     }
 
     if (paintInfo.phase == PaintPhaseClippingMask && (!m_layoutReplaced.hasLayer() || !m_layoutReplaced.layer()->hasCompositedClippingMask()))
         return;
 
@@ skipping 51 matching lines @@
     }
 }
 
 bool ReplacedPainter::shouldPaint(const PaintInfo& paintInfo, const LayoutPoint& adjustedPaintOffset) const
 {
     if (paintInfo.phase != PaintPhaseForeground && !shouldPaintSelfOutline(paintInfo.phase)
         && paintInfo.phase != PaintPhaseSelection && paintInfo.phase != PaintPhaseMask && paintInfo.phase != PaintPhaseClippingMask)
         return false;
 
     // If we're invisible or haven't received a layout yet, just bail.
-    if (m_layoutReplaced.style()->visibility() != VISIBLE)
+    // But if it's SVG Root, just ignore this that check whether visible or not.

[fs, 2016/03/24 15:39:03]

I'd imagine something more like: "But if it's an SVG root, there can be
children, so we'll check visibility later."

[hyunjunekim2, 2016/03/24 16:09:00]

Done.

+    if (!m_layoutReplaced.isSVGRoot() && m_layoutReplaced.style()->visibility() != VISIBLE)
         return false;
 
     LayoutRect paintRect(m_layoutReplaced.visualOverflowRect());
     paintRect.unite(m_layoutReplaced.localSelectionRect());
     paintRect.moveBy(adjustedPaintOffset);
 
     if (!paintInfo.cullRect().intersectsCullRect(paintRect))
         return false;
 
     return true;
 }
 
 } // namespace blink