Bug 1706376 - Remove loud warning from BlockingResourceBase::Release(), r=xpcom-reviewers,mccr8

This warning previously fired loudly from the ports logic being added in bug
1706374 were acquired. As the warning also occasionally fires in existing code
and provides no safety guarantees, I think we should consider removing it.

Differential Revision: https://phabricator.services.mozilla.com/D112762
This commit is contained in:
Nika Layzell 2021-05-06 20:05:33 +00:00
parent c4086dd08d
commit ddfc865bf0

View File

@ -286,11 +286,6 @@ void BlockingResourceBase::Release() {
if (chainFront == this) {
ResourceChainRemove();
} else {
// not an error, but makes code hard to reason about.
NS_WARNING("Resource acquired is being released in non-LIFO order; why?\n");
nsCString tmp;
Print(tmp);
// remove this resource from wherever it lives in the chain
// we walk backwards in order of acquisition:
// (1) ...node<-prev<-curr...