Handle additional arguments to setTimeout in Timer.jsm

RESOLVED FIXED in mozilla22

Status

()

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: mbrubeck, Assigned: mbrubeck)

Tracking

Trunk
mozilla22
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 720223 [details] [diff] [review]
patch

window.setTimeout takes additional optional parameters and passes them to the callback function.  This patch makes Timer.jsm match this behavior.
Attachment #720223 - Flags: review?(neil)

Comment 1

6 years ago
Comment on attachment 720223 [details] [diff] [review]
patch

Well at least we don't have to emulate the lateness argument ;-)

>+  let args = Array.prototype.slice.call(arguments, 2);
Array.slice(arguments, 2) suffices.
Attachment #720223 - Flags: review?(neil) → review+

Comment 2

6 years ago
https://hg.mozilla.org/mozilla-central/rev/412dc022e86e
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.