[Webkit-unassigned] [Bug 13598] <td> tag can cause Webkit to generate empty space

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=13598





--- Comment #26 from Arpita Bahuguna <arpitabahuguna at gmail.com>  2012-09-17 22:52:50 PST ---
Hi Julien, the last patch has fix for the following things:
1. percent width handling for colspan.
2. fixed width handling for colspan.
3. handling when colspan count is more than the number of spanning <td>s in the other row, introducing empty cells.

The fix for 
1. percent width handling for colspan, and
2. fixed width handling for colspan
cause the following two tests to fail but it is expected and we behave as per FF and Opera after the patch.
fast/table/003.html
tables/mozilla_expected_failures/bugs/bug80762-2.html
(this is the last but one patch)

The fix for
3. handling when colspan count is more than the number of spanning <td>s in the other row, introducing empty cells.
cause the following tests to fail which is again expected as we now follow similar to FF and Opera:
tables/mozilla/core/misc.html
tables/mozilla_expected_failures/bugs/bug1647.html
fast/table/colspanMinWidth-vertical.html
fast/table/colspanMinWidth.html
(this is the last patch)

Do you think I should first patch 1. & 2. (as mentioned above) and 3. as a separate patch?

Also. not only for this bug but for others as well, I am unable to patch the new expected results properly causing the cr-linux bot to always go red.
This seems to happen only for the tests that have the expected image in the cr-linux path as well as its fallback cr-win path whereas the .txt is only available in the cr-win fallback path.

I understand that I might be doing something wrong, but can you please tell me where I am going wrong as I have tried various combinations to get the patch right but have been unsuccessful so far.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.