Summary: | [GTK][CAIRO] Some canvas arc tests fail because of a bug in Cairo arc rendering | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Zan Dobersek <zan> | ||||||
Component: | Canvas | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED DUPLICATE | ||||||||
Severity: | Normal | CC: | d-r, leandro, mdelaney7, mrobinson, otte, rakuco | ||||||
Priority: | P2 | Keywords: | Gtk, LayoutTestFailure | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | PC | ||||||||
OS: | Linux | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 54458 | ||||||||
Attachments: |
|
Created attachment 82480 [details]
Use smaller tolerance
This patch sets Cairo's tolerance to 0.01 and enables two canvas tests that now succeed.
This is a Cairo bug. I committed a testcase to the cairo testsuite at http://cgit.freedesktop.org/cairo/commit/?id=4d4056872db94573183473610ad1d81d5439fdc6 and hope our developers will take care of it there. Increasing the tolerance hides this bug (just increase the test size to get it back) at the cost of performance, so it's definitely not the right fix. (In reply to comment #2) > This is a Cairo bug. I committed a testcase to the cairo testsuite at http://cgit.freedesktop.org/cairo/commit/?id=4d4056872db94573183473610ad1d81d5439fdc6 and hope our developers will take care of it there. > > Increasing the tolerance hides this bug (just increase the test size to get it back) at the cost of performance, so it's definitely not the right fix. Thanks for the update, Benjamin. Is there a work-around (other than decreasing the tolerance) that we can make on our side? I'm guessing not. :( I can't think of a workaround, no. Comment on attachment 82480 [details]
Use smaller tolerance
Removing the review flag.
(In reply to comment #5) > (From update of attachment 82480 [details]) > Removing the review flag. Benjamin, is there a bug report somewhere so we can track this and unskip the tests once the bots have the appropriate version of Cairo? (In reply to comment #6) > (In reply to comment #5) > > (From update of attachment 82480 [details] [details]) > > Removing the review flag. > > Benjamin, is there a bug report somewhere so we can track this and unskip the tests once the bots have the appropriate version of Cairo? A test case has already been committed into cairo tree[1], but I do not think this has been fixed yet. [1] http://cgit.freedesktop.org/cairo/commit/?id=4d4056872db94573183473610ad1d81d5439fdc6 |
Created attachment 82473 [details] Comparison image There are two tests that fail because Cairo's preset tolerance is too big. The tests are: -canvas/philip/tests/2d.path.arc.selfintersect.1.html -canvas/philip/tests/2d.path.arc.shape.3.html Both tests draw arcs, but arcs in Cairo are drawn by being split into segments. Because of the default tolerance being too big these segments are not precise enough (there are too many and are too big) so they end up expanding the arc furthermore, making these tests fail. I'm attaching a comparison image that furthermore illustrates the problem.