Closed Bug 786736 Opened 13 years ago Closed 13 years ago

Initialize nsSVGOuterSVGFrame::mFullZoom to the actual value in its ctor

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla18

People

(Reporter: jwatt, Assigned: jwatt)

Details

(Keywords: perf)

Attachments

(1 file)

I noticed while debugging a different bug that in nsSVGOuterSVGFrame::Reflow we end up passing FULL_ZOOM_CHANGED to NotifyViewportOrTransformChanged on first reflow because the initial value of mFullZoom doesn't match PresContext()->GetFullZoom(). That results in us wastefully calling NotifySVGChanged() and recursing over the frame tree.
Attached patch patchSplinter Review
Attachment #656506 - Flags: review?(roc)
Flags: in-testsuite-
Target Milestone: --- → mozilla18
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: