Summary: | [CSS Regions] Test absolute positioning of an element inside a CSS region | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Razvan Caliman <rcaliman> | ||||||||||
Component: | CSS | Assignee: | Razvan Caliman <rcaliman> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | donggwan.kim, mibalan, mihnea, rcaliman, shanestephens, tony, WebkitBugTracker, webkit.review.bot | ||||||||||
Priority: | P2 | Keywords: | AdobeTracked | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
URL: | http://www.w3.org/TR/css3-regions/#positioned-flow0 | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 57312 | ||||||||||||
Attachments: |
|
Description
Razvan Caliman
2011-08-19 05:52:18 PDT
Created attachment 104496 [details]
Patch
Comment on attachment 104496 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=104496&action=review > LayoutTests/fast/regions/hit-test-absolute-position.html:42 > + var hitElement = document.elementFromPoint(270, 330); The coordinates of the box seems unreliable because of the text at the top of the page. E.g., other platforms may have different font size or spacing causing the box to be positioned higher or lower. I would either compute the location of the 'region' div via javascript (you will find many existing tests that use eventSender.mouse* functions that do this) or put the text below the region div. Created attachment 104680 [details]
Patch
Thank you, Tony!
I have updated the test to place the tested elements before the descriptive text. I also added a css reset for margins for good measure.
Mihnea, if you could also have a look, I appreciate it.
Comment on attachment 104680 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=104680&action=review > LayoutTests/platform/mac-snowleopard/fast/regions/hit-test-absolute-position-expected.txt:2 > +PASS > +Test that absolutely positioned elements inside a flow respect the initial containing block defined by the first region. You don't need this file since it's the same as LayoutTests/fast/regions/hit-test-absolute-position-expected.txt. Created attachment 104849 [details]
Patch
I have reworked the patch and removed the platform-specific test files.
Attachment 104849 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/fast..." exit_code: 1
LayoutTests/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] [5]
Total errors found: 1 in 3 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 104852 [details]
Patch
Fixed the patch by removing the platform-specific baseline. With correct ChangeLog entry this time.
Comment on attachment 104852 [details]
Patch
Sorry for the slow review.
Comment on attachment 104852 [details] Patch Rejecting attachment 104852 [details] from commit-queue. New failing tests: fast/regions/hit-test-absolute-position.html Full output: http://queues.webkit.org/results/11569121 (In reply to comment #9) > (From update of attachment 104852 [details]) > Rejecting attachment 104852 [details] from commit-queue. > > New failing tests: > fast/regions/hit-test-absolute-position.html > Full output: http://queues.webkit.org/results/11569121 Oh, this might be because of the recent work to make regions runtime switchable? There are already some tests for this, like those for the bug 88133 and others. Closing as Resolved/Fixed. |