Bug 71260 - Leaks seen in RenderFlowThread::setRegionRangeForBox on Leaks bot
Summary: Leaks seen in RenderFlowThread::setRegionRangeForBox on Leaks bot
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Darin Adler
URL: http://build.webkit.org/LeaksViewer/?...
Keywords: InRadar, MakingBotsRed
: 71620 (view as bug list)
Depends on:
Blocks: 57312
  Show dependency treegraph
 
Reported: 2011-10-31 17:58 PDT by Adam Roben (:aroben)
Modified: 2011-11-06 14:27 PST (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2011-10-31 17:58:24 PDT
http://build.webkit.org/LeaksViewer/?url=http%3A%2F%2Fbuild.webkit.org%2F%2Fresults%2FSnowLeopard%20Intel%20Leaks%2Fr98879%20(19849)%2F

malloc_zone_malloc > malloc > fastMalloc > RenderFlowThread::setRegionRangeForBox

We're leaking in this function. Looks like we're allocating a new RenderRegionRange, storing it in a HashMap, and never deleting it.
Comment 1 Radar WebKit Bug Importer 2011-10-31 17:58:56 PDT
<rdar://problem/10374428>
Comment 2 Darin Adler 2011-11-05 18:19:47 PDT
*** Bug 71620 has been marked as a duplicate of this bug. ***
Comment 3 Darin Adler 2011-11-05 18:20:43 PDT
This is the same leak as 71259.

*** This bug has been marked as a duplicate of bug 71259 ***
Comment 4 Adam Roben (:aroben) 2011-11-05 18:22:29 PDT
How are they the same? Bug 71259 is about leaks of RenderBoxRegionInfo objects, this bug is about leaks of RenderRegionRange objects.
Comment 5 Adam Roben (:aroben) 2011-11-05 18:23:18 PDT
I guess one of those objects could own the other…
Comment 6 Darin Adler 2011-11-06 08:42:04 PST
Adam’s right. This is not a duplicate.
Comment 7 Mihnea Ovidenie 2011-11-06 12:37:21 PST
In function RenderFlowThread::removeRenderBoxRegionInfo, the RenderRegionRange* for box is removed from the map but not deleted.
Comment 8 Darin Adler 2011-11-06 14:24:29 PST
Yes, you can see the patch I already posted and got reviewed to fix this in bug 71620.
Comment 9 Darin Adler 2011-11-06 14:27:40 PST
Committed r99384: <http://trac.webkit.org/changeset/99384>