428973 - Rewrite Mac OS X default plugin for 64-bit

Status: RESOLVED FIXED

(Core Graveyard :: Plug-ins, defect)

Description

[Josh Aas]

We're going to need to rewrite our default plugin for Mac OS X for 64-bit. The following changes need to be made:
- Use the CoreGraphics NPAPI drawing model
- Remove all Quickdraw/Carbon usage, use only CoreGraphics, CoreFoundation, and Cocoa
- Get rid of resource file usage, use plist and image files instead

We'll probably only need to support Mac OS X 10.5 and up on x86 and x86-64. This work should be done for Mozilla 2.

Comment 1

[Josh Aas]

Attachment: fix v0.2

I've messed around with this before, backing up the patch here. This won't compile or run, it is just a start.

Comment 2

[Josh Aas]

When I did this I did it to figure out what problems we're going to have with NPAPI. The big one is that our current NPAPI event model on Mac OS X is based on Carbon, and that stuff doesn't exist in 64-bit Mac OS X. We need to spec out a new event model, probably offer it for negotiation on 32-bit and make it mandatory on 64-bit. That is being worked on.

Comment 3

[Josh Aas]

Attachment: fix v1.0

This is a zip file containing a new default plugin. It is *way* simpler, written using current best-practices and it doesn't include any of that old Netscape plugin finder stuff.

Comment 4

[Josh Aas]

All the new plugin does is draw a thin blue border around a white background and paint the broken-plugin image in the center of the plugin rect. Do we need to do anything else?

Comment 5

[Josh Aas]

Attachment: fix v1.1

Forgot to clean up a couple of things.

Comment 6

[Josh Aas]

Attachment: fix v1.1 (patch)

This is a patch that integrates fix v1.1 into our tree. There are changes from the "fix v1.1" zip file but they are build changes only.

This has sr+ from jst.

Comment 7

[Josh Aas]

landed on trunk

Comment 8

[Robert Kaiser]

Looks like this has compile problems on 10.4 and busted the SeaMonkey mac builder, see http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1225307530.1225310208.32734.gz#err0

Comment 9

[Josh Aas]

landed a fix, we shouldn't have been explicitly targeting the 10.5 sdk