Bug 117697

Summary:	[CSS Shapes] lineOverflowsFromShapeInside should not be always initialized to false
Product:	WebKit	Reporter:	Andrei Bucur <abucur>
Component:	WebCore Misc.	Assignee:	Zoltan Horvath <zoltan>
Status:	RESOLVED INVALID
Severity:	Normal	CC:	betravis, WebkitBugTracker, zoltan
Priority:	P2
Version:	528+ (Nightly build)
Hardware:	Unspecified
OS:	Unspecified
Bug Depends on:
Bug Blocks:	89256, 116296

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.