Bug 123993

Summary: Region based multicol: support explicit column breaks
Product: WebKit Reporter: Morten Stenshorne <mstensho>
Component: Layout and RenderingAssignee: Morten Stenshorne <mstensho>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, commit-queue, esprehn+autocc, glenn, kondapallykalyan, rniwa, WebkitBugTracker
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Test case
none
Patch
none
Patch
none
Patch
none
Patch none

Description Morten Stenshorne 2013-11-07 07:44:51 PST 
Explicit column breaks are currently not supported in the region based multicol implementation.

Apart from simply making the implementation support it at all (which is trivial), we also have to make the column balancer aware of explicit breaks, which is a bit more work.
Comment 1 Morten Stenshorne 2013-11-07 07:58:12 PST 
Created attachment 216302 [details]
Test case
Comment 2 Morten Stenshorne 2014-01-14 01:28:56 PST 
Created attachment 221122 [details]
Patch
Comment 3 Build Bot 2014-01-14 01:35:39 PST 
Comment on attachment 221122 [details]
Patch

Attachment 221122 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6094733210288128
Comment 4 Morten Stenshorne 2014-01-14 01:48:28 PST 
Created attachment 221126 [details]
Patch
Comment 5 Dave Hyatt 2014-01-17 11:41:14 PST 
Comment on attachment 221126 [details]
Patch

r=me
Comment 6 WebKit Commit Bot 2014-01-20 00:27:36 PST 
Comment on attachment 221126 [details]
Patch

Rejecting attachment 221126 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'apply-attachment', '--no-update', '--non-interactive', 221126, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
html
patching file LayoutTests/fast/multicol/newmulticol/breaks-3-columns-3-expected.html
patching file LayoutTests/fast/multicol/newmulticol/breaks-3-columns-3.html
patching file LayoutTests/fast/multicol/newmulticol/fixed-height-fill-balance-2-expected.html
patching file LayoutTests/fast/multicol/newmulticol/fixed-height-fill-balance-2.html

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force', '--reviewer', u'David Hyatt']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Full output: http://webkit-queues.appspot.com/results/6444814115536896
Comment 7 Morten Stenshorne 2014-01-20 00:59:52 PST 
Created attachment 221630 [details]
Patch
Comment 8 Morten Stenshorne 2014-01-20 01:16:13 PST 
Created attachment 221632 [details]
Patch
Comment 9 Dave Hyatt 2014-01-20 11:34:10 PST 
Comment on attachment 221632 [details]
Patch

r=me
Comment 10 WebKit Commit Bot 2014-01-20 11:51:38 PST 
Comment on attachment 221632 [details]
Patch

Clearing flags on attachment: 221632

Committed r162366: <http://trac.webkit.org/changeset/162366>
Comment 11 WebKit Commit Bot 2014-01-20 11:51:41 PST 
All reviewed patches have been landed.  Closing bug.