[webkit-reviews] review denied: [Bug 27706] WebCore NPAPI implementations should share more code : [Attachment 38861] Join the various PluginView destructors into one shared implementation
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Sep 1 06:34:00 PDT 2009
Eric Seidel <eric at webkit.org> has denied Tor Arne Vestbø <vestbo at webkit.org>'s
request for review:
Bug 27706: WebCore NPAPI implementations should share more code
https://bugs.webkit.org/show_bug.cgi?id=27706
Attachment 38861: Join the various PluginView destructors into one shared
implementation
https://bugs.webkit.org/attachment.cgi?id=38861&action=review
------- Additional Comments from Eric Seidel <eric at webkit.org>
Seems this could easily be shared in a platformStop() or
destroyPlatformPluginWidget() method:
0 #if defined(XP_WIN)
261 DestroyWindow(platformPluginWidget());
262 #elif defined(XP_MACOSX)
263 setPlatformPluginWidget(0);
264 #elif defined(XP_UNIX) && PLATFORM(QT)
265 // TODO: Should this be done for GTK too?
266 delete platformPluginWidget();
267 #endif
r- for the above.
More information about the webkit-reviews
mailing list