Closed
Bug 1047408
Opened 10 years ago
Closed 5 years ago
Remove nsTWeakRef
Categories
(Core :: XPCOM, defect)
Tracking
()
RESOLVED
FIXED
mozilla69
Tracking | Status | |
---|---|---|
firefox69 | --- | fixed |
People
(Reporter: smaug, Assigned: gsvelto)
References
Details
Attachments
(1 file)
I didn't even know we have such beast as nsTWeakRef. It is used very rarely, apparently only in some old plugin code, and recently in ServiceWorkers. We have nsWeakPtr and WeakPtr for com and non-com usecases, and those should be enough, and their API is better than odd nsTWeakRef.
Reporter | ||
Comment 1•10 years ago
|
||
Hopefully bug 1047398 will remove the usage from ServiceWorkers, and then we have to just remove usage from Windows plugin code. I don't have access to a Windows machine atm, but the change should be rather trivial.
Assignee | ||
Comment 2•5 years ago
|
||
I had already eyed this while working on bug 1503207.
Assignee: nobody → gsvelto
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•5 years ago
|
||
Pushed by gsvelto@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c347f8783421 Remove nsTWeakRef.h r=smaug
Comment 5•5 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox69:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
You need to log in
before you can comment on or make changes to this bug.
Description
•