919672 - lower the suspect referent count for the observer service's memory reporter

Status: RESOLVED FIXED

(Core :: General, defect)

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: lower the suspect referent count for the observer service's memory reporter

I have a couple score (hundreds?) of open tabs and have been noticing of late
that the observer service has 4k+ observers for such a session.  I was a bit
surprised by this, and lowered the suspect referent count a bit to see if I
could determine what was going on.

There's nothing particularly unusual with my setup, but I'd like to propose
that the count be lowered somewhat anyway.  I think it's useful for about:memory
to provide as much information as possible about what might be going on.  I've
also tried to update the explanatory text to indicate that merely having high
counts is not necessarily symptomatic of a leak.

Comment 2

[Nicholas Nethercote [inactive]]

Comment on attachment 808731 [details] [diff] [review]
lower the suspect referent count for the observer service's memory reporter

Review of attachment 808731 [details] [diff] [review]:
-----------------------------------------------------------------

Seems ok to me, but I'll r? wchen, since he's the original author of this code.

Comment 3

[William Chen [:wchen]]

Comment on attachment 808731 [details] [diff] [review]
lower the suspect referent count for the observer service's memory reporter

Review of attachment 808731 [details] [diff] [review]:
-----------------------------------------------------------------

::: xpcom/ds/nsObserverService.cpp
@@ +148,5 @@
>              nsIMemoryReporter::KIND_OTHER,
>              nsIMemoryReporter::UNITS_COUNT,
>              suspect.referentCount,
>              NS_LITERAL_CSTRING("A topic with a suspiciously large number "
> +                               "referents.  This may be symptomatic of a leak "

There is a grammatical error in the original comment that might as well be corrected in this patch.

larger number referents -> large number of referents.

Comment 4

[Nathan Froyd [:froydnj]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6aa2044de89d

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/6aa2044de89d