818931 - Google Map preview in Gmail leaks compartments until the page is refreshed

Status: RESOLVED INVALID

(Tech Evangelism Graveyard :: English US, defect)

Description

[Fanolian]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:20.0) Gecko/20121206 Firefox/20.0
Build ID: 20121206030737

Steps to reproduce:

1. In a new profile, log in to Gmail, enable "Google Maps previews in mail" from Gmail's Settings > Labs.
2. Open about:compartments in another tab. There is no compartments from googleusercontent.com at this point.
3. Open an email which includes an (geographical) address. It should trigger a Google Map gadget below the email body. There is a user compartment from googleusercontent.com now. (Loading the gadget may take several seconds.)
  *Alternatively, send yourself an email with message "1600 Amphitheatre Parkway, Mountain View, CA 94043, USA." and open it.
4. Press Gmail's back icon or Nightly's back button and return to Gmail's main page.



Actual results:

A zombie compartment is created. The URL looks like 
https://4lam9a1ki27mb9p1h5q3furvvf58ss02-a-gm-opensocial.googleusercontent.com/gadgets/ifr?url=http%3A%2F%2Fwww.gstatic.com%2Fig%2Fmodules%2Fgm%2Fgmaps%2Fcard-gmaps.xml... [2]

Minimising memory usage in about:memory does not destroy the zombie compartment. Wait for 10 minutes and it is still alive.

Note:
1. Opening and leaving the mail multiple times create multiple zombie compartments.
2. Although reloading the page using browser's reload button, or closing the tab, destroy the compartment, using Gmail's refresh function does not. Since Gmail works properly without reloading from the browser, this may be a potential problem for heavy Gmail users who pin the tab and rely on Gmail's refresh.

This is reproducible in current Nightly and FF17.0.1release.

Comment 1

[Fanolian]

Extract from about:memory:

