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

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

Index: third_party/WebKit/Source/core/paint/ReplacedPainter.cpp
diff --git a/third_party/WebKit/Source/core/paint/ReplacedPainter.cpp b/third_party/WebKit/Source/core/paint/ReplacedPainter.cpp
index 6f1c65510f542981c346be17e0dba4c2985e314d..76d0bdcf1245794c68bd737a3eb87e91ba7c4941 100644
--- a/third_party/WebKit/Source/core/paint/ReplacedPainter.cpp
+++ b/third_party/WebKit/Source/core/paint/ReplacedPainter.cpp
@@ -30,7 +30,7 @@ void ReplacedPainter::paint(const PaintInfo& paintInfo, const LayoutPoint& paint
 
     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) {
@@ -102,7 +102,8 @@ bool ReplacedPainter::shouldPaint(const PaintInfo& paintInfo, const LayoutPoint&
         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());