619835 - Optimize <svg:image> single-observer use case by caching override preserveAspectRatio between paints

Status: ASSIGNED

(Core :: SVG, defect)

Description

[Daniel Holbert [:dholbert]]

(Quoting bug 272288 comment #68)
> roc: You'd also mentioned in IRC possibly caching the last-used override
> preserveAspectRatio, e.g. on the VectorImage or SVGDocumentWrapper, so that we
> don't have to flush a viewport change on every single draw in the
> single-<image>-client case.
> 
> If it's ok with you, I'd like to do that as a small followup bug, since it's an
> optimization & shouldn't affect behavior, and this bug's already suffered more
> than its fair share of patch-spam. :)  (And also on the off chance that the
> optimization causes regressions or breakage, I'd like to be able to back out
> the optimization without backing out the <image> functionality entirely).

Filing this bug for the above-described optimization.

Comment 1

[Jonathan Watt [:jwatt]]

I think the fix for bug 764299 (cache SVG images some of the time) may have essentially fixed this.