Bug 24743 - REGRESSION (r41840): Incorrect hit testing at hsbccreditcard.com
Summary: REGRESSION (r41840): Incorrect hit testing at hsbccreditcard.com
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.5
: P1 Major
Assignee: Simon Fraser (smfr)
URL: https://www.hsbccreditcard.com/ecare/...
Keywords: HasReduction, Regression
: 24766 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-03-21 17:19 PDT by mitz
Modified: 2009-03-23 19:47 PDT (History)
3 users (show)

See Also:


Attachments
Reduction (373 bytes, text/html)
2009-03-21 22:17 PDT, Cameron Zwarich (cpst)
no flags Details
Patch, testcase, changelog (4.28 KB, patch)
2009-03-22 11:32 PDT, Simon Fraser (smfr)
mitz: review+
Details | Formatted Diff | Diff
Patch, testcase, changelog (4.58 KB, patch)
2009-03-22 11:36 PDT, Simon Fraser (smfr)
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2009-03-21 17:19:44 PDT
At hsbccreditcard.com, everything below the logo does not respond to clicking and control-clicking.
Comment 1 Cameron Zwarich (cpst) 2009-03-21 21:07:31 PDT
I'll try to reduce this, but I won't be the one to fix it.
Comment 2 Cameron Zwarich (cpst) 2009-03-21 21:11:43 PDT
This regressed between the r41760 and r41844 nightlies. There are a few changes to hit testing code in that range.
Comment 3 Cameron Zwarich (cpst) 2009-03-21 22:17:24 PDT
Created attachment 28830 [details]
Reduction

Here's a really simple reduction. I'll bisect local builds to find the point of failure.
Comment 4 Cameron Zwarich (cpst) 2009-03-21 22:49:30 PDT
This regressed in r41840:

http://trac.webkit.org/changeset/41840

I am unassigning this. One of the hit testing wizards should fix this. ;-)
Comment 5 Simon Fraser (smfr) 2009-03-22 10:13:14 PDT
Hm, r41840 should not have affected non-3d hit testing.
Comment 6 Simon Fraser (smfr) 2009-03-22 11:32:48 PDT
Created attachment 28834 [details]
Patch, testcase, changelog
Comment 7 mitz 2009-03-22 11:36:24 PDT
Comment on attachment 28834 [details]
Patch, testcase, changelog

> +        * fast/layers/zindex-hit-test.html: Added.

Please generate results, add them to the change log and check them in.

> +  #container {
> +  /*    position: absolute;*/
> +  }

You can just remove this rule.

> \ No newline at end of file

Please add a newline.

r=me
Comment 8 Simon Fraser (smfr) 2009-03-22 11:36:47 PDT
Created attachment 28835 [details]
Patch, testcase, changelog

Add test expected file.
Comment 9 mitz 2009-03-22 11:38:27 PDT
Comment on attachment 28835 [details]
Patch, testcase, changelog

The rest of my previous comments still apply to this patch.
r=me
Comment 10 Simon Fraser (smfr) 2009-03-22 11:43:40 PDT
http://trac.webkit.org/changeset/41900
Comment 11 Cameron Zwarich (cpst) 2009-03-23 19:47:47 PDT
*** Bug 24766 has been marked as a duplicate of this bug. ***