RESOLVED FIXED86441
CSS3 Multicolumn: Content in normal flow extending into column gaps should be clipped at middle of column-gap 
https://bugs.webkit.org/show_bug.cgi?id=86441
Summary CSS3 Multicolumn: Content in normal flow extending into column gaps should be...
Attachments
testcase (1.40 KB, text/html)
2012-05-14 22:12 PDT, Uday Kiran 		no flags
Details
Proposed Patch (6.88 KB, patch)
2012-05-14 22:44 PDT, Uday Kiran 		no flags
DetailsFormatted DiffDiff
Archive of layout-test-results from ec2-cr-linux-02 (836.07 KB, application/zip)
2012-05-15 00:27 PDT, WebKit Review Bot 		no flags
Details
Patch (7.25 KB, patch)
2012-05-15 04:30 PDT, Uday Kiran 		no flags
DetailsFormatted DiffDiff
Archive of layout-test-results from ec2-cr-linux-03 (4.28 MB, application/zip)
2012-05-15 04:51 PDT, WebKit Review Bot 		no flags
Details
Fixed testcase (1.69 KB, text/html)
2012-05-16 00:09 PDT, Uday Kiran 		no flags
Details
Patch with fixed testcase (7.09 KB, patch)
2012-05-16 00:40 PDT, Uday Kiran 		no flags
DetailsFormatted DiffDiff
Archive of layout-test-results from ec2-cr-linux-04 (842.22 KB, application/zip)
2012-05-16 04:39 PDT, WebKit Review Bot 		no flags
Details
Updated Patch (11.33 KB, patch)
2012-05-16 07:09 PDT, Uday Kiran 		no flags
DetailsFormatted DiffDiff
Rebased Patch (11.32 KB, patch)
2012-05-16 21:36 PDT, Uday Kiran 		no flags
DetailsFormatted DiffDiff
Updated Patch (11.33 KB, patch)
2012-05-17 00:49 PDT, Uday Kiran 		no flags
DetailsFormatted DiffDiff
Rebased Patch (11.37 KB, patch)
2012-05-17 00:59 PDT, Uday Kiran 		no flags
DetailsFormatted DiffDiff
Show Obsolete (7) View All Add attachment proposed patch, testcase, etc.
Uday Kiran
Comment 1 2012-05-14 22:44:59 PDT
Created attachment 141856 [details] Proposed Patch
Vivek Galatage
Comment 2 2012-05-15 00:16:48 PDT
Comment on attachment 141856 [details] Proposed Patch View in context: https://bugs.webkit.org/attachment.cgi?id=141856&action=review > LayoutTests/fast/multicol/overflow-into-columngap-expected.html:1 > +<!DOCTYPE html> Some extra characters seen. Are these intentional characters? Also should the expected file be txt file or html file?
WebKit Review Bot
Comment 3 2012-05-15 00:27:49 PDT
Comment on attachment 141856 [details] Proposed Patch Attachment 141856 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12681747 New failing tests: fast/borders/border-antialiasing.html fast/multicol/shadow-breaking.html fast/multicol/vertical-rl/border-padding-pagination.html fast/multicol/vertical-lr/border-padding-pagination.html
WebKit Review Bot
Comment 4 2012-05-15 00:27:53 PDT
Created attachment 141875 [details] Archive of layout-test-results from ec2-cr-linux-02 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-02 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Uday Kiran
Comment 5 2012-05-15 04:30:38 PDT
Created attachment 141915 [details] Patch For checking EWS
WebKit Review Bot
Comment 6 2012-05-15 04:51:44 PDT
Comment on attachment 141915 [details] Patch Attachment 141915 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12679910 New failing tests: fast/borders/border-antialiasing.html http/tests/security/sandboxed-iframe-origin-add.html fast/multicol/span/anonymous-split-block-crash.html fast/multicol/positive-leading.html fast/multicol/span/span-as-immediate-child-complex-splitting.html fast/loader/text-document-wrapping.html fast/multicol/span/anonymous-style-inheritance.html fast/multicol/vertical-rl/border-padding-pagination.html fast/multicol/span/span-as-immediate-child-generated-content.html fast/multicol/span/span-as-immediate-columns-child.html fast/multicol/span/span-as-nested-columns-child-dynamic.html fast/line-grid/line-grid-into-columns.html fast/multicol/span/clone-anonymous-block-non-inline-child-crash.html fast/multicol/vertical-lr/column-rules.html fast/multicol/span/anonymous-before-child-parent-crash.html fast/canvas/webgl/shader-precision-format.html fast/multicol/span/span-as-immediate-columns-child-dynamic.html fast/multicol/span/generated-child-split-flow-crash.html fast/multicol/nested-columns.html fast/multicol/single-line.html fast/multicol/column-rules-stacking.html fast/multicol/vertical-lr/nested-columns.html fast/multicol/vertical-lr/border-padding-pagination.html fast/multicol/column-rules.html fast/multicol/overflow-unsplittable.html fast/multicol/shadow-breaking.html fast/loader/javascript-url-in-object.html fast/multicol/span/span-margin-collapsing.html fast/multicol/span/span-as-nested-columns-child.html http/tests/inspector/resource-har-pages.html
WebKit Review Bot
Comment 7 2012-05-15 04:51:55 PDT
Created attachment 141917 [details] Archive of layout-test-results from ec2-cr-linux-03 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-03 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Uday Kiran
Comment 8 2012-05-16 00:09:54 PDT
Created attachment 142167 [details] Fixed testcase Fixed test case. Previously attached test case was about clipping content when overflow outside multi-column element. This test case is about overflowing into column-gap.
Uday Kiran
Comment 9 2012-05-16 00:40:15 PDT
Created attachment 142179 [details] Patch with fixed testcase
WebKit Review Bot
Comment 10 2012-05-16 04:39:31 PDT
Comment on attachment 142179 [details] Patch with fixed testcase Attachment 142179 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12718157 New failing tests: fast/borders/border-antialiasing.html fast/multicol/shadow-breaking.html
WebKit Review Bot
Comment 11 2012-05-16 04:39:35 PDT
Created attachment 142222 [details] Archive of layout-test-results from ec2-cr-linux-04 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-04 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Uday Kiran
Comment 12 2012-05-16 07:09:15 PDT
Created attachment 142252 [details] Updated Patch Failing tests need to be rebased.
Uday Kiran
Comment 13 2012-05-16 21:36:47 PDT
Created attachment 142410 [details] Rebased Patch
Eric Seidel (no email)
Comment 14 2012-05-17 00:05:59 PDT
Seems reasonable, but should you be using LayoutUnits here instead?
Uday Kiran
Comment 15 2012-05-17 00:49:46 PDT
Created attachment 142435 [details] Updated Patch Thanks for reviewing Eric. Adding Hyatt in cc for reviewing this multicolumn patch.
Uday Kiran
Comment 16 2012-05-17 00:59:17 PDT
Created attachment 142437 [details] Rebased Patch
Eric Seidel (no email)
Comment 17 2012-05-17 01:00:51 PDT
Comment on attachment 142437 [details] Rebased Patch Hopefully this will do the right thing. :)
WebKit Review Bot
Comment 18 2012-05-17 01:13:49 PDT
Comment on attachment 142437 [details] Rebased Patch Clearing flags on attachment: 142437 Committed r117412: <http://trac.webkit.org/changeset/117412>
WebKit Review Bot
Comment 19 2012-05-17 01:13:56 PDT
All reviewed patches have been landed. Closing bug.
Andy Estes
Comment 20 2012-05-23 16:57:50 PDT
<rdar://problem/11520511>
Note You need to log in before you can comment on or make changes to this bug.