[Webkit-unassigned] [Bug 72208] [chromium] Fix scaleDelta zoom-out visibility rect bug

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=72208





--- Comment #19 from James Robinson <jamesr at chromium.org>  2011-11-17 12:47:46 PST ---
(In reply to comment #17)
> (In reply to comment #7)
> > (In reply to comment #3)
> > > setDeltaScale() just seems to support a scale change, but zoomAnimatorTransform needs to support translation as well.
> > 
> > Can you expand a bit on why?  That seems wrong, translations should be an orthogonal concept to zoom - you typically want to change the scroll offsets when zooming but that's handled by separate code.
> 
> The zoom animator only redraws existing textures, so pageScale doesn't change during the animation. If I remember correctly, given the mis-match between page scale and the zoom animation, scrolling does not work correctly *during* the animation.

Right, that's why you need to adjust the scroll offsets when adjusting the scale during the animation.'

> 
> When the final page scale is set, we use an animation-less scroll to align the content around the zoom point.

Correct.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.