Closed Bug 791975 Opened 7 years ago Closed 7 years ago

move test_bug760802.html from mochitest-plain to mochitest-chrome

Categories

(Core :: Widget, defect)

18 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla18

People

(Reporter: foudil.newbie+bmo, Assigned: foudil.newbie+bmo)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

While writing a patch for Bug 760802, I wasn't aware of the ongoing effort for removing  |enablePrivilege| in mochitest (Bug 462483)
Blocks: 462483
Depends on: 760802
Attachment #662130 - Flags: review?(roc)
Isn't XUL going to disappear also?
Comment on attachment 662130 [details] [diff] [review]
convert test from plain to chrome

Review of attachment 662130 [details] [diff] [review]:
-----------------------------------------------------------------

::: widget/tests/test_bug760802.html
@@ +29,5 @@
>  const Ci = Components.interfaces;
>  
> +function debug(msg) {
> +  dump("*** "+msg+"\n");
> +}

Take this out or at leas tcomment out th e'dump'.

I think it's fine to use XUL for now. We have lots of XUL tests and getting rid of it isn't a priority.
Assignee: nobody → foudil.newbie+bugzilla.mozilla.org
Attachment #662130 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #662130 - Flags: review?(roc)
Attachment #662530 - Flags: review?(roc)
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/190cd1d48a94
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in before you can comment on or make changes to this bug.