[webkit-reviews] review denied: [Bug 59699] Global object is recreated on teardown, for no good reason : [Attachment 91486] Possible fix
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Apr 28 11:39:09 PDT 2011
Geoffrey Garen <ggaren at apple.com> has denied review:
Bug 59699: Global object is recreated on teardown, for no good reason
https://bugs.webkit.org/show_bug.cgi?id=59699
Attachment 91486: Possible fix
https://bugs.webkit.org/attachment.cgi?id=91486&action=review
------- Additional Comments from Geoffrey Garen <ggaren at apple.com>
I think this patch would crash, because setWindow(JSGlobalData&, JSDOMWindow*)
assumes the window is not NULL.
It might work to change setWindow(JSGlobalData&, JSDOMWindow*) to use jsNull()
as the window shell's prototype if the JSDOMWindow* is NULL, or you could just
change setWindow(JSGlobalData&, JSDOMWindow*) to take an explicit prototype
argument.
More information about the webkit-reviews
mailing list