Handle 'visibility:collapse' for SVG shapes/images/text-decorations

third_party/WebKit/Source/core/paint/SVGImagePainter.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/SVGImagePainter.h"
 
 #include "core/layout/ImageQualityController.h"
 #include "core/layout/LayoutImageResource.h"
 #include "core/layout/svg/LayoutSVGImage.h"
 #include "core/layout/svg/SVGLayoutSupport.h"
 #include "core/paint/LayoutObjectDrawingRecorder.h"
 #include "core/paint/ObjectPainter.h"
 #include "core/paint/PaintInfo.h"
 #include "core/paint/SVGPaintContext.h"
 #include "core/paint/TransformRecorder.h"
 #include "core/svg/SVGImageElement.h"
 #include "core/svg/graphics/SVGImage.h"
 #include "platform/graphics/GraphicsContext.h"
 #include "third_party/skia/include/core/SkPicture.h"
 
 namespace blink {
 
 void SVGImagePainter::paint(const PaintInfo& paintInfo)
 {
     if (paintInfo.phase != PaintPhaseForeground
         || m_layoutSVGImage.style()->visibility() == HIDDEN

[pdr., 2016/06/07 23:02:25]

We should just check for "visibility() != VISIBLE" here and in the cases below.

[Shanmuga Pandi, 2016/06/08 05:17:13]

Done.

+        || m_layoutSVGImage.style()->visibility() == COLLAPSE
         || !m_layoutSVGImage.imageResource()->hasImage())
         return;
 
     FloatRect boundingBox = m_layoutSVGImage.paintInvalidationRectInLocalSVGCoordinates();
     if (!paintInfo.cullRect().intersectsCullRect(m_layoutSVGImage.localToSVGParentTransform(), boundingBox))
         return;
 
     PaintInfo paintInfoBeforeFiltering(paintInfo);
     // Images cannot have children so do not call updateCullRect.
     TransformRecorder transformRecorder(paintInfoBeforeFiltering.context, m_layoutSVGImage, m_layoutSVGImage.localToSVGParentTransform());
@@ skipping 54 matching lines @@
     if (cachedImage->errorOccurred())
         return FloatSize();
 
     if (cachedImage->getImage()->isSVGImage())
         return toSVGImage(cachedImage->getImage())->concreteObjectSize(m_layoutSVGImage.objectBoundingBox().size());
 
     return FloatSize(cachedImage->getImage()->size());
 }
 
 } // namespace blink