| Summary: | Hit test through 3d transforms | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> | ||||||
| Component: | Layout and Rendering | Assignee: | Simon Fraser (smfr) <simon.fraser> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | cmarrin, dino, hausmann, hyatt, kari.hiitola, kim.1.gronholm | ||||||
| Priority: | P2 | ||||||||
| Version: | 528+ (Nightly build) | ||||||||
| Hardware: | Mac | ||||||||
| OS: | OS X 10.5 | ||||||||
| Bug Depends on: | 24313, 24317 | ||||||||
| Bug Blocks: | 23359 | ||||||||
| Attachments: |
|
||||||||
|
Description
Simon Fraser (smfr)
2009-03-02 20:08:56 PST
Created attachment 28214 [details]
Patch, changelog.
Testcases will be added once we turn on support for 3d rendering.
Created attachment 28224 [details]
Prequel patch: refactor code into transformFromContainer()
Comment on attachment 28224 [details] Prequel patch: refactor code into transformFromContainer() > diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog > index e7b3153..0256ec1 100644 > --- a/WebCore/ChangeLog > +++ b/WebCore/ChangeLog > @@ -2,6 +2,22 @@ > > Reviewed by NOBODY (OOPS!). > > + https://bugs.webkit.org/show_bug.cgi?id=24312 > + > + Factor the code that compute a transform relative to the container > + into RenderObject::transformFromContainer(). > + "that computes a transform" r=me Comment on attachment 28214 [details]
Patch, changelog.
"Recurses through the layer hieraarchy"
Actually just remove this comment. I think it's just confusing. hit testing doesn't go through the parent/child hierarchy, which is what the comment implies.
Prequel patch: http://trac.webkit.org/changeset/41387 *** Bug 24573 has been marked as a duplicate of this bug. *** I tested with the latest nightly on a Mac, and the hit testing really doesn't seem to work as it should. If you try the test page of the duplicate bug: https://bugs.webkit.org/attachment.cgi?id=28576 the child div can not be clicked. I would suggest reopening this bug. |