Closed Bug 842710 Opened 12 years ago Closed 12 years ago

Identify TabChildGlobals in about:memory/compartment

Categories

(Core :: DOM: Core & HTML, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: smaug, Assigned: smaug)

Details

(Whiteboard: [MemShrink:P2])

Attachments

(1 file)

Whiteboard: [MemShrink]
It looks like this names the compartments as compartment(inProcessTabChildGlobal?ownedby=http://foobar) ? We do system principal compartments as compartment([System Principal], http://foobar) so perhaps we should stay in line with that and do something like compartment([In-process TabChild], http://foobar) ? I bet njn has an opinion too. :)
TabChildGlobal use system principal, so they should have [System Principal]
and http://... doesn't make sense. TabChildGlobals aren't really in any URL. Some page may own the element which then owns the TabChildGlobal (in case of in-process). But I can change the ID to whatever feels the best :) I rarely use about:memory/compartment so I'm not really familiar with conventions there.
So the current patch gives us compartment([System Principal], inProcessTabChildGlobal?ownedby=...) ?
Yes [System Principal], inProcessTabChildGlobal?ownedBy=chrome://browser/content/browser.xul [3]
More specific naming of compartments is always good... What name would these compartments have had before? How likely is the case where there isn't an "ownedBy" part?
Before the patch out-of-process tabchildglobals are just [System Principal] and in-process tabchildglobals are [System Principal|, chrome://browser/content/browser.xul which less-nicely mixes them with other system compartments. I mean, the top level chrome doc has chrome://browser/content/browser.xul as document uri, and the same is used for tabchildglobals, so they all are reported to be similar things.
Whiteboard: [MemShrink] → [MemShrink:P2]
> [System Principal], inProcessTabChildGlobal?ownedBy=chrome://browser/content/browser.xul [3] It's a bit ugly but I can't think of anything obviously better. Go for it.
Attachment #715642 - Flags: review?(justin.lebar+bug) → review+
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
This change just led to bug 842710 being filed -- another case of "if we measure it, we find problems with it". Nice work.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: