Bug 16723 - tables/mozilla/bugs/bug30418.html test failing due to problems updating dynamic border rules
Summary: tables/mozilla/bugs/bug30418.html test failing due to problems updating dynam...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tables (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Darin Adler
URL:
Keywords:
: 16724 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-01-03 10:16 PST by Darin Adler
Modified: 2008-01-03 17:18 PST (History)
1 user (show)

See Also:


Attachments
patch in progress (8.31 KB, patch)
2008-01-03 10:16 PST, Darin Adler
no flags Details | Formatted Diff | Diff
patch (9.29 KB, patch)
2008-01-03 16:13 PST, Darin Adler
no flags Details | Formatted Diff | Diff
patch (with tests this time; oops!) (28.22 KB, patch)
2008-01-03 17:03 PST, Darin Adler
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Adler 2008-01-03 10:16:22 PST
There's a table regression tests failing intermittently. I know why. More details to come.
Comment 1 Darin Adler 2008-01-03 10:16:46 PST
Created attachment 18253 [details]
patch in progress
Comment 2 David Kilzer (:ddkilzer) 2008-01-03 10:39:52 PST
Is this LayoutTests/tables/mozilla/bugs/bug30418.html?  If so, Bug 16724 is a duplicate.

Comment 3 Darin Adler 2008-01-03 13:17:59 PST
(In reply to comment #2)
> Is this LayoutTests/tables/mozilla/bugs/bug30418.html?  If so, Bug 16724 is a
> duplicate.

Yes.
Comment 4 Darin Adler 2008-01-03 13:18:14 PST
*** Bug 16724 has been marked as a duplicate of this bug. ***
Comment 5 Darin Adler 2008-01-03 13:18:38 PST
The checked-in result for this test used to omit borders. I checked in a result that has the borders, but it's only intermittently working.
Comment 6 Darin Adler 2008-01-03 16:13:35 PST
Created attachment 18255 [details]
patch
Comment 7 Darin Adler 2008-01-03 17:03:58 PST
Created attachment 18257 [details]
patch (with tests this time; oops!)
Comment 8 mitz 2008-01-03 17:13:18 PST
Comment on attachment 18257 [details]
patch (with tests this time; oops!)

r=me
Comment 9 Darin Adler 2008-01-03 17:18:33 PST
Committed revision 29138.