689615 - string as first argument of setTimeout throw an exception in content script

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect, P2)

Description

[Alexandre Poirot [:ochameau] [PTO back August 21]]

Currently, if we try to use setTimeout in a content script with a string instead of a function, we receive this incomprehensible exception:
Traceback (most recent call last):
   File "resource://jid1-9krhwinmiaqbia-at-jetpack-api-utils-lib/content/worker.js", line 108, in
     callback.apply(null, params);
TypeError: callback.apply is not a function

We should at least throw a meaningfull message or implement string.

Comment 1

[Gabor Krizsanits (INACTIVE)]

I cannot reproduce this.

Comment 2

[Tomislav Jovanovic :zombie]

Attachment: Link to pull request 1281

Comment 3

[[github robot]]

Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/c3a26bd603a36407481cbf19925333fd15fd715e
bug 689615 - setTimeout works with string arg.

and some tests too..

https://github.com/mozilla/addon-sdk/commit/004a50706f2f4333359470249b7393f226f2dad5
Merge pull request #1283 from zombie/689615-setTimeout-str-arg

fix bug 689615 - setTimeout works with string arg. r=@jsantell