1821379 - Fix some tests related to content-visibility so that it can be enabled

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, task)

Description

[Daniel Holbert [:dholbert] (PTO until 2/24)]

See bug 1820058 comment 3; this bug is to fix up the tests mentioned there, to avoid unexpected test failures when content-visibility is enabled.

Comment 1

[Daniel Holbert [:dholbert] (PTO until 2/24)]

Attachment: Bug 1821379 part 1: Correct an obsolete 'content-visibility' keyword in the property_database.js file used by various mochitests. r?#layout-reviewers

We had 'invisible', but the correct keyword (and what we have implemented) is
'hidden'.

See list of values in the spec at
https://drafts.csswg.org/css-contain-2/#content-visibility

Comment 2

[Daniel Holbert [:dholbert] (PTO until 2/24)]

Attachment: Bug 1821379 part 2: Explicitly enable 'content-visibility' about:config pref for some WPT tests that use it, and update their expectations. r?#layout-reviewers

Depends on D172135

Comment 3

[Daniel Holbert [:dholbert] (PTO until 2/24)]

Attachment: Bug 1821379 part 3: Update a DOM test to make it aware of ContentVisibilityAutoStateChangeEvent. r?emilio

Depends on D172136

Comment 4

[Daniel Holbert [:dholbert] (PTO until 2/24)]

Try run (with another patch to pref content-visibility on, demonstrating that we don't need any other fixups here, I think):
https://treeherder.mozilla.org/jobs?repo=try&revision=76674b8231c98c7743f6aca7016a0d5a76d9b44b

Try run without the pref flip, to make sure I didn't inadvertently build in some expectation that the pref is enabled:
https://treeherder.mozilla.org/jobs?repo=try&revision=6cfe208522f985ac344b3322bd44bb6d0753c910
(This one just started, I'll be sure it looks OK before landing.)

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e2672521d0ac
part 1: Correct an obsolete 'content-visibility' keyword in the property_database.js file used by various mochitests. r=emilio
https://hg.mozilla.org/integration/autoland/rev/116ca9c61aec
part 2: Explicitly enable 'content-visibility' about:config pref for some WPT tests that use it, and update their expectations. r=emilio
https://hg.mozilla.org/integration/autoland/rev/fe4dcfdb64df
part 3: Update a DOM test to make it aware of ContentVisibilityAutoStateChangeEvent. r=emilio

Comment 6

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/e2672521d0ac
https://hg.mozilla.org/mozilla-central/rev/116ca9c61aec
https://hg.mozilla.org/mozilla-central/rev/fe4dcfdb64df