stylo: Yet another reftest-stylo.list update

Assigned to



CSS Parsing and Computation
2 months ago
2 months ago


(Reporter: shinglyu, Assigned: shinglyu)


(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)


MozReview Requests


Submitter Diff Changes Open Issues Last Updated
Error loading review requests:


(1 attachment)



2 months ago
Sorry for yet another reftest-stylo.list change. The reftest expectation changed again. Looks like there are some load failures for no clear reason:

I skip them and it's all green except an Rs3 intermittent:

This blocks me from landing Bug 1344132 (again!). I'll land this first and address the unknown failures when I deal with 1351548.
Comment hidden (mozreview-request)


2 months ago
Assignee: nobody → slyu
Blocks: 1344132
No longer blocks: 1324348

Comment 2

2 months ago
Comment on attachment 8855610 [details]
Bug 1354404 - Update reftest-stylo.list expectations.

I guess the problem with these test loads in the fragment is that by using the same test and reference file, the second load ends up just scrolling the test file rather than loading a whole new document, and that's the reason we have some manually specified reference files in the reftest-stylo.list manifests that are different from the test file.  In your followup where you will get these tests unskipped again, maybe you could make the reftest harness just not automatically use the test file as the reference file if it finds either the test or reference has a URL fragment?  That would make it do a "normal" reftest comparison, rather than Gecko to Stylo, but at least we could still be running these tests.  (And the first one you've marked skip specifically helped me in my bug 1345695 investigations, so it would be good to have it re-enabled again soon.
Attachment #8855610 - Flags: review?(cam) → review+
Priority: -- → P1
Blocks: 1243581
You need to log in before you can comment on or make changes to this bug.