[Webkit-unassigned] [Bug 30078] [GTK] Gtk-directfb and plugins
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Oct 8 16:03:32 PDT 2009
https://bugs.webkit.org/show_bug.cgi?id=30078
Gustavo Noronha (kov) <gns at gnome.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #40780|review? |review-
Flag| |
--- Comment #8 from Gustavo Noronha (kov) <gns at gnome.org> 2009-10-08 16:03:32 PDT ---
(From update of attachment 40780)
> Index: WebCore/plugins/npapi.cpp
> ===================================================================
> --- WebCore/plugins/npapi.cpp (revision 49240)
> +++ WebCore/plugins/npapi.cpp (working copy)
> @@ -108,9 +108,11 @@ const char* NPN_UserAgent(NPP instance)
> {
> PluginView* view = pluginViewForInstance(instance);
>
> +#if ENABLE(NETSCAPE_PLUGIN_API)
> if (!view)
> return PluginView::userAgentStatic();
>
> +#endif
> return view->userAgent();
> }
>
> @@ -138,8 +140,10 @@ NPError NPN_GetValue(NPP instance, NPNVa
> {
> PluginView* view = pluginViewForInstance(instance);
>
> +#if ENABLE(NETSCAPE_PLUGIN_API)
> if (!view)
> return PluginView::getValueStatic(variable, value);
> +#endif
>
> return pluginViewForInstance(instance)->getValue(variable, value);
> }
These two changes look a bit weird to me. This file is supposed to implement
the netscape plugin API, so disabling it should probably not include the files
at all, or have empty implementations for everything? Can these checks be moved
up?
The changelog is pretty detailed, thanks for that! I think you'll want to wrap
the lines at a reasonable width, though, say, between 80 and 100 characters.
> +#if ENABLE(NETSCAPE_PLUGIN_API)
> initializeBrowserFuncs();
> +#endif
This is a bit strange, too, specially given that just a few lines bellow NPAPI
seems to be being called:
>
> #if defined(XP_UNIX)
> npErr = NP_Initialize(&m_browserFuncs, &m_pluginFuncs);
Thanks,
--
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