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

Status: RESOLVED FIXED

(Core :: Widget, defect)

Description

[foudfou]

While writing a patch for Bug 760802, I wasn't aware of the ongoing effort for removing  |enablePrivilege| in mochitest (Bug 462483)

Comment 1

[foudfou]

Attachment: convert test from plain to chrome

Comment 2

[foudfou]

Try'd: https://tbpl.mozilla.org/?tree=Try&rev=bde8203eeb5e

Comment 3

[Martijn Wargers (dead)]

Isn't XUL going to disappear also?

Comment 4

[Robert O'Callahan (:roc) (email my personal email if necessary)]

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.

Comment 5

[foudfou]

Attachment: replace dump() with $("display").innerHTML

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/190cd1d48a94

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/190cd1d48a94