Closed Bug 1410086 Opened 3 years ago Closed 3 years ago

avoid unnecessary refcounting in LabeledEventQueue::HasReadyEvent

Categories

(Core :: XPCOM, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

We can do better than what we are currently doing.
There's no need to copy events in the queue to check whether they are
ready to run or not.
Attachment #8920159 - Flags: review?(erahm)
Attachment #8920159 - Flags: review?(erahm) → review+
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/dae2d770affc
don't copy QueueEntry(ies) when checking LabeledEventQueue for ready events; r=erahm
https://hg.mozilla.org/mozilla-central/rev/dae2d770affc
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.