Bug 112068

Summary: Hit testing should use ancestorInThisScope to get the non-shadow ancestor
Product: WebKit Reporter: Christian Biesinger <cbiesinger>
Component: New BugsAssignee: Christian Biesinger <cbiesinger>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, eric.carlson, eric, esprehn+autocc, esprehn, feature-media-reviews, jer.noble, ojan.autocc, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 110913    
Attachments:
Description Flags
Patch
none
Patch none

Christian Biesinger
Reported 2013-03-11 14:37:33 PDT
Hit testing should use ancestorInThisScope to get the non-shadow ancestor
Attachments
Patch (3.57 KB, patch)
2013-03-11 14:39 PDT, Christian Biesinger 		no flags
DetailsFormatted DiffDiff
Patch (3.89 KB, patch)
2013-03-11 14:44 PDT, Christian Biesinger 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Christian Biesinger
Comment 1 2013-03-11 14:39:48 PDT
Created attachment 192561 [details] Patch
Christian Biesinger
Comment 2 2013-03-11 14:44:19 PDT
Created attachment 192566 [details] Patch
Elliott Sprehn
Comment 3 2013-03-11 14:51:12 PDT
This doesn't look right, you're forcing the hit test to jump all the way out of every shadow back into the main document. What bug is this trying to fix?
Elliott Sprehn
Comment 4 2013-03-11 15:35:30 PDT
(In reply to comment #3) > This doesn't look right, you're forcing the hit test to jump all the way out of every shadow back into the main document. > > What bug is this trying to fix? Resolved this on IRC, patch looks good, we filed a bug about the cases this breaks.
WebKit Review Bot
Comment 5 2013-03-11 18:10:47 PDT
Comment on attachment 192566 [details] Patch Clearing flags on attachment: 192566 Committed r145448: <http://trac.webkit.org/changeset/145448>
WebKit Review Bot
Comment 6 2013-03-11 18:10:50 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.