462338 - Make executeSoon widely available

Status: RESOLVED FIXED

(Testing :: Mochitest, defect)

Description

[Dave Townsend [:mossop]]

Attachment: patch rev 1

This creates SimpleTest.executeSoon, copying it from the download tests. I'm not sure though whether we can edit SimpleTest or not.

The two tests I updated seem to be happyish. One of them shows two failures with or without my patch so I'm not sure what that is all about.

Ben you seem to be a peer for this, what say you?

Comment 1

[[e-mail] (:bhearsum)]

Comment on attachment 345481 [details] [diff] [review]
patch rev 1

Huh. I didn't know I was a peer for this. I'm totally not the right person to review this - maybe rob?

Comment 2

[Rob Campbell [:rc] (:robcee)]

Comment on attachment 345481 [details] [diff] [review]
patch rev 1

looks reasonable, no visible nits. Try it out!

Comment 3

[Dave Townsend [:mossop]]

Attachment: browser-chrome patch

Forgot to add this to browser chrome too, this is just an addition to the previous patch.

Comment 4

[Dave Townsend [:mossop]]

Pushed both patches: http://hg.mozilla.org/mozilla-central/rev/46a0fed0b3df

Comment 5

[Dave Townsend [:mossop]]

Landed on the 1.9 branch too.