WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
37813
[Gtk] LEAK: WebCore::Document instances are leaked in certain conditions.
https://bugs.webkit.org/show_bug.cgi?id=37813
Summary
[Gtk] LEAK: WebCore::Document instances are leaked in certain conditions.
Stephan Aßmus
Reported
2010-04-19 12:40:43 PDT
This is something I've found out by trying to track down an issue in the Haiku port and cross-referencing the Gtk port. My debugging facilities are somewhat crude, but I can't see at the moment how my findings could be wrong: In the WebCore/dom/Document.cpp, I've added a variable "static int instanceCount = 0". Then in the Document constructor, I increase the variable, print the instance count, and in the destructor, I decrease the value and print it again. Normally, the instance count ballances out just fine, but in the following test case, but also in many other cases, it does not: Run GtkLauncher with "
http://www.wikipedia.com
" as argument on the command line. The instance count will go up to 2 and back to 1 until the page is loaded. The Frame instance which is passed to the Document constructor is always the same. I think it loads an initial about:blank and then the requested URL. Now enter a search string in the Wikipedia text input. I used "amiga". This will immediately navigate to the resulting page. A second Document instance has been created, while the first one is put into the page cache (I enabled the page cache in the GtkLauncher). Now close GtkLauncher, you will see that neither instance is being freed. If you don't enter a search string and close GtkLauncher, you will see that the only Document instance is freed correctly. Just to make sure, I repeated the test with the page cache disabled. In that case, the second of the two instances is freed. The revision I tested is
r57813
.
Attachments
Add attachment
proposed patch, testcase, etc.
Martin Robinson
Comment 1
2015-05-07 16:46:45 PDT
WebKit1 is gone, so I'm just going to close this. Please reopen if it's still an issue with a modern WebKitGTK+.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug