Closed Bug 693727 Opened 13 years ago Closed 7 years ago

Remove (non invalidating) Core Animation Drawing model

Categories

(Core Graveyard :: Plug-ins, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: BenWa, Unassigned)

Details

Attachments

(1 file)

We still allow non-invalidating CoreAnimation but the performance implication are terrible if it even runs correctly. We should remove the support for non-invalidating CoreAnimation since we already provide a InvalidatingCoreAnimation (ICA):
https://wiki.mozilla.org/NPAPI:InvalidatingCoreAnimation

Open questions:
- Will plug-ins correctly fall back to Core Graphics or ICA? Special care must be taken for 32-bit out-of-process plugins where the default drawing model is QuickDraw if nothing is selected (not supported).
Attached patch Remove CASplinter Review
Here's a patch that simply removes support for NPAPI for testing. We can remove dead code if we're satisfied with this patch.
Started a try-build to test with CoreAnimation:
https://tbpl.mozilla.org/?tree=Try&pusher=b56girard@gmail.com
Try run for edcf2ab6c1c4 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=edcf2ab6c1c4
Results (out of 1 total builds):
    success: 1
Builds available at http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/b56girard@gmail.com-edcf2ab6c1c4
Resolving old bugs which are likely not relevant any more, since NPAPI plugins are deprecated.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: