nsTimerImpl does not really need mCallbackDuringFire

RESOLVED FIXED in Firefox -esr60

Status

()

defect
RESOLVED FIXED
a month ago
26 days ago

People

(Reporter: bwc, Assigned: bwc)

Tracking

unspecified
mozilla68
Points:
---

Firefox Tracking Flags

(firefox-esr60 fixed, firefox67 fixed, firefox68 fixed)

Details

Attachments

(2 attachments)

It does not seem that the extra complexity to avoid refcount manipulation (by swapping) is worth it here. mCallbackDuringFire really ought to be a stack copy of mCallback.

Assignee: nobody → docfaraday

Comment 3

a month ago
Pushed by bcampen@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bd62ad8bf26d
Simplify this callback management stuff a little. r=froydnj

Comment 4

a month ago
bugherder
Status: NEW → RESOLVED
Last Resolved: a month ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.