If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Circular includes in IPDL headers leading to odd behavior

RESOLVED FIXED in Firefox 50

Status

()

Core
IPC
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: aklotz, Assigned: aklotz)

Tracking

(Blocks: 1 bug)

Trunk
mozilla50
Unspecified
Windows
Points:
---

Firefox Tracking Flags

(firefox49 affected, firefox50 fixed)

Details

(Whiteboard: btpp-active, aes+)

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

I want to clean up some dependencies here since they are causing some problems such as a11y protocols having to include completely unrelated gunk due to a include cycle in a managing protocol.
Whiteboard: btpp-active
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9684cd371d3f

Updated

a year ago
Whiteboard: btpp-active → btpp-active, aes-win

Updated

a year ago
OS: Unspecified → Windows
Whiteboard: btpp-active, aes-win → btpp-active, aes+
Created attachment 8771111 [details]
Bug 1268151: Clean up some dependencies in IPDL headers;

Review commit: https://reviewboard.mozilla.org/r/64374/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/64374/
Attachment #8771111 - Flags: review?(wmccloskey)
Attachment #8771111 - Flags: review?(wmccloskey) → review+
Comment on attachment 8771111 [details]
Bug 1268151: Clean up some dependencies in IPDL headers;

https://reviewboard.mozilla.org/r/64374/#review61388
Comment on attachment 8771111 [details]
Bug 1268151: Clean up some dependencies in IPDL headers;

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/64374/diff/1-2/

Comment 5

a year ago
Pushed by aklotz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ef57886bcaa6
Clean up some dependencies in IPDL headers; r=billm

Comment 6

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/ef57886bcaa6
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.