Re: [dev-servo] Categorizing Stylo reftest failures

On 2017-08-10 1:55 AM, James Graham wrote: On 10/08/17 02:53, Manish Goregaokar wrote: https://gist.github.com/Manishearth/086118c940ff86a6cfc573f53c508279 FWIW I made a list of the differences between reftest results in web-platform-tests with stylo enabled vs not (based on the expectation

Re: [dev-servo] Categorizing Stylo reftest failures

On 10/08/17 02:53, Manish Goregaokar wrote: https://gist.github.com/Manishearth/086118c940ff86a6cfc573f53c508279 FWIW I made a list of the differences between reftest results in web-platform-tests with stylo enabled vs not (based on the expectation metadata files). Paths are relative to the

Re: [dev-servo] Categorizing Stylo reftest failures

https://gist.github.com/Manishearth/086118c940ff86a6cfc573f53c508279 I've now gone through the failures and categorized them as yes/no/maybe based on whether they should block shipping. I've filed bugs for and partially investigated all of the "yes" ones (and fixed some of them). Some of these

Re: [dev-servo] Categorizing Stylo reftest failures

We aren't! I'm just not seeing that bug in the reftests outside of audio and video, and TYLin is already looking into it anyway. Thanks, -Manish Goregaokar On Mon, Jul 24, 2017 at 8:20 AM, Boris Zbarsky wrote: > On 7/24/17 1:53 AM, Manish Goregaokar wrote: >> >> and the

Re: [dev-servo] Categorizing Stylo reftest failures

On 7/24/17 1:53 AM, Manish Goregaokar wrote: and the styloVsGecko / issue (which does not block landing since it's a slight discrepancy between stylo and gecko) Looks like it's a difference in vertical alignment. How sure are we that this won't affect web pages outside of media controls?

Re: [dev-servo] Categorizing Stylo reftest failures

Now that we've fixed most of these I went through it again and checked stuff. Filtering out scoped style and web components failures we only have a hundred tests left I recategorized most of them: https://gist.github.com/Manishearth/086118c940ff86a6cfc573f53c508279 The major issues right now are

Re: [dev-servo] Categorizing Stylo reftest failures

And, it's done! All reftest failures classified. Permalink to current revision: https://gist.github.com/Manishearth/086118c940ff86a6cfc573f53c508279/a181e327a4e2d83d75e28862d1fc01e121923bff Bear in mind that the try run this is based on is a few days old so some of the things there may be fixed

Re: [dev-servo] Categorizing Stylo reftest failures

First link in the file! https://treeherder.mozilla.org/#/jobs?repo=try=bff0878649f10db8946d50e760b0d13f0f3be9be Also, each section in the file already links to the analyzer results for that subset of the tests. -Manish Goregaokar ___ dev-servo mailing