Closed Bug 453785 Opened 16 years ago Closed 16 years ago

need a new basic sample plugin for Mac OS X

Categories

(Core Graveyard :: Plug-ins, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jaas, Assigned: jaas)

Details

Attachments

(1 file, 3 obsolete files)

20.03 KB, application/octet-stream
Details
Our current basic sample plugin for Mac OS X is hopelessly out of date. It has a codewarrior build config and won't build, it has 68k mac support, it draws with quickdraw, and in many other ways it doesn't do what we recommend that plugin developers do. We need a new basic sample plugin for Mac OS X in our NPAPI SDK.
Attached file fix v0.5 (obsolete) —
This is a zip file containing a work-in-progress version of the new basic sample plugin for Mac OS X.

The new plugin draws with CoreGraphics, uses NP_GetEntryPoints instead of main to dish out function pointers, uses a plist file instead of a resource file for plugin info, and uses a modern Xcode project file to build. It is rewritten from scratch, it does not use the existing basic plugin sample's code. This new plugin also doesn't use the shared "common" code structure that exists in our SDK. The shared "common" code makes plugin samples more confusing for everyone, much better to just have each sample build on its own.

TODO: draw the UA string centered and wrapped, audit memory mgmt, and improve comments.
Attached file fix v1.0 (obsolete) —
Attachment #337085 - Flags: superreview?(jst)
Right now the plugin has our normal tri-license, on checkin I'd like to make it BSD licensed so that anyone can use it as a starting point for a plugin.
Attached file fix v1.1 (obsolete) —
Includes BSD license and fixes a default include path.
Attachment #337001 - Attachment is obsolete: true
Attachment #337085 - Attachment is obsolete: true
Attachment #337096 - Flags: superreview?(jst)
Attachment #337085 - Flags: superreview?(jst)
Attached file fix v1.2
includes integer type fixes for new npapi header
Attachment #337096 - Attachment is obsolete: true
Attachment #338056 - Flags: superreview?(jst)
Attachment #337096 - Flags: superreview?(jst)
Attachment #338056 - Flags: superreview?(jst) → superreview+
landed on trunk
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Hey Josh,

We're you still planning on splitting something out for windows?
Yes but I am waiting for some other big changes to land. Some npapi header updates which will affect how sample plugins compile. Thanks for checking in!
QA Contact: ui → plugins
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: