RESOLVED FIXED Bug 62283
[Qt] Support for the gnash flash player plugin 
https://bugs.webkit.org/show_bug.cgi?id=62283
Summary [Qt] Support for the gnash flash player plugin
Dawit A.
Reported 2011-06-08 08:12:56 PDT
Currently the gnash flash player does not work with QtWebKit. Instead it prints out the following error message: ERROR: NPAPI ERROR: No GTK2 support in this browser! Have version 0 The fix for it is to use the same workaround as the one used for the nspluginwrapper (npwrapper) in PluginPackageQt.cpp.
Attachments
proposed patch (1.51 KB, patch)
2011-06-08 08:17 PDT, Dawit A. 		eric: review-
eric: commit-queue-
DetailsFormatted DiffDiff
proposed patch v2 (1.55 KB, patch)
2011-06-15 06:58 PDT, Dawit A. 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dawit A.
Comment 1 2011-06-08 08:17:14 PDT
Created attachment 96424 [details] proposed patch
Eric Seidel (no email)
Comment 2 2011-06-13 15:25:55 PDT
Comment on attachment 96424 [details] proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=96424&action=review Do we not have any tests in the repository which use flash? > Source/WebCore/ChangeLog:10 > + No new tests. (OOPS!) The OOPS in this line will cause your patch to fail the commit queue. You need to replace this line with a list of tests or reasons why testing is impossible/impractical.
Dawit A.
Comment 3 2011-06-15 06:58:50 PDT
Created attachment 97289 [details] proposed patch v2 Fixed changelog.
Alexis Menard (darktears)
Comment 4 2011-06-15 07:15:12 PDT
Comment on attachment 97289 [details] proposed patch v2 LGTM.
WebKit Review Bot
Comment 5 2011-06-16 15:21:05 PDT
Comment on attachment 97289 [details] proposed patch v2 Clearing flags on attachment: 97289 Committed r89076: <http://trac.webkit.org/changeset/89076>
WebKit Review Bot
Comment 6 2011-06-16 15:21:10 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.