[webkit-reviews] review denied: [Bug 63044] [EFL] Makes themeForPage() return static RenderThemeEfl. : [Attachment 97930] Proposed patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Sep 12 15:47:42 PDT 2011
Eric Seidel <eric at webkit.org> has denied Jaehun Lim <ljaehun.lim at samsung.com>'s
request for review:
Bug 63044: [EFL] Makes themeForPage() return static RenderThemeEfl.
https://bugs.webkit.org/show_bug.cgi?id=63044
Attachment 97930: Proposed patch
https://bugs.webkit.org/attachment.cgi?id=97930&action=review
------- Additional Comments from Eric Seidel <eric at webkit.org>
Does the RenderTheme hold onto the Page? If it doesn't, then it seems this
will crash. If it does, then your'e extending the lifetime of the first page
forever (leaking it). Both seem bad.
More information about the webkit-reviews
mailing list