Bug 117697 - [CSS Shapes] lineOverflowsFromShapeInside should not be always initialized to false
Summary: [CSS Shapes] lineOverflowsFromShapeInside should not be always initialized to...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Zoltan Horvath
URL:
Keywords:
Depends on:
Blocks: 89256 116296
  Show dependency treegraph
 
Reported: 2013-06-17 08:12 PDT by Andrei Bucur
Modified: 2013-06-18 14:52 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrei Bucur 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.
Comment 1 Andrei Bucur 2013-06-17 08:43:50 PDT
Tests are failing with the patch for 116296.
Comment 2 Bear Travis 2013-06-17 10:47:32 PDT
Linking to the master bug
Comment 3 Zoltan Horvath 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!
Comment 4 Andrei Bucur 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.
Comment 5 Zoltan Horvath 2013-06-18 14:52:25 PDT
I got rid of lineOverflowsFromShapeInside in bug #117757, thus this bug is not valid anymore.