Closed Bug 1339676 Opened 4 years ago Closed 7 months ago
Label runnables in netwerk/
Meta bug for labeling all runnables in necko.
Hi Bevis, I would like to make sure that my understanding about labeling on IPC actors is right. Do we have to label all IPC actors under PNecko like what we did for HttpChannelChild in bug 1318506? If yes, there are 15 actors  needed to be labeled. Is there a more efficient way instead of labeling them one by one? Thanks.  http://searchfox.org/mozilla-central/rev/ac8a72f2d5204ced2004c93a9c193430c63a6a71/netwerk/ipc/PNecko.ipdl#47-61
Per offline discussion, there is no efficient way in this case, because these actors are at the same managee level, and we can't inherit the event target from their manager unless they are expected to be the same in your design. BTW, if some of these actors are not touched by the content and the order of the callback won't be an impact to other docgroups, then you can just label them in SystemGroup.
Summary: Labeling runnables in netwerk/ → Label runnables in netwerk/
Whiteboard: [necko-next] → [necko-next][QDL][TDC-MVP][NECKO]
Whiteboard: [necko-next][QDL][TDC-MVP][NECKO] → [necko-next]
Bulk priority update: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P1
Is this targeted for 57 or will we punt to 58?
(In reply to Jason Duell [:jduell] (needinfo me) from comment #4) > Is this targeted for 57 or will we punt to 58? This should be targeted for 57. Only bug 1388247 is left to 58.
Status: NEW → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.