Bug 11236

Summary:

REGRESSION: Many memory leaks related to NSHTMLReader

Product:

WebKit

Reporter:

Mark Rowe (bdash) <mrowe>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Major

CC:

mitz

Priority:

Keywords:

InRadar, Regression

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Bug Depends on:

Bug Blocks:

12649

Attachments:

Description	Flags
ObjectAlloc output	none
Two CFDictionary (mutable-fixed) instances leaked by NSHTMLReader	none

Mark Rowe (bdash)

Reported 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.

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
View All Add attachment proposed patch, testcase, etc.

mitz

Comment 1 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.

mitz

Comment 2 2006-10-31 09:55:44 PST

Created attachment 11300 [details] ObjectAlloc output

mitz

Comment 3 2006-10-31 10:28:16 PST

Created attachment 11303 [details] Two CFDictionary (mutable-fixed) instances leaked by NSHTMLReader

Stephanie Lewis

Comment 4 2006-11-08 14:54:02 PST

radar 4826861

Maciej Stachowiak

Comment 5 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.

Darin Adler

Comment 6 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.

Darin Adler

Comment 7 2007-02-07 18:31:18 PST

Committed revision 19486.

Note You need to log in before you can comment on or make changes to this bug.