Not clear if this is a test problem or if the new multi-column layout still has a bug. Will need to investigate the test to find out.
Test is fine. Balancing is getting confused by overflow, so this is a legitimate bug.
<rdar://problem/16656277>
This is a blocker for turning the new columns on.
Created attachment 229931 [details] Patch
overflow-content.html is flaky and the new one in newmulticol/compare-with-old-impl/ will need the fixes for bug 113781 and bug 113058 in order to pass reliably.
Comment on attachment 229931 [details] Patch r=me
Comment on attachment 229931 [details] Patch Clearing flags on attachment: 229931 Committed r167707: <http://trac.webkit.org/changeset/167707>
All reviewed patches have been landed. Closing bug.
This made fast/multicol/newmulticol/compare-with-old-impl/overflow-content.html fail most of the time on several bots: http://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=fast%2Fmulticol%2Fnewmulticol%2Fcompare-with-old-impl%2Foverflow-content.html http://build.webkit.org/results/Apple%20MountainLion%20Release%20WK1%20(Tests)/r167711%20(19379)/fast/multicol/newmulticol/compare-with-old-impl/overflow-content-diffs.html
(In reply to comment #5) > overflow-content.html is flaky and the new one in newmulticol/compare-with-old-impl/ will need the fixes for bug 113781 and bug 113058 in order to pass reliably. I'm not quite sure what this means. Did you intentionally make a test fail, and not even mark it accordingly in TestExpectations?
This shouldn't have landed before the fix for bug 113058, but it did. Once that one has landed, the test will become stable.
This is my mistake. EWS was green so I just changed to cq+ without reading.
Anyway can land the other fix and then it will be fine.