diff options
| author | Botond Ballo <botond@mozilla.com> | 2018-03-09 17:26:24 -0500 |
|---|---|---|
| committer | wolfbeast <mcwerewolf@gmail.com> | 2018-04-19 11:38:48 +0200 |
| commit | 85d6e8c1b47d826f574b320b690a510d65632c03 (patch) | |
| tree | 2802ed586a7d9fbb26bbb061794f433ff83bcbd5 /layout | |
| parent | 098fb55f0928adc78c8c471361f19d25810ae7ea (diff) | |
| download | uxp-85d6e8c1b47d826f574b320b690a510d65632c03.tar.gz | |
Bug 1443092 - Avoid calling SVGAnimatedEnumeration::AnimVal() from nsSVGUtils::GetBBox(). r=jwatt, a=RyanVM
AnimVal() is a DOM getter, and it flushes animations, which we don't want
in GetBBox() which is called from display list building cide and
FrameLayerBuilder.
MozReview-Commit-ID: DWgm7wAV7C0
Diffstat (limited to 'layout')
| -rw-r--r-- | layout/svg/nsSVGUtils.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/layout/svg/nsSVGUtils.cpp b/layout/svg/nsSVGUtils.cpp index ff74d5bafc..344ebf645a 100644 --- a/layout/svg/nsSVGUtils.cpp +++ b/layout/svg/nsSVGUtils.cpp @@ -1127,8 +1127,7 @@ nsSVGUtils::GetBBox(nsIFrame *aFrame, uint32_t aFlags) if (clipPathFrame && isOK) { SVGClipPathElement *clipContent = static_cast<SVGClipPathElement*>(clipPathFrame->GetContent()); - RefPtr<SVGAnimatedEnumeration> units = clipContent->ClipPathUnits(); - if (units->AnimVal() == SVG_UNIT_TYPE_OBJECTBOUNDINGBOX) { + if (clipContent->IsUnitsObjectBoundingBox()) { matrix.Translate(gfxPoint(x, y)); matrix.Scale(width, height); } else if (aFrame->GetType() == nsGkAtoms::svgForeignObjectFrame) { |