Allows for setTimeout and setInterval to take a string as the first argument

[matthewfl]

SetTimeout and SetInterval should be able to take a string as well as function as their first argument.
https://developer.mozilla.org/en/DOM/window.setTimeout

[ry]

Okay - can you add a test?

[matthewfl]

I added test for this, and I also rebased/reworked this for the changes you made in moving the timers code.

[aikar]

Wasn't this discussed on -dev and most people agreed it shouldn't be added since it was a mistake in the first place to allow strings in the browser?

[matthewfl]

I posted this to the mailing list. There was a lot of back and forth, but I don't think that any official decision came out of it. I am thinking that this is not going to get merged into node, seeing how long it has been here, but it is still here, in case there is some change of mind.

[ry]

rebase please.

[ry]

no need to change wscript.

[Ayms]

If I understand correctly it was decided not to allow strings for timers in nodejs.

Even if it is not good and browsers mistakes, it is used by developpers, so I would think this very small change should be in nodejs or at least should not return an error.

The subject was rapidly discussed here too : https://github.com/tmpvar/jsdom/issues/231 --> the fact that strings are not taken into account provokes third parties scripts (websites scripts loaded in jsdom) execution failure, then global failure if we don't prevent it.

Changes could be made in jsdom for example but probably it would be better that it is in nodejs, thanks to let us know your position.

[Ayms]

@ry @MatthewF I don't know if you saw that comment, I would suggest to fix it, very easy to fix compared to problems that it can cause