Bug 12391

Summary: SVG containers with viewports should respect overflow when hit testing
Product: WebKit Reporter: Rob Buis <rwlbuis>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
First attempt
mjs: review-
Improved patch mjs: review+

Description Rob Buis 2007-01-24 02:04:20 PST 
The summary says it all. You can note this effect in carto.net/window.svg on the buttons.
Comment 1 Rob Buis 2007-01-24 02:28:02 PST 
Created attachment 12642 [details]
First attempt

This patch should fix the problem. I'll look whether there are problems too for other svg elements that create a viewport.
Cheers,

Rob.
Comment 2 Maciej Stachowiak 2007-01-24 18:33:38 PST 
Shouldn't you respect the overflow property in every case, not just on the SVG element? It's a little weird to have a tag name check like that in the render tree. r- to consider this, otherwise patch and test case look great.
Comment 3 Rob Buis 2007-01-25 01:28:29 PST 
Created attachment 12663 [details]
Improved patch

This patch should address Maciej's issue.
Cheers,

Rob.
Comment 4 Maciej Stachowiak 2007-01-25 02:38:47 PST 
Comment on attachment 12663 [details]
Improved patch

r=me
Comment 5 David Kilzer (:ddkilzer) 2007-01-25 06:40:32 PST 
Committed by rwlbuis in r19115.