Open
Bug 968800
Opened 11 years ago
Updated 2 years ago
mtransport runnable templates silently allow already_AddRefed members
Categories
(Core :: WebRTC, defect, P5)
Core
WebRTC
Tracking
()
NEW
| backlog | tech-debt |
People
(Reporter: froydnj, Unassigned)
References
(Blocks 1 open bug)
Details
We shouldn't do that; already_AddRefed members are footguns. Fixing this may require an extra AddRef/Release pair or generalizing NS_ProxyRelease.
I have ideas for fixing this, but runnable_utils.py needs some cleanup love first.
|
||
Updated•11 years ago
|
Component: Video/Audio → WebRTC
|
||
Comment 1•9 years ago
|
||
Still relevant given the WrapRunnable work ongoing?
backlog: --- → tech-debt
Flags: needinfo?(nfroyd)
|
|
||
Updated•9 years ago
|
Rank: 45
Priority: -- → P4
|
Reporter | |
Comment 2•9 years ago
|
||
(In reply to Maire Reavy [:mreavy] (Plz needinfo me) from comment #1)
> Still relevant given the WrapRunnable work ongoing?
Yes. It is much easier to do now, though! :)
Flags: needinfo?(nfroyd)
|
||
Comment 3•7 years ago
|
||
Mass change P4->P5 to align with new Mozilla triage process.
Priority: P4 → P5
|
||
Updated•3 years ago
|
Assignee: froydnj+bz → nobody
|
||
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•