Closed Bug 1134598 Opened 10 years ago Closed 10 years ago

enable the vertical writing-mode reftest directory

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla38
Tracking Status
firefox39 --- fixed

People

(Reporter: jfkthame, Assigned: jfkthame)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

We don't need to wait for the pref to be flipped by default; now that we're building with vertical mode enabled in the code, we can just set the pref from within the reftest manifest, and begin running these tests regularly.
Attachment #8566485 - Flags: review?(smontagu)
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Try run with this patch: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a51d32b80d2b. I'm expecting 1131013-vertical-bidi.html to fail here, as it needs the followup patch (bug 1131013 comment 7) to work across all platforms.
Depends on: 1131013
Attachment #8566485 - Flags: review?(smontagu) → review+
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/338cb106bfc4 - you'll be wanting to spend rather a lot of time on try with debug builds, since at the very least ua-style-sheet-input-number-1.html has between zero and sixteen "Shouldn't be incomplete if availableBSize is UNCONSTRAINED" assertion failures.
Oh, drat. That's thanks to bug 1123299, which is also why that test was already marked as known-failing. I've disabled the offending part of the test for the time being (as we've also done in some other comparable cases), and will add a comment in bug 1123299 to remind us to restore it once it's implemented. Re-landed without that problem, I trust: https://hg.mozilla.org/integration/mozilla-inbound/rev/e37d57033b13
Ugh - sorry about that. :( Filed bug 1135622 for the unexpected failure, and restored the failure annotation that we had in the original landing (the one that was backed out for assertions). Tryserver says we should be ok now: https://treeherder.mozilla.org/#/jobs?repo=try&revision=fa27c631c2e7 (the failures there are a new test that's not actually present in the tree, or in this landing). So here we go again: https://hg.mozilla.org/integration/mozilla-inbound/rev/45d4cc092de1
Depends on: 1139646
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: