[Webkit-unassigned] [Bug 58580] Make plugins compile during WebKit GTK Windows build
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Apr 14 15:41:32 PDT 2011
https://bugs.webkit.org/show_bug.cgi?id=58580
Martin Robinson <mrobinson at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #89663|review?, commit-queue? |review-
Flag| |
--- Comment #2 from Martin Robinson <mrobinson at webkit.org> 2011-04-14 15:41:32 PST ---
(From update of attachment 89663)
View in context: https://bugs.webkit.org/attachment.cgi?id=89663&action=review
Looks good, but I think it needs another iteration.
> Source/WebCore/plugins/PluginView.h:366
> +#if defined(XP_UNIX) || PLATFORM(GTK)
> bool m_needsXEmbed;
Is the m_needsXEmbed variable really necessary for Windows?
> Source/WebCore/plugins/gtk/PluginViewGtk.cpp:83
> +#include <windows.h>
> #include "PluginMessageThrottlerWin.h"
> #include <gdk/gdkwin32.h>
Please put these in alphabetical order if you can.
> Source/WebCore/plugins/gtk/PluginViewGtk.cpp:667
> - *static_cast<HGIOBJ*>(value) = GDK_WINDOW_HWND(gdkWindow);
> + *static_cast<HGDIOBJ*>(value) = GDK_WINDOW_HWND(gdkWindow);
Whoops. I think this was my fault.
> Source/WebCore/plugins/gtk/PluginViewGtk.cpp:798
> PluginView::setCurrentPluginView(this);
> JSC::JSLock::DropAllLocks dropAllLocks(JSC::SilenceAssertionsOnly);
> setCallingPlugin(true);
> +#if defined(XP_UNIX)
> m_plugin->pluginFuncs()->getvalue(m_instance, NPPVpluginNeedsXEmbed, &m_needsXEmbed);
> +#endif
If you're going to avoid the plugin call on !XP_UNIX, it makes sense to avoid all the preparation for the call as well.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list