Last Comment Bug 760007 - IPDL: Fix the C++ test harness and (cross-process) TestOpens
: IPDL: Fix the C++ test harness and (cross-process) TestOpens
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: IPC (show other bugs)
: Trunk
: x86_64 Linux
: -- normal (vote)
: mozilla16
Assigned To: Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
:
: [PTO to Dec5] Bill McCloskey (:billm)
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-31 00:59 PDT by Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
Modified: 2012-06-05 06:16 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Bug 760007, part 1: Fix the IPDL C++ test harness. r=bent (1.80 KB, patch)
2012-05-31 01:03 PDT, Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
bent.mozilla: review+
Details | Diff | Splinter Review
part 2: Fix shutdown race condition in cross-process TestOpens (2.06 KB, patch)
2012-05-31 01:04 PDT, Chris Jones [:cjones] inactive; ni?/f?/r? if you need me
bent.mozilla: review+
Details | Diff | Splinter Review

Description Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2012-05-31 00:59:52 PDT
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.
Comment 1 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2012-05-31 01:03:01 PDT
Created attachment 628620 [details] [diff] [review]
Bug 760007, part 1: Fix the IPDL C++ test harness. r=bent
Comment 2 Chris Jones [:cjones] inactive; ni?/f?/r? if you need me 2012-05-31 01:04:16 PDT
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.

Note You need to log in before you can comment on or make changes to this bug.