From 7cf4ae71220fc2307cf387205fc64dab13ecf7bf Mon Sep 17 00:00:00 2001 From: fmalita Date: Mon, 19 Sep 2016 06:42:39 -0700 Subject: [SVGDom] Only apply opacity when needed Opacity: 1 is a no-op, but it shows up quite a bit in existing documents. R=robertphillips@google.com,stephana@google.com,reed@google.com GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2349143002 Review-Url: https://codereview.chromium.org/2349143002 --- experimental/svg/model/SkSVGRenderContext.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'experimental/svg') diff --git a/experimental/svg/model/SkSVGRenderContext.cpp b/experimental/svg/model/SkSVGRenderContext.cpp index 46ed5a28c4..7e2e90e22c 100644 --- a/experimental/svg/model/SkSVGRenderContext.cpp +++ b/experimental/svg/model/SkSVGRenderContext.cpp @@ -267,7 +267,8 @@ void SkSVGRenderContext::applyPresentationAttributes(const SkSVGPresentationAttr // Uninherited attributes. Only apply to the current context. - if (auto* opacity = attrs.fOpacity.getMaybeNull()) { + auto* opacity = attrs.fOpacity.getMaybeNull(); + if (opacity && opacity->value() < 1) { SkPaint opacityPaint; opacityPaint.setAlpha(opacity_to_alpha(opacity->value())); // Balanced in the destructor, via restoreToCount(). -- cgit v1.2.3