1092264 - Fix testing of 'isolation' property

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Comment 1

[Rik Cabanier]

Attachment: Fix keyword in test

Comment 2

[Rik Cabanier]

try build: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=27553f066dab

Comment 3

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

(In reply to Rik Cabanier from comment #2)
> try build:
> https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=27553f066dab

You probably don't need an all-platform try build for that.  (Also, it doesn't seem like the try push will actually test anything useful given that you didn't push the pref change in that try push.)  See also https://secure.pub.build.mozilla.org/builddata/reports/reportor/daily/highscores/highscores.html ; we don't want to use resources unnecessarily, though we should certainly keep using what we need.

Honestly, I think you can just push this without a try push given that it's pref'd out.  But you'd want a try push of bug 1091885 *and* this patch before pushing bug 1091885.

Comment 4

[Max Vujovic]

Comment on attachment 8515140 [details] [diff] [review]
Fix keyword in test

https://hg.mozilla.org/integration/mozilla-inbound/rev/6dda3b5dc0a9

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6dda3b5dc0a9