Open Bug 1134069 Opened 5 years ago Updated 2 months ago

Submit appropriate CSS Ruby tests to W3C test suite

Categories

(Core :: Layout: Ruby, defect)

defect
Not set

Tracking

()

People

(Reporter: xidorn, Assigned: xidorn, NeedInfo)

References

(Blocks 1 open bug)

Details

(Keywords: leave-open)

Attachments

(2 files)

Per bug 1039006 comment 6, we should submit appropriate tests for CSS Ruby to W3C's test suite. Most of them may simply be done via moving them to layout/reftests/w3c-css/submitted/ruby/.
Let's do it step by step.
Keywords: leave-open
Attachment #8565833 - Flags: review?(dbaron)
Attachment #8565834 - Flags: review?(dbaron)
Attachment #8565833 - Flags: review?(dbaron) → review+
Comment on attachment 8565834 [details] [diff] [review]
patch 2 - inlinize blocks

r=dbaron

(Though one suggestion dholbert had was using "CSS Reference" instead of "CSS Test" in the title of the reference.)
Attachment #8565834 - Flags: review?(dbaron) → review+
Attachment #8565833 - Flags: checkin+
Attachment #8565834 - Flags: checkin+
Component: Layout → Layout: Ruby

The leave-open keyword is there and there is no activity for 6 months.
:svoisen, maybe it's time to close this bug?

Flags: needinfo?(svoisen)
Flags: needinfo?(svoisen) → needinfo?(xidorn+moz)

Majority of the tests under layout/reftests/css-ruby can probably be moved. Some of the tests were not written in a way which I feel satisified to submit, but I guess that's fine. Having the tests shared is better than nothing... I'll try to find a time to do that next week or so.

The leave-open keyword is there and there is no activity for 6 months.
:svoisen, maybe it's time to close this bug?

Flags: needinfo?(svoisen)

Xidorn: Do you by chance have time to finish this up? Otherwise I'll find someone to finish the move. Feels like this is still worthwhile to have.

Flags: needinfo?(svoisen)

It's still worth to have, but I don't think this is super urgent though. I don't think any other implementation has plan to move forward on their ruby implementation this year (Koji said Chrome team is considering this some time next year).

I'll try to find time to go through the tests and submit them within this year.

You need to log in before you can comment on or make changes to this bug.