Open Bug 991331 Opened 10 years ago Updated 2 years ago

Add MOZ_COUNT_CTOR/DTOR to LinkedListElement

Categories

(Core :: MFBT, defect)

defect

Tracking

()

People

(Reporter: khuey, Unassigned)

Details

(Whiteboard: [MemShrink:P2])

Yeah, I have a list of MFBT bugs to file about these kinds of issues.  This was one of them.

We can use a variant of my hack for this stuff as well, which would cover all of the libxul usage.  Or we can spend some time to figure out how to move the underlying stuff to somewhere that can be shared between MFBT and XPCOM.  The latter takes a considerable amount of time.

How urgent is this Kyle?
They also need owning-thread assertions. We would have caught at least one bug easily instead of having to grok intermittent orange logs.
(In reply to comment #2)
> They also need owning-thread assertions. We would have caught at least one bug
> easily instead of having to grok intermittent orange logs.

Yes, also on my list of the stuff to file...
Of course then we have to teach mfbt about threads...
(In reply to :Ehsan Akhgari (lagging on bugmail, needinfo? me!) from comment #3)
> Yes, also on my list of the stuff to file...

FWIW, I have now filed this as bug 991710.
Whiteboard: [MemShrink] → [MemShrink:P2]
Assignee: nobody → continuation
The bundle cache CTOR/DTOR stuff should be reverted when this lands: https://hg.mozilla.org/integration/mozilla-inbound/rev/36879243d685
This is probably still a good idea.
Assignee: continuation → nobody
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.