Fix up that no render SVG shape when change clip-path to visible after hidden.

third_party/WebKit/Source/core/paint/SVGShapePainter.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/SVGShapePainter.h"
 
 #include "core/layout/svg/LayoutSVGResourceMarker.h"
 #include "core/layout/svg/LayoutSVGShape.h"
 #include "core/layout/svg/SVGLayoutSupport.h"
 #include "core/layout/svg/SVGMarkerData.h"
@@ skipping 33 matching lines @@
 }
 
 void SVGShapePainter::paint(const PaintInfo& paintInfo) {
   if (paintInfo.phase != PaintPhaseForeground ||
       m_layoutSVGShape.style()->visibility() != EVisibility::Visible ||
       m_layoutSVGShape.isShapeEmpty())
     return;
 
   FloatRect boundingBox =
       m_layoutSVGShape.paintInvalidationRectInLocalSVGCoordinates();
-  if (!paintInfo.cullRect().intersectsCullRect(
-          m_layoutSVGShape.localSVGTransform(), boundingBox))
-    return;
 
   PaintInfo paintInfoBeforeFiltering(paintInfo);
   // Shapes cannot have children so do not call updateCullRect.
   SVGTransformContext transformContext(paintInfoBeforeFiltering.context,
                                        m_layoutSVGShape,
                                        m_layoutSVGShape.localSVGTransform());
   {
     SVGPaintContext paintContext(m_layoutSVGShape, paintInfoBeforeFiltering);
     if (paintContext.applyClipMaskAndFilterIfNecessary() &&
         !LayoutObjectDrawingRecorder::useCachedDrawingIfPossible(
@@ skipping 60 matching lines @@
             paintMarkers(paintContext.paintInfo(), boundingBox);
             break;
           default:
             ASSERT_NOT_REACHED();
             break;
         }
       }
     }
   }
 
+  if (!paintInfo.cullRect().intersectsCullRect(

[fs, 2016/10/31 09:59:22]

This does not look like the right fix. Here you're just always painting the
shape (possibly sans the outline.)

[hyunjunekim2, 2016/10/31 13:18:30]

I will check it for the right fix.

+          m_layoutSVGShape.localSVGTransform(), boundingBox))
+    return;
+
   if (m_layoutSVGShape.style()->outlineWidth()) {
     PaintInfo outlinePaintInfo(paintInfoBeforeFiltering);
     outlinePaintInfo.phase = PaintPhaseSelfOutlineOnly;
     ObjectPainter(m_layoutSVGShape)
         .paintOutline(outlinePaintInfo, LayoutPoint(boundingBox.location()));
   }
 }
 
 class PathWithTemporaryWindingRule {
  public:
@@ skipping 100 matching lines @@
       marker.markerTransformation(position.origin, position.angle,
                                   strokeWidth));
   Optional<FloatClipRecorder> clipRecorder;
   if (SVGLayoutSupport::isOverflowHidden(&marker))
     clipRecorder.emplace(paintInfo.context, marker, paintInfo.phase,
                          marker.viewport());
   SVGContainerPainter(marker).paint(paintInfo);
 }
 
 }  // namespace blink