Bug 126528

Summary: [CSS Shapes] shape-outside layout incorrect when line spans rounded box rounded corners
Product: WebKit Reporter: Hans Muller <giles_joplin>
Component: CSSAssignee: Hans Muller <giles_joplin>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, glenn, kling, kondapallykalyan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Test case.
none
Test case screenshot.
none
Patch
none
Patch none

Description Hans Muller 2014-01-06 10:11:26 PST
Created attachment 220439 [details]
Test case.

If a line spans a shape-outside box's top/bottom rounded corners, it's starting point is based on (only) the intersection of the top/bottom edges of the line and the corner arcs.

The attached example demonstrates the problem.
Comment 1 Hans Muller 2014-01-06 10:16:35 PST
Created attachment 220440 [details]
Test case screenshot.
Comment 2 Hans Muller 2014-01-06 12:05:43 PST
Created attachment 220444 [details]
Patch

BoxShape::getExcludedIntervals() now checks for the special case where the line spans the top and bottom rounded corners - which implies that the excluded interval is the box's horizontal extent.
Comment 3 Hans Muller 2014-01-06 16:17:04 PST
Created attachment 220467 [details]
Patch

Corrected the test filenames.
Comment 4 Andreas Kling 2014-01-07 08:03:45 PST
Comment on attachment 220467 [details]
Patch

r=me
Comment 5 WebKit Commit Bot 2014-01-07 10:26:31 PST
Comment on attachment 220467 [details]
Patch

Clearing flags on attachment: 220467

Committed r161434: <http://trac.webkit.org/changeset/161434>
Comment 6 WebKit Commit Bot 2014-01-07 10:26:32 PST
All reviewed patches have been landed.  Closing bug.