[Webkit-unassigned] [Bug 99489] Incorrect embedded SVG image sizing on first load

[bugzilla-daemon at webkit.org]

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


Florin Malita <fmalita at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|webkit-unassigned at lists.web |fmalita at chromium.org
                   |kit.org                     |




--- Comment #1 from Florin Malita <fmalita at chromium.org>  2012-10-17 10:59:37 PST ---
Looks like a race between layout and the image loader:

* SVGImageCache::lookupOrCreateBitmapImageForRenderer() needs a m_sizeAndScalesMap entry for the renderer
* the only time this is set is from  RenderSVGImage::layout() (via updateImageViewport() >> m_imageResource->setContainerSizeForRenderer() >> m_cachedImage->setContainerSizeForRenderer() >> m_svgImageCache->setRequestedSizeAndScales())


The problem is that for this to work, the cached image needs to be present/loaded when RenderSVGImage::layout() is called. If loading finishes after layout, setRequestedSizeAndScales() never gets called.

Extending RenderSVGImage::imageChanged() to either force a relayout or just call setRequestedSizeAndScales() should cover this case.

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