377731 - [meta] pass style system mochitests (without asserting)

Status: NEW

(Core :: CSS Parsing and Computation, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I'm writing a bunch of mochitests to test the style system.  I've been filing bugs on some of the failures, but I'm not keeping up with the bug filing that needs to be done, never mind the fixing.

So this bug is a meta-bug on getting us to pass all of the tests in layout/style/test/ that don't have bug numbers in the test filenames.  Dependency bugs should be added as they are filed.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: some one-line patches

I said I'd file separate bugs, but these are one-liners that don't seem worthy of their own bug reports:
 * serialize 'cue' shorthand in the correct order
 * serialize 'pause' shorthand in the correct order
 * accept 'none' for 'speak-punctuation' (this is all that's needed since we don't compute values for any of the aural properties)
 * accept 'none' followed by a cue-after value in 'cue' parsing
 * don't check '-moz-box-orient: inherit' when computing '-moz-box-flex'

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 262084 [details] [diff] [review]
some one-line patches

Looks good.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 262084 [details] [diff] [review]
some one-line patches

Checked in to trunk.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

We also need to look at the assertions that fire during the tests.