Summary: | [GTK] Layout Test mathml/presentation/bug95404.html is failing | ||
---|---|---|---|
Product: | WebKit | Reporter: | Simon Pena <spenap> |
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | cfleizach, fred.wang, gur.trio, zan |
Priority: | P2 | Keywords: | Gtk, LayoutTestFailure |
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Bug Depends on: | 116600 | ||
Bug Blocks: |
Description
Simon Pena
2013-07-19 07:08:52 PDT
Do you have screenshots of the failing test? It worked for me on GTK and when I submitted it to the review queue. (In reply to comment #1) > Do you have screenshots of the failing test? > > It worked for me on GTK and when I submitted it to the review queue. Hi Frédéric, you can take a look here: http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release/r152777%20%2839211%29/results.html. I am not familiar with MathML, and I don't know how to interpret the results, but I'll be happy to help you in anything I can to get this test unskipped again. (In reply to comment #2) > I am not familiar with MathML, and I don't know how to interpret the results, but I'll be happy to help you in anything I can to get this test unskipped again. This is a reftest, the pages mathml/presentation/bug95404.html and mathml/presentation/bug95404-expected.html should render exactly the same. By the naked eye I don't see any differences between the two and diff.png is just black. But it's possible that there is a very small pixel difference that one can not see by the naked eye... (In reply to comment #3) > (In reply to comment #2) > > I am not familiar with MathML, and I don't know how to interpret the results, but I'll be happy to help you in anything I can to get this test unskipped again. > > This is a reftest, the pages > > mathml/presentation/bug95404.html > > and > > mathml/presentation/bug95404-expected.html > > should render exactly the same. By the naked eye I don't see any differences between the two and diff.png is just black. But it's possible that there is a very small pixel difference that one can not see by the naked eye... Yes, you're right. I will ask someone more knowledgeable than me with the bots to see if there's a way to treat this as a pass or if there's a way to make that small difference go away by maybe modifying the reftest. (In reply to comment #4) > Yes, you're right. > > I will ask someone more knowledgeable than me with the bots to see if there's a way to treat this as a pass or if there's a way to make that small difference go away by maybe modifying the reftest. OK thanks. If the latter way is necessary, I can help to modify the reftest later. There's a minimal difference appearing in the top left corner due to the square root sign overflowing the red box at the very bottom. There are a couple of tests already failing in similar fashion, though I can't at the moment remember the specific ones. Still, the cause is probably the same, so a master bug for these test failures should be created. The reference tests only pass if the test output matches the reference output 100%, so there's no real way to make this test seemingly pass unless the outputs match completely. As for the problem itself, I think it could be font-related. Comment #1 makes me think that there are still some specifics that are leaking through the Jhbuild environment configuration. (In reply to comment #6) > There's a minimal difference appearing in the top left corner due to the square root sign overflowing the red box at the very bottom. OK, that's what I suspected. This will probably pass after bug 116600 is fixed. @Simon: can you try again to run the test now that bug 116600 is fixed? (In reply to comment #8) > @Simon: can you try again to run the test now that bug 116600 is fixed? Actually there is another square root that is visible so bug 116600 does not solve the problem. One workaround is modify the test to paint the radical in white (but its content still in black). Manually committed r165889: http://trac.webkit.org/changeset/165889 I added a 2px border around the rectangles to prevent the remaining failure due to antialiasing. |