Closed Bug 1376860 Opened 7 years ago Closed 3 years ago

Give names to all non-nsINamed runnables

Categories

(Core :: XPCOM, enhancement, P3)

enhancement

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.
I think this is superseded by your recent work in bug 1372405?
Flags: needinfo?(wmccloskey)
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)
I didn't mean to assign this to myself.
Assignee: wmccloskey → nobody
This would be desirable for moving more behavior into nsIRunnable/mozilla::Runnable.
Priority: -- → P3
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.