Bug 74636 – td element ignores zero width/height input element

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

Created attachment 119472 [details]
demonstrates the bug

If you have an input element with style width=0px and height=0px inside a td element, webkit will give the td element the incorrect size (ignoring the zero width and height). If you inspect the element you'll see that the input element is correctly sized, it is just the td element that has the wrong size. Setting the width and height to 1px fixes the problem. The problem also only occurs with input elements...if you try it with a <P> it works fine.

The bug occurs on Chrome and Safari on both OS X and Windows. Firefox and IE work fine.

Oh, and please don't tell me I shouldn't be using zero sized inputs...there is a good reason for it.

[Florin Malita]

The strict inequality in RenderTextControl::computePreferredLogicalWidths ignores width == 0 attributes and falls back on estimating a default preferred width:

    if (style()->width().isFixed() && style()->width().value() > 0)
        m_minPreferredLogicalWidth = m_maxPreferredLogicalWidth = computeContentBoxLogicalWidth(style()->width().value());
    else {
        ...

The following patch relaxes the test.

[Florin Malita]

Created attachment 119909 [details]
Patch

[Kent Tamura]

Comment on attachment 119909 [details]
Patch

The code change looks good.
Can you convert the test to a ref test?
http://trac.webkit.org/wiki/Writing%20Reftests

[Florin Malita]

(In reply to comment #3)
> Can you convert the test to a ref test?
> http://trac.webkit.org/wiki/Writing%20Reftests

Will do.

There's still an off-by-one issue lurking (unless I'm missing some styling interference): the calculated enclosing <td> width seems to be 1. I was planning to investigate that separately, but to get a correct ref test I think I'll have to tackle it upfront.

[Florin Malita]

(In reply to comment #4)
> There's still an off-by-one issue lurking (unless I'm missing some styling interference): the calculated enclosing <td> width seems to be 1.

Tracked it down to AutoTableLayout::recalcColumn():

                if (cell->colSpan() == 1) {
                    // A cell originates in this column.  Ensure we have
                    // a min/max width of at least 1px for this column now.
                    columnLayout.minLogicalWidth = max<int>(columnLayout.minLogicalWidth, cellHasContent ? 1 : 0);
                    columnLayout.maxLogicalWidth = max<int>(columnLayout.maxLogicalWidth, 1);


I can't say that I understand the reason, but it's definitely intentional so I'll leave it alone (unless someone thinks it's wrong). I'll work around it in the ref test.

[Florin Malita]

Created attachment 120033 [details]
Patch

Updated reftest.

[Kent Tamura]

Comment on attachment 120033 [details]
Patch

Looks good.

[WebKit Review Bot]

Comment on attachment 120033 [details]
Patch

Clearing flags on attachment: 120033

Committed r103377: <http://trac.webkit.org/changeset/103377>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.