Closed Bug 1399308 Opened 2 years ago Closed 2 years ago

nsISupportsImpl.cpp uses GetCurrentVirtualThread() without including nsThreadUtils.h

Categories

(Core :: XPCOM, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: cpearce, Assigned: cpearce)

References

Details

Attachments

(1 file)

nsISupportsImpl.cpp is calling GetCurrentVirtualThread() without including nsThreadUtils.h. This results in a compilation error if unified builds are disabled.
Comment on attachment 8907377 [details]
Bug 1399308 - Include nsThreadUtils.h in nsISupportsImpl.cpp.

https://reviewboard.mozilla.org/r/179054/#review184158
Attachment #8907377 - Flags: review?(nfroyd) → review+
Pushed by cpearce@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b393da8b7c59
Include nsThreadUtils.h in nsISupportsImpl.cpp. r=froydnj
https://hg.mozilla.org/mozilla-central/rev/b393da8b7c59
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.