Summary: | [CSS Exclusions] Add shape-inside tests for rounded rectangles | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Bear Travis <betravis> | ||||||||
Component: | CSS | Assignee: | Bear Travis <betravis> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | donggwan.kim, giles_joplin, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 89256, 89707, 98462 | ||||||||||
Attachments: |
|
Description
Bear Travis
2012-09-13 10:48:17 PDT
Created attachment 166763 [details]
Adding rounded rectangle tests
(In reply to comment #1) > Created an attachment (id=166763) [details] > Adding rounded rectangle tests These tests look good to me although you might want make the height of one of the test case shapes greater than its width. Likewise for the corner radii. And it would simple and useful to verify that setting either corner radius value to zero produced the same results as a simple rectangle. > "The expected results use float "sandbags" to mark off the shape-inside area." You might qualify this, since shape-inside-rounded-rectangle-001-expected.html doesn't use sandbags, just carefully placed breaks. Created attachment 167201 [details] Updating tests Incorporating feedback from Hans. * Making one of the test cases have height > width and cornerRadiusY > cornerRadiusX * Testing the case where cornerRadiusX or cornerRadiusY is 0 * Fixing up the comments to be more specific about only some of the tests using float sandbags Corner radii greater than half their dimension (shape width or height) currently produce odd results. Linking to new bug 98462. Created attachment 167353 [details]
Adding shape outline
Updating patch to use the ::before pseudo element to represent the outline of the shape inside.
Comment on attachment 167353 [details] Adding shape outline View in context: https://bugs.webkit.org/attachment.cgi?id=167353&action=review > LayoutTests/fast/exclusions/shape-inside/shape-inside-rounded-rectangle-002-expected.html:15 > + function createSimulatedShapeElement() { > + var stylesheet = document.getElementById("stylesheet"); > + var dimensions = { width: 200, height: 300, > + shapeX: 25, shapeY: 45, > + shapeWidth: 100, shapeHeight: 200, > + shapeRadiusX: 0, shapeRadiusY: 55 }; > + generateSimulatedShapeElement("shape-inside1", stylesheet.sheet, dimensions, 9); > + dimensions.shapeRadiusX = 55; > + dimensions.shapeRadiusY = 0; > + generateSimulatedShapeElement("shape-inside2", stylesheet.sheet, dimensions, 9); Super cool way to do ref expectations. Comment on attachment 167353 [details]
Adding shape outline
Just setting cq+
Comment on attachment 167353 [details] Adding shape outline Clearing flags on attachment: 167353 Committed r130557: <http://trac.webkit.org/changeset/130557> All reviewed patches have been landed. Closing bug. |