|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
This is a pretty simple method that's dispatched to the main thread in two places. In both cases we can label it with the SystemGroup. http://searchfox.org/mozilla-central/search?q=nsIThread%3A%3AAsyncShutdown%22&case=false®exp=false&path=
Maybe this bug could also cover labeling nsThreadShutdownAckEvent with the SystemGroup.
First time submitting a patch to Mozilla. Please take a look. Sean
Comment on attachment 8901634 [details] Bug 1376858 - Use SystemGroup to label nsIThread::AsyncShutdown, https://reviewboard.mozilla.org/r/173040/#review179262
Attachment #8901634 - Flags: review?(wmccloskey) → review+
Thanks for the patch!
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/778b209ee3d9 Use SystemGroup to label nsIThread::AsyncShutdown, r=billm
Status: NEW → RESOLVED
Last Resolved: 9 months ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.