[webkit-reviews] review requested: [Bug 55017] SVGImage causes MainResourceLoader leaks : [Attachment 84507] new patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 2 18:53:50 PST 2011


Xianzhu Wang <wangxianzhu at google.com> has asked  for review:
Bug 55017: SVGImage causes MainResourceLoader leaks
https://bugs.webkit.org/show_bug.cgi?id=55017

Attachment 84507: new patch
https://bugs.webkit.org/attachment.cgi?id=84507&action=review

------- Additional Comments from Xianzhu Wang <wangxianzhu at google.com>
Hi, Adam,

This new patch removed the loader->init() code which is unnecessary because
frame->init() has already initialized the loader.
Also updated the test case suitable for all platforms. Could you please take
another look?

About chromium-linux's unnecessary SVGImage creation, I created bug 55643 to
track it.

Thanks,
Xianzhu


More information about the webkit-reviews mailing list