The default bug view has changed. See this FAQ.

IPDL: Fix the C++ test harness and (cross-process) TestOpens

RESOLVED FIXED in mozilla16

Status

()

Core
IPC
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: cjones, Assigned: cjones)

Tracking

Trunk
mozilla16
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Something to do with nsRgnRectAllocator broke the test harness recently.  TestOpens has had a latent bug for a while that I just stepped on after unbreaking the harness.
Created attachment 628620 [details] [diff] [review]
Bug 760007, part 1: Fix the IPDL C++ test harness. r=bent
Assignee: nobody → jones.chris.g
Attachment #628620 - Flags: review?(bent.mozilla)
Created attachment 628621 [details] [diff] [review]
part 2: Fix shutdown race condition in cross-process TestOpens

The test depends on TestOpensOpenedChild being deleted before its underlying Transport, but nothing was guaranteeing that ordering.  Intermittent crashes resulted.
Attachment #628621 - Flags: review?(bent.mozilla)
Attachment #628620 - Flags: review?(bent.mozilla) → review+
Attachment #628621 - Flags: review?(bent.mozilla) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/a0a708c9250a
https://hg.mozilla.org/integration/mozilla-inbound/rev/9985e43c2c49
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/a0a708c9250a
https://hg.mozilla.org/mozilla-central/rev/9985e43c2c49
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.