Closed
Bug 1376860
Opened 7 years ago
Closed 3 years ago
Give names to all non-nsINamed runnables
Categories
(Core :: XPCOM, enhancement, P3)
Core
XPCOM
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: billm, Unassigned)
References
Details
This will help with the labeling effort. I think the easiest way to do this would be to make nsIRunnable inherit from nsINamed. Then every nsIRunnable would be required to have a name.
Comment 1•7 years ago
|
||
I think this is superseded by your recent work in bug 1372405?
Flags: needinfo?(wmccloskey)
Reporter | ||
Comment 2•7 years ago
|
||
Bug 1372405 only covered stuff that inherits from mozilla::Runnable. We have some runnables that inherit directly from nsIRunnable. We could just fix those I guess so that they inherit from Runnable. But there are probably some legitimate use cases.
This is a probably complete list:
http://searchfox.org/mozilla-central/search?q=public%20nsIRunnable
I guess JS runnables also fall into this category. Depending on how important they are, we could probably convince xpconnect to label them automatically.
Flags: needinfo?(wmccloskey)
Reporter | ||
Comment 3•7 years ago
|
||
I didn't mean to assign this to myself.
Assignee: wmccloskey → nobody
Comment 4•7 years ago
|
||
This would be desirable for moving more behavior into nsIRunnable/mozilla::Runnable.
Priority: -- → P3
Updated•3 years ago
|
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•