Open Bug 619835 Opened 14 years ago Updated 2 years ago

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

Categories

(Core :: SVG, defect)

defect

Tracking

()

ASSIGNED

People

(Reporter: dholbert, Assigned: dholbert)

References

Details

(Keywords: perf)

(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.
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Keywords: perf
I think the fix for bug 764299 (cache SVG images some of the time) may have essentially fixed this.
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.