Remove (non invalidating) Core Animation Drawing model

RESOLVED INCOMPLETE

Status

()

Core
Plug-ins
RESOLVED INCOMPLETE
7 years ago
a year ago

People

(Reporter: BenWa, Unassigned)

Tracking

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
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).
(Reporter)

Comment 1

7 years ago
Created attachment 566293 [details] [diff] [review]
Remove CA

Here's a patch that simply removes support for NPAPI for testing. We can remove dead code if we're satisfied with this patch.
(Reporter)

Comment 2

7 years ago
Started a try-build to test with CoreAnimation:
https://tbpl.mozilla.org/?tree=Try&pusher=b56girard@gmail.com

Comment 4

7 years ago
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

Comment 5

a year ago
Resolving old bugs which are likely not relevant any more, since NPAPI plugins are deprecated.
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.