Closed Bug 1677466 Opened 1 year ago Closed 1 year ago

Reduce the includes pulled in by some IPC headers

Categories

(Core :: IPC, task)

task

Tracking

()

RESOLVED FIXED
85 Branch
Tracking Status
firefox85 --- fixed

People

(Reporter: sg, Assigned: sg)

References

Details

Attachments

(3 files, 1 obsolete file)

In particular:

  • unnecessary includes should be removed from ipc/glue/MessageChannel.h and ipc/glue/MessageLink.h
  • ipc/glue/ProtocolUtils.h and ipc/glue/IPCMessageUtils.h should be split up and only those parts actually needed should be included where these files are currently included

I think a bunch of changes to IPC headers should be in the IPC component.

Component: General → IPC
Product: Firefox Build System → Core
Attachment #9188095 - Attachment is obsolete: true
Pushed by malexandru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/09346a5b2030
Remove unnecessary includes from MessageChannel.h and MessageLink.h. r=mccr8
https://hg.mozilla.org/integration/autoland/rev/267fdd186740
Split Endpoint.h and ProtocolMessageUtils.h from ProtocolUtils.h. r=mccr8
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Pushed by sgiesecke@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b70eee6f3946
Move ParamTraits specializations with extra dependencies out of IPCMessageUtils.h. r=mccr8
Status: REOPENED → RESOLVED
Closed: 1 year ago1 year ago
Resolution: --- → FIXED
Regressions: 1681987
You need to log in before you can comment on or make changes to this bug.