463664 - leak detected in fennec while viewing popuptest.com website

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

While developing tests for fennec, I was testing out http://www.popuptest.com/popuptest1.html.  Upon intially loading up this page, I get the yellow bar asking if I want to allow popups.  I selected "always" and reloaded the web page.  After loading, I closed down fennec, and see this in the debug window:

--DOMWINDOW == 6 (0xb4957100) [serial = 24] [outer = (nil)] [url = about:blank]
--DOMWINDOW == 5 (0xb4954a30) [serial = 12] [outer = 0xb4953770] [url = http://www.popuptest.com/popuptest1.html]
--DOMWINDOW == 4 (0xb49537a0) [serial = 9] [outer = (nil)] [url = http://www.popuptest.com/popuptest1.html]
--WEBSHELL 0xb7923ba0 == 2
WARNING: sqlite3_close failed. There are probably outstanding statements!: file /home/mozilla/mozilla/src/storage/src/mozStorageConnection.cpp, line 249
WARNING: nsExceptionService ignoring thread destruction after shutdown: file /home/mozilla/mozilla/src/xpcom/base/nsExceptionService.cpp, line 194
--DOMWINDOW == 3 (0xb79b7ff0) [serial = 1] [outer = (nil)] [url = resource://gre/res/hiddenWindow.html]
--DOMWINDOW == 2 (0xb79b8a70) [serial = 4] [outer = (nil)] [url = resource://gre/res/hiddenWindow.html]
WARNING: Leaking the RDF Service.: file /home/mozilla/mozilla/src/rdf/build/nsRDFModule.cpp, line 236
WARNING: Textrun cache not empty!: 'mCache.Count() == 0', file /home/mozilla/mozilla/src/gfx/thebes/src/gfxTextRunWordCache.cpp, line 69
Assertion failed at /home/mozilla/mozilla/src/gfx/cairo/cairo/src/cairo-hash.c:199: hash_table->live_entries == 0
nsStringStats
 => mAllocCount:          88942
 => mReallocCount:         3236
 => mFreeCount:           85873  --  LEAKED 3069 !!!
 => mShareCount:          23488
 => mAdoptCount:           2737
 => mAdoptFreeCount:       2735  --  LEAKED 2 !!!
mozilla@mozilla-desktop:~/mozilla/src$

Comment 1

[Ben Combee (:bcombee)]

Current debug build also shows leaks -- this is just starting on Win32 then immediately shutting down while in about:firstrun

WARNING: Leaking the RDF Service.: file c:/src/mozilla-central/rdf/build/nsRDFModule.cpp, line 236
WARNING: Textrun cache not empty!: 'mCache.Count() == 0', file c:/src/mozilla-central/gfx/thebes/src/gfxTextRunWordCache.cpp, line 85
WARNING: Fonts still alive while shutting down gfxFontCache: 'mFonts.Count() == 0', file c:\src\build\xulrunner\dist\include\gfxFont.h, line 401
Assertion failed at c:/src/mozilla-central/gfx/cairo/cairo/src/cairo-hash.c:198: hash_table->live_entries == 0
Leaked URLs:
  file:///c:/src/build/mobile/dist/bin/xulrunner/chrome/toolkit.jar
  ... (deleted many strings)
  resource://gre/res/loading-image.png
  resource://gre/res/broken-image.png
nsStringStats
 => mAllocCount:          13202
 => mReallocCount:         2316
 => mFreeCount:           10222  --  LEAKED 2980 !!!
 => mShareCount:          10751
 => mAdoptCount:           1614
 => mAdoptFreeCount:       1612  --  LEAKED 2 !!!

Comment 2

[Aakash Desai [:aakashd]]

Is this still happening guys?

Comment 3

[Michael Wu [:mwu]]

Still leaks here on my OSX build.

Comment 4

[Michael Wu [:mwu]]

These leaks are due to the popups on unload that the page has. Taking a closer look.

Comment 5

[Michael Wu [:mwu]]

I can't figure this out. I can also get it to leak just by opening fennec and then closing.

Comment 6

[Aakash Desai [:aakashd]]

Martijn, can you take a look at this and talk to Joel.

Comment 7

[Martijn Wargers (dead)]

I can reproduce this on the latest 1.9.2 build on windows, but not with the latest 1.9.3 build on windows.

Comment 8

[Martijn Wargers (dead)]

Attachment: testcase

The onunload popup window showing up, triggers the leak.

Comment 9

[Martijn Wargers (dead)]

I can't find a fix range, because for windows, there are no older 1.9.3 fennec builds.
I guess the question is, is this bug bad enough, that it should be fixed for 1.9.2 builds?

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #9)

> I guess the question is, is this bug bad enough, that it should be fixed for
> 1.9.2 builds?

I vote no

Comment 11

[BMO Automation]

Closing all opened bug in a graveyard component