Fix testing of 'isolation' property

RESOLVED FIXED in mozilla36

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: cabanier, Assigned: cabanier)

Tracking

Trunk
mozilla36
Points:
---
Bug Flags:
qe-verify -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

4 years ago
Created attachment 8515140 [details] [diff] [review]
Fix keyword in test
Attachment #8515140 - Flags: review?(dbaron)
(Assignee)

Updated

4 years ago
Blocks: 1091885
Attachment #8515140 - Flags: review?(dbaron) → review+
(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.
(Assignee)

Updated

4 years ago
Keywords: checkin-needed
(Assignee)

Updated

4 years ago
Assignee: nobody → cabanier

Updated

4 years ago
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/6dda3b5dc0a9
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.