Bug 33229

Summary: Don't use renderer for Resources in SVGRenderBase::prepareToRenderSVGContent
Product: WebKit Reporter: Dirk Schulze <krit>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: webkit.review.bot, zimmermann
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 68469, 26389    
Attachments:
Description Flags
Don't use Renderer for SVGResource calculations zimmermann: review-

Description Dirk Schulze 2010-01-05 14:09:03 PST 
Don't use renderer for Resources in SVGRenderBase::prepareToRenderSVGContent. Some Renderer don't provide correct boundingBoxes or don't inherit from SVGRenderBase. This can cause an ASSERT in RenderObject.
Comment 1 Dirk Schulze 2010-01-05 14:24:04 PST 
Created attachment 45931 [details]
Don't use Renderer for SVGResource calculations

Don't use Renderer for SVGResource calculations.
Comment 2 WebKit Review Bot 2010-01-05 14:28:20 PST 
style-queue ran check-webkit-style on attachment 45931 [details] without any errors.
Comment 3 Nikolas Zimmermann 2010-01-06 19:09:13 PST 
Comment on attachment 45931 [details]
Don't use Renderer for SVGResource calculations

Postponed, as discussed with Dirk. Let's fix RenderSVGInline, and pass around renderers.
Comment 4 Dirk Schulze 2010-04-21 13:32:02 PDT 
The bounding box / repaint rect problems are fixed. Closing this bug now.