[Webkit-unassigned] [Bug 24413] border of <td> not displayed when border-color and border-style are used in CSS

[bugzilla-daemon at webkit.org]

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


Mustafizur Rahaman <mrahaman at innominds.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|mustaf.here at gmail.com       |kbolisetty at innominds.com,
                   |                            |mrahaman at innominds.com




--- Comment #3 from Mustafizur Rahaman <mrahaman at innominds.com>  2012-03-05 03:19:39 PST ---
I tried to look at this issue again as it is still reproducible. The problem is if we want to specify border properties for a <td> inside <table>, mentioning border-color & border-style is not sufficient. We have to explicitly specify border-width, too.

The reason being, in CSSStyleSelector::matchAllRules(), we try to add additional attribute style for table & table cell, and if no border-width is mentioned, the border-widths are all set to zero (in HTMLTableElement::createSharedCellStyle())

When we mention style property as "border: blue dashed", the border-width is assigned default value(while handling CSSPropertyBorder in CSSParser::parseValue()). So, even if the table cell sets the border-width as zero, the border-width from the CSSParser gets assigned to the Style element & therefore, the border is drawn properly. I have provided a summary of my analysis here, but I can discuss in details, if required.

Therefore, my question is:
#1> Do i need to explicitly mention border-width for <td> inside <table> i.e. is the given test case an invalid one?
#2> What is the need for the code we have in HTMLTableElement::createSharedCellStyle()? If we don't call that logic, the test is passing fine.
#3> It is working fine for FF.

Any help is greatly appreciated so that I can take this to logical closure...

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