94,225,221 B (100.0%) -- explicit
├──38,901,048 B (41.29%) -- window-objects
│  ├──29,338,496 B (31.14%) ++ top(https://mail.google.com/mail/u/0/?shva=1#inbox, id=8)/active
│  ├───4,272,552 B (04.53%) ++ top(chrome://browser/content/browser.xul, id=3)/active
│  ├───2,827,800 B (03.00%) -- top(none)
│  │   ├──2,350,240 B (02.49%) -- ghost/window(https://4lam9a1ki27mb9p1h5q3furvvf58ss02-a-gm-opensocial.googleusercontent.com/gadgets/ifr?url=http%3A%2F%2Fwww.gstatic.com%2Fig%2Fmodules%2Fgm%2Fgmaps%2Fcard-gmaps.xml&container=gm&view=card&lang=en&country=HK&sanitize=0&v=8f206b9030c11dae&libs=core%3Adynamic-height%3Aflash%3Agoogle.contentmatch%3Agoogle.debug%3Agoogle.sharebox%3Agoogle.waitforload%3Alocked-domain%3Aoauthpopup%3Arpc%3Asecurity-token%3Asetprefs%3Askins%3Aviews%3Aauth-refresh&parent=https%3A%2F%2Fmail.google.com%2Fhtml&skintype=gm&skinval=GM_BG_COLOR%3A%23F2F2F2%3BGM_TEXT_COLOR%3A%23222%3BGM_LINK_COLOR%3A%2315c%3BGM_SELECTED_BG_COLOR%3A%23ffc%3BGM_SELECTED_TEXT_COLOR%3A%23222%3BGM_SELECTED_LINK_COLOR%3A%23222%3BGM_MENU_BORDER_COLOR%3A%23888%3BGM_MENU_BG_COLOR%3A%23fff%3BGM_MENU_TEXT_COLOR%3A%23333%3BGM_MENU_LINK_COLOR%3A%230065cc#c=gm&rpctoken=qhbhmppyrrf8&cob=%257B%2522GOOGLE_ADDRESS_EXTRACTOR%2522%253A%255B%257B%2522address%2522%253A%25221600%2520Amphitheatre%2520Parkway%255CnMountain%2520View%252C%2520CA%252094043%2522%257D%255D%257D&mid=1&st=e%3DAA6WCYabtxMB6K7DmY6Dut8s%252Bv3bvdN6VgNLH1w%252FcjxsOn%252FfIgdd5Kk7I88btwOQzGPu5fpDBx8hOD1MFD%252Fyd4gXGNzPfZOs3ngtT%252FOHS5vwnT6OxQyh0O5FEZQJvrgiL7or%252BZxGUzrB%26c%3Dgm)
│  │   │  ├──2,349,280 B (02.49%) -- js/compartment(https://4lam9a1ki27mb9p1h5q3furvvf58ss02-a-gm-opensocial.googleusercontent.com/gadgets/ifr?url=http%3A%2F%2Fwww.gstatic.com%2Fig%2Fmodules%2Fgm%2Fgmaps%2Fcard-gmaps.xml&container=gm&view=card&lang=en&country=HK&sanitize=0&v=8f206b9030c11dae&libs=core%3Adynamic-height%3Aflash%3Agoogle.contentmatch%3Agoogle.debug%3Agoogle.sharebox%3Agoogle.waitforload%3Alocked-domain%3Aoauthpopup%3Arpc%3Asecurity-token%3Asetprefs%3Askins%3Aviews%3Aauth-refresh&parent=https%3A%2F%2Fmail.google.com%2Fhtml&skintype=gm&skinval=GM_BG_COLOR%3A%23F2F2F2%3BGM_TEXT_COLOR%3A%23222%3BGM_LINK_COLOR%3A%2315c%3BGM_SELECTED_BG_COLOR%3A%23ffc%3BGM_SELECTED_TEXT_COLOR%3A%23222%3BGM_SELECTED_LINK_COLOR%3A%23222%3BGM_MENU_BORDER_COLOR%3A%23888%3BGM_MENU_BG_COLOR%3A%23fff%3BGM_MENU_TEXT_COLOR%3A%23333%3BGM_MENU_LINK_COLOR%3A%230065cc#c=gm&rpctoken=qhbhmppyrrf8&cob=%257B%2522GOOGLE_ADDRESS_EXTRACTOR%2522%253A%255B%257B%2522address%2522%253A%25221600%2520Amphitheatre%2520Parkway%255CnMountain%2520View%252C%2520CA%252094043%2522%257D%255D%257D&mid=1&st=e%3DAA6WCYabtxMB6K7DmY6Dut8s%252Bv3bvdN6VgNLH1w%252FcjxsOn%252FfIgdd5Kk7I88btwOQzGPu5fpDBx8hOD1MFD%252Fyd4gXGNzPfZOs3ngtT%252FOHS5vwnT6OxQyh0O5FEZQJvrgiL7or%252BZxGUzrB%26c%3Dgm)
│  │   │  │  ├──1,290,240 B (01.37%) -- gc-heap
│  │   │  │  │  ├────400,624 B (00.43%) -- objects
│  │   │  │  │  │    ├──214,352 B (00.23%) ── function
│  │   │  │  │  │    ├──116,816 B (00.12%) ── ordinary
│  │   │  │  │  │    └───69,456 B (00.07%) ── dense-array
│  │   │  │  │  ├────305,320 B (00.32%) ── unused-gc-things
│  │   │  │  │  ├────257,880 B (00.27%) ── scripts
│  │   │  │  │  ├────229,208 B (00.24%) -- shapes
│  │   │  │  │  │    ├──116,448 B (00.12%) ── tree/global-parented
│  │   │  │  │  │    ├───70,584 B (00.07%) ── dict
│  │   │  │  │  │    └───42,176 B (00.04%) ── base
│  │   │  │  │  ├─────83,424 B (00.09%) ── type-objects
│  │   │  │  │  └─────13,784 B (00.01%) ── sundries
│  │   │  │  ├────613,560 B (00.65%) ── script-data
│  │   │  │  ├────176,704 B (00.19%) -- shapes-extra
│  │   │  │  │    ├───78,336 B (00.08%) ── compartment-tables
│  │   │  │  │    ├───48,896 B (00.05%) ── tree-tables
│  │   │  │  │    ├───30,368 B (00.03%) ── dict-tables
│  │   │  │  │    └───19,104 B (00.02%) ── tree-shape-kids
│  │   │  │  ├────147,488 B (00.16%) -- type-inference
│  │   │  │  │    ├───98,304 B (00.10%) ── type-pool
│  │   │  │  │    └───49,184 B (00.05%) ── allocation-site-tables
│  │   │  │  ├────106,480 B (00.11%) -- objects-extra
│  │   │  │  │    ├───86,400 B (00.09%) ── slots
│  │   │  │  │    └───20,080 B (00.02%) ── elements
│  │   │  │  └─────14,808 B (00.02%) ── other-sundries
│  │   │  └────────960 B (00.00%) ── dom/other [2]

Comment 2

[Justin Lebar (not reading bugmail)]

> 2. Although reloading the page using browser's reload button, or closing the tab, 
> destroys the compartment, using Gmail's refresh function does not.

This sounds like it's a bug in gmail, not a bug in Firefox.  Firefox is doing the right thing and releasing the compartment when the tab is closed.

I'm going to get rid of the "zombie compartment" part of this bug, because we reserve that for bugs where Firefox holds onto a compartment itself.

Comment 3

[Justin Lebar (not reading bugmail)]

Hi, Kev.  We could use a contact at Google here.

Comment 4

[Kev Needham [:kev]]

Will try and get a product contact for maps.

Comment 5

[Justin Lebar (not reading bugmail)]

(In reply to Kev Needham [:kev] from comment #4)
> Will try and get a product contact for maps.

FWIW I'd guess the bug is in gmail, not maps.

Comment 6

[Karl Dubost💡 :karlcow]

Is it still the case? Google Maps has changed a lot since. Do you still have the issue in gmail?

Comment 7

[Fanolian]

I cannot reproduce this anymore. Actually the Google Map preview function does not work for me now.