Closed Bug 1316172 Opened 4 years ago Closed 4 years ago

Add reftests for logical properties


(Core :: CSS Parsing and Computation, defect)

Not set



Tracking Status
firefox52 --- fixed


(Reporter: manishearth, Assigned: manishearth)




(2 files)

Attached image Screenshot of test
Test passes locally.
Tests for logical properties should probably belong to css-logical-props rather than css-writing-modes. Also I guess jfkthame is a better reviewer for this patch.
The specific part of the spec it tests is in writing-modes though.
Probably not, though I can tweak that easily.
Comment on attachment 8808836 [details]
Bug 1316172 - Add a reftest for logical properties;

Looks good, except for one comment:

::: layout/reftests/w3c-css/submitted/writing-modes-3/reftest-stylo.list:18
(Diff revision 3)
>  == text-combine-upright-compression-006a.html text-combine-upright-compression-006a.html
>  == text-combine-upright-compression-007.html text-combine-upright-compression-007.html
>  == text-orientation-upright-directionality-001.html text-orientation-upright-directionality-001.html
> +
> +== logical-physical-mapping-001.html logical-physical-mapping-001-ref.html

This is wrong, since the reftest-stylo.list files repeat the test twice and don't use the reference.  They also say "DO NOT EDIT", which I suspect should be honored.
Attachment #8808836 - Flags: review?(dbaron) → review+
> repeat the test twice and don't use the reference

good point, fixed.

> They also say "DO NOT EDIT".

That's a lie :) It's fine to edit to add new tests, and we're not relying on these in stylo-land yet (using them as general smoketests to ensure that things are mostly still working and that there are no major regressions -- but individual test failures are still okay)

Marking as == in stylo since the relevant stuff for this to work in stylo will land soon.
Pushed by
Add a reftest for logical properties; r=dbaron
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Oops, the import gave:

remote: vendor-imports/mozilla/mozilla-central-reftests/writing-modes-3/logical-physical-mapping-001.html status changed to 'Needs Work' due to error:
remote: Title not specified. Linked to unversioned specification URL Use or instead.
Flags: needinfo?(manishearth)
Would you mind landing the fix so it gets picked up on the next import?
Flags: needinfo?(manishearth)
Depends on: 1317092
Hmm, I think you just need to land a followup commit directly.
Flags: needinfo?(manishearth)
You need to log in before you can comment on or make changes to this bug.