Closed Bug 1706376 Opened 4 years ago Closed 4 years ago

Remove loud out-of-order warning from BlockingResourceBase::Release

Categories

(Core :: XPCOM, task)

task

Tracking

()

RESOLVED FIXED
90 Branch
Tracking Status
firefox90 --- fixed

People

(Reporter: nika, Assigned: nika)

References

Details

Attachments

(1 file)

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.

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.

Attachment #9217106 - Attachment description: Bug 1706376 - Remove loud warning from BlockingResurceBase::Release(), → Bug 1706376 - Remove loud warning from BlockingResourceBase::Release(),

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:nika, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(nika)
Flags: needinfo?(continuation)

Patch will be landed as part of bug 1706374's stack, as it doesn't provide much value until those changes are landed.

Flags: needinfo?(nika)
Flags: needinfo?(continuation)
Pushed by nlayzell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/19fe1e3e60f3 Remove loud warning from BlockingResourceBase::Release(), r=xpcom-reviewers,mccr8
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: