Bug 55474

Summary: Table outlines are not drawn
Product: WebKit Reporter: Abhishek Arya <inferno>
Component: TablesAssignee: Nobody <webkit-unassigned>
Severity: Minor CC: hyatt, jamesr, mitz, trobinson
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
Patch hyatt: review+

Description Abhishek Arya 2011-03-01 09:18:01 PST
Currently we are not painting outlines for tables.
Comment 1 Abhishek Arya 2011-03-01 09:39:01 PST
Created attachment 84241 [details]
Comment 2 Abhishek Arya 2011-03-01 09:41:42 PST
note that table row outlines does not look supported in IE, opera and firefox draws it incorrectly (try adding a outline to tr in layouttest and try in firefox, you will see incorrect table row and table cell outline overlap). in our code, we do look to paint cell directly without calling rendertablerow paint. So, leaving my patch for checking table outlines and table cell outlines only.
Comment 3 Dave Hyatt 2011-03-01 13:56:17 PST
Comment on attachment 84241 [details]

Comment 4 Abhishek Arya 2011-03-01 14:18:58 PST
Committed r80040: <http://trac.webkit.org/changeset/80040>
Comment 5 Daniel Bates 2011-03-29 09:43:17 PDT
*** Bug 41030 has been marked as a duplicate of this bug. ***
Comment 6 Alexey Proskuryakov 2011-03-30 09:58:57 PDT
See also: bug 35813, bug 18179.
Comment 7 Abhishek Arya 2011-03-30 10:12:57 PDT
Yeah this 2 liner outline fix has to be added to a couple of Render* classes. I discovered this while fixing one of the security bugs. Also, rendering outline for table rows and columns is tricky because of the overlap problem in certain scenarios. bug 57439 on iframe should be easy enough (try adding these 2 lines to RenderWidget::paint(PaintInfo& paintInfo, int tx, int ty))