I spent a while last night debugging something where Close() was getting called twice on a top-level protocol. It's extra boilerplate we have to roll out each time, so we should probably just move it into MessageChannel. Any objections, Bill?
Err, the boilerplate being "check that we haven't already called Close or that ActorDestroy hasn't run".
Attachment #8790577 - Flags: review?(wmccloskey) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/a706f77e10af Don't allow double-closing of IPC channels. (bug 1301481, r=billm)
You need to log in before you can comment on or make changes to this bug.