Closed
Bug 1447529
Opened 6 years ago
Closed 6 years ago
remove USEWEAKREFS code from nsWindowWatcher
Categories
(Core :: DOM: Core & HTML, enhancement, P2)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: froydnj, Assigned: froydnj)
Details
Attachments
(1 file)
5.38 KB,
patch
|
smaug
:
review+
|
Details | Diff | Splinter Review |
Not really sure where to file this one...
Assignee | ||
Comment 1•6 years ago
|
||
It's been over fifteen years, and we haven't figured out how to use weakrefs in this code (as the accompanying comment suggets). We might as well go ahead and delete this.
Attachment #8960847 -
Flags: review?(bugs)
Updated•6 years ago
|
Priority: -- → P2
Comment 2•6 years ago
|
||
Comment on attachment 8960847 [details] [diff] [review] remove USEWEAKREFS code from nsWindowWatcher huh, lovely code :p
Attachment #8960847 -
Flags: review?(bugs) → review+
Pushed by nfroyd@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/0052594a7fda remove USEWEAKREFS code from nsWindowWatcher; r=smaug
Comment 4•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0052594a7fda
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•