Bug 140789

Summary: Simple line layout: Reset LineState when starting a new line.
Product: WebKit Reporter: zalan <zalan>
Component: Layout and RenderingAssignee: zalan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, esprehn+autocc, glenn, koivisto, kondapallykalyan, mmaxfield
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch koivisto: review+

zalan
Reported 2015-01-22 14:38:54 PST
To be more explicit about what data we need to transfer from the previous line.
Attachments
Patch (6.00 KB, patch)
2015-01-22 15:15 PST, zalan 		koivisto: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
zalan
Comment 1 2015-01-22 15:15:02 PST
Created attachment 245170 [details] Patch
Antti Koivisto
Comment 2 2015-01-22 15:19:07 PST
Comment on attachment 245170 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=245170&action=review > Source/WebCore/rendering/SimpleLineLayout.cpp:362 > +static void initializeNewLine(LineState& lineState, const RenderBlockFlow& flow, const FlowContents& flowContents, unsigned lineStartRunIndex) You could return the new line instead of mutating the argument.
zalan
Comment 3 2015-01-22 15:42:41 PST
Committed r178964: <http://trac.webkit.org/changeset/178964>
Note You need to log in before you can comment on or make changes to this bug.