[Webkit-unassigned] [Bug 14750] [gtk] Implement plugin support in GTK backend

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=14750


mrowe at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #18239|review?                     |review-
               Flag|                            |




------- Comment #23 from mrowe at apple.com  2008-01-02 09:45 PDT -------
(From update of attachment 18239)
Sorry that this has been relatively untouched for so long.

There are a few relatively serious issues here:  PluginViewGtk has a large
amount of duplicated code with PluginViewWin, some of which could trivially be
factored out (handlePost, parseRFC822HeaderFields and friends).  It also
contains many chunks of commented-out code, unused variables
(kWebPluginViewWindowClassName, kWebPluginViewProperty) and plugin quirks which
are probably not needed on non-Windows platforms.  I'm not sure that
PluginMessageThrottlerGtk would be needed on non-Windows platforms as I believe
the Windows equivalent exists due to peculiarities with paint events and the
Windows message queue, but I suspect Anders would know more about that.  I
suspect the references to XID might need to be conditional on X11 being
available, both in PluginViewGtk.cpp and PluginViewGtk.h.  PluginViewGtk seems
to contain a bunch of references to "wndProc" still which is a Windows concept
that seems out of place in this Gtk code.

PluginStreamGtk seems to have a large amount of duplicate code with
PluginStreamWin, only the Gtk version looks to be broken in the case of
NP_ASFILE/NP_ASFILEONLY streams.  It should be possible to share the majority
of this code with Windows, and eventually share with the Mac when that code is
refactored.

With a bit more polish we should be able to get this landed soon.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.