Bug 11236 - REGRESSION: Many memory leaks related to NSHTMLReader
Summary: REGRESSION: Many memory leaks related to NSHTMLReader
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Major
Assignee: Nobody
URL:
Keywords: InRadar, Regression
Depends on:
Blocks: 12649
  Show dependency treegraph
 
Reported: 2006-10-09 16:55 PDT by Mark Rowe (bdash)
Modified: 2007-02-07 18:31 PST (History)
1 user (show)

See Also:


Attachments
ObjectAlloc output (9.29 KB, application/octet-stream)
2006-10-31 09:55 PST, mitz
no flags Details
Two CFDictionary (mutable-fixed) instances leaked by NSHTMLReader (3.83 KB, application/octet-stream)
2006-10-31 10:28 PST, mitz
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Rowe (bdash) 2006-10-09 16:55:51 PDT
For several weeks now there have been 4 or 5 memory leaks reported by run-webkit-tests, and around 20 node leaks.  See <http://build.webkit.org/results/post-commit-leaks-powerpc-mac-os-x/2977/DumpRenderTree-leaks.txt> for stack traces on the memory leaks.
Comment 1 mitz 2006-10-27 17:25:31 PDT
Using ObjectAlloc on a test case derived from editing/pasteboard/paste-as-text-012.html, I observed that sometimes NSHTMLReader leaks a DOMNode. This translates to a node leak. I believe this can make it harder to ignore the leak in run-webkit-tests if it's deemed external to WebKit.
Comment 2 mitz 2006-10-31 09:55:44 PST
Created attachment 11300 [details]
ObjectAlloc output
Comment 3 mitz 2006-10-31 10:28:16 PST
Created attachment 11303 [details]
Two CFDictionary (mutable-fixed) instances leaked by NSHTMLReader
Comment 4 Stephanie Lewis 2006-11-08 14:54:02 PST
radar 4826861
Comment 5 Maciej Stachowiak 2007-02-07 05:14:06 PST
This bug is actually outside of WebKit. The Radar bug has been put in the AppKit team's hands. Closing this for now.
Comment 6 Darin Adler 2007-02-07 18:30:59 PST
Believe it or not, it was a WebKit bug. Caused by the method name "hash" being used in the DOM bindings.
Comment 7 Darin Adler 2007-02-07 18:31:18 PST
Committed revision 19486.