RESOLVED INVALID 117697
[CSS Shapes] lineOverflowsFromShapeInside should not be always initialized to false 
https://bugs.webkit.org/show_bug.cgi?id=117697
Summary [CSS Shapes] lineOverflowsFromShapeInside should not be always initialized to...
Andrei Bucur
Reported 2013-06-17 08:12:26 PDT
During RenderBlock::layoutRunsAndFloatsInRange the lineOverflowsFromShapeInside is set to true if the lines don't fit the shape inside any more and must be placed at the bottom of the content box. This value is always initialized to false even though the block height is already after the shape bottom. This is wrong for partial layouts when only some of the lines are recomputed (see RenderBlock::determineStartPosition). If the start line is after the shape bottom, the flag will be initialized to false and the block height incorrectly updated to the bottom of the shape content box.
Attachments
Add attachment proposed patch, testcase, etc.
Andrei Bucur
Comment 1 2013-06-17 08:43:50 PDT
Tests are failing with the patch for 116296.
Bear Travis
Comment 2 2013-06-17 10:47:32 PDT
Linking to the master bug
Zoltan Horvath
Comment 3 2013-06-17 13:41:49 PDT
(In reply to comment #1) > Tests are failing with the patch for 116296. The latest patch on bug #116296 no longer applies to the ToT, please update it and I'll take a look at it. As I see from the change only one shape-inside on regions test is failing due to this issue, please skip and link it to this bug if it's a blocker for you and I'll remove from the skipped list later. Thanks!
Andrei Bucur
Comment 4 2013-06-18 08:56:18 PDT
(In reply to comment #3) > (In reply to comment #1) > > Tests are failing with the patch for 116296. > > The latest patch on bug #116296 no longer applies to the ToT, please update it and I'll take a look at it. As I see from the change only one shape-inside on regions test is failing due to this issue, please skip and link it to this bug if it's a blocker for you and I'll remove from the skipped list later. > > Thanks! I've updated the WIP patch. The failing tests can be found in the TestExpectations file. For now, I think it's fine if I disable the tests in my patch.
Zoltan Horvath
Comment 5 2013-06-18 14:52:25 PDT
I got rid of lineOverflowsFromShapeInside in bug #117757, thus this bug is not valid anymore.
Note You need to log in before you can comment on or make changes to this bug.