diff options
author | fmalita <fmalita@chromium.org> | 2016-09-19 06:42:39 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-09-19 06:42:39 -0700 |
commit | 7cf4ae71220fc2307cf387205fc64dab13ecf7bf (patch) | |
tree | c658b0d4ecb4cd8770d8c57cd67c69ecd941f2ec /experimental/svg | |
parent | 12f752723b31bb015ded118bfa466c2f1e9b2905 (diff) |
[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
Diffstat (limited to 'experimental/svg')
-rw-r--r-- | experimental/svg/model/SkSVGRenderContext.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
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(). |