Summary: | [npapi] NPP_GetMIMEDescription should return const char* | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | vanuan | ||||||||
Component: | Plug-ins | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | andersca, ap, stuartmorgan, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 77200 | ||||||||||
Attachments: |
|
Description
vanuan
2012-01-29 07:41:17 PST
Created attachment 124469 [details]
patch
Can you review, please? I'm not a WebKit reviewer, so no. Comment on attachment 124469 [details]
patch
Please also fix references to this in PluginPackage{Qt,Blackberry,Efl}.cpp. They should use NP_GetMIMEDescriptionFuncPtr, not a local typedef.
Created attachment 124587 [details]
patch_v2
See updated patch. However, I'm not sure whether all mentioned ports define XP_UNIX. So build can brake. I didn't check all ports, cause I don't have its dependencies installed.
Comment on attachment 124587 [details] patch_v2 View in context: https://bugs.webkit.org/attachment.cgi?id=124587&action=review > Source/WebCore/ChangeLog:11 > + * plugins/npfunctions.h: Seems like I forgot to update the changelog Comment on attachment 124587 [details]
patch_v2
OK
Created attachment 124997 [details]
Patch to land
Updated changelog. Patch is ready to land.
Comment on attachment 124997 [details] Patch to land Clearing flags on attachment: 124997 Committed r106503: <http://trac.webkit.org/changeset/106503> All reviewed patches have been landed. Closing bug. |