Bug 126646
| Summary: | Subpixel layout: desiredColumnWidth in RenderBlock::calcColumnWidth() needs flooring. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | alan <zalan> |
| Component: | Layout and Rendering | Assignee: | alan <zalan> |
| Status: | RESOLVED INVALID | ||
| Severity: | Normal | CC: | jonlee, simon.fraser, webkit-bug-importer |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 126283 | ||
alan
LayoutUnit desiredColumnWidth = std::max<LayoutUnit>(0, ((availWidth - ((desiredColumnCount - 1) * colGap)) / desiredColumnCount)) produces different CSS pixel results when subpixel is off.
failed case:
fast/events/document-elementFromPoint.html
fast/multicol/break-properties.html
fast/multicol/column-break-with-balancing.html
fast/multicol/float-paginate-complex.html
fast/multicol/layers-in-multicol.html
fast/multicol/overflow-across-columns-percent-height.html
fast/multicol/overflow-across-columns.html
fast/multicol/overflow-unsplittable.html
fast/multicol/scrolling-overflow.html
fast/multicol/span/clone-flexbox.html
fast/multicol/span/clone-summary.html
fast/multicol/table-vertical-align.html
fast/multicol/vertical-lr/break-properties.html
fast/multicol/vertical-lr/column-break-with-balancing.html
fast/multicol/vertical-lr/float-multicol.html
fast/multicol/vertical-rl/break-properties.html
fast/multicol/vertical-rl/column-break-with-balancing.html
fast/multicol/vertical-rl/float-multicol.html
fast/multicol/newmulticol/column-rules-fixed-height.html
fast/multicol/newmulticol/columns-shorthand-parsing.html
fast/multicol/newmulticol/layers-in-multicol.html
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
alan
If we start snapping columns widths to CSS pixels, we lose subpixels while layouting the columns content.
Simon Fraser (smfr)
Need to talk to Dave Hyatt.
alan
Matches FF(v26) rendering.