1352027 - Investigate simplifying SVGSVGElement::SetCurrentScaleTranslate for removal of SVGZoomEvent

Status: RESOLVED FIXED

(Core :: SVG, enhancement, P3)

Description

[Aryeh Gregor (:ayg) (no longer with Mozilla)]

Bug 1314388 removes SVGZoomEvent.  The comment in SVGSVGElement::SetCurrentScaleTranslate suggests that a bunch of the code there is no longer needed:

   // IMPORTANT: If either mCurrentTranslate *or* mCurrentScale is changed then
   // mPreviousTranslate_x, mPreviousTranslate_y *and* mPreviousScale must all
   // be updated otherwise SVGZoomEvents will end up with invalid data. I.e. an
   // SVGZoomEvent's properties previousScale and previousTranslate must contain
   // the state of currentScale and currentTranslate immediately before the
   // change that caused the event's dispatch, which is *not* necessarily the
   // same thing as the values of currentScale and currentTranslate prior to
   // their own last change.

But I'm not sure if it's accurate, so I didn't try to actually remove it.