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

Description Christian Biesinger 2013-03-11 14:37:33 PDT
Hit testing should use ancestorInThisScope to get the non-shadow ancestor
Comment 1 Christian Biesinger 2013-03-11 14:39:48 PDT
Created attachment 192561 [details]
Patch
Comment 2 Christian Biesinger 2013-03-11 14:44:19 PDT
Created attachment 192566 [details]
Patch
Comment 3 Elliott Sprehn 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?
Comment 4 Elliott Sprehn 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.
Comment 5 WebKit Review Bot 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>
Comment 6 WebKit Review Bot 2013-03-11 18:10:50 PDT
All reviewed patches have been landed.  Closing bug.