[Webkit-unassigned] [Bug 43716] Use a HashMap for m_continuation to save memory

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=43716





--- Comment #9 from Kenichi Ishibashi <bashi at google.com>  2010-08-31 18:53:48 PST ---
I counted up how many RenderBlock and RenderInline are created, and check how many continuations are created for some web pages. Continuations are rarely created. For instance, when I load cnn.com, the total creation number of RenderBlock and RenderInline objects is 1081, while the number of continuation created is just 9. In this case, approximately 4KBytes might be able to reduce on 32bit machines ideally (Of course, there are many other factors affecting memory usage, such as memory allocation strategy and alignments, so it's just for expectation).

I also checked actual memory usage by using activity monitor. In most cases, this change would consume less memory. As an instance, when I load cnn.com, I observed following memory usage on my 64bit SnowLeopard:
- with the patch
Real Mem: 72.7MB, Private Mem: 43.2MB
- without the patch
Real Mem: 73.8MB, Private Mem: 43.4MB

Needless to say, this result doesn't convince that this change will reduce memory usage in general, but I think it might show the effect of this patch.

I'll send the revised patch later.

Thanks,

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.