Follow-up to bug 540886: address review comments I forgot to before pushing.

This commit is contained in:
Chris Jones 2010-01-27 02:17:17 -06:00
parent 7ec5b63ca4
commit 85cfeb583f
2 changed files with 3 additions and 7 deletions

View File

@ -91,6 +91,7 @@ AsyncChannel::AsyncChannel(AsyncListener* aListener)
mCvar(mMutex, "mozilla.ipc.AsyncChannel.mCvar"),
mIOLoop(),
mWorkerLoop(),
mChild(false),
mChannelErrorTask(NULL)
{
MOZ_COUNT_CTOR(AsyncChannel);

View File

@ -472,7 +472,7 @@ RPCChannel::BlockOnParent()
NS_RUNTIMEABORT("attempt to block child when it's already blocked");
mBlockedOnParent = true;
while (1) {
do {
// XXX this dispatch loop shares some similarities with the
// one in Call(), but the logic is simpler and different
// enough IMHO to warrant its own dispatch loop
@ -502,12 +502,7 @@ RPCChannel::BlockOnParent()
AsyncChannel::OnDispatchMessage(recvd);
}
}
// the last message, if async, may have been the one that
// unblocks us
if (!mBlockedOnParent)
break;
}
} while (mBlockedOnParent);
EnqueuePendingMessages();
}