Closed
Bug 702614
Opened 13 years ago
Closed 13 years ago
Flash should use Skia to implement ANPCanvas and similar interfaces
Categories
(Firefox for Android Graveyard :: General, defect, P1)
Tracking
(firefox11 fixed, fennec11+)
RESOLVED
FIXED
People
(Reporter: snorp, Assigned: snorp)
Details
Attachments
(1 file, 1 obsolete file)
62.62 KB,
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
Right now we try to implement ANPCanvas/ANPPath/ANPTypeface/ANPPaint etc. with gfxContext and friends. This is a lot of work, and we have only implemented a small fraction of the interfaces. It is pretty clear that the ANP API is designed to be a thin wrapper over Skia, so it should be trivial to take the WebKit Skia-based implementation directly. My understanding is that the gfx folks are moving to Skia anyway (it's already in-tree).
Assignee | ||
Comment 1•13 years ago
|
||
Assignee | ||
Updated•13 years ago
|
Attachment #574592 -
Flags: review?(blassey.bugs)
Comment 2•13 years ago
|
||
Comment on attachment 574592 [details] [diff] [review] Use Skia to implement the Android canvas plugin APIs Review of attachment 574592 [details] [diff] [review]: ----------------------------------------------------------------- Move the apache licenced files into other-lisenced/skia-npapi and build a static library the that you can link to in dom/plugin/base/android With that change this its a rubber stamp
Attachment #574592 -
Flags: review?(blassey.bugs) → review-
Assignee | ||
Comment 3•13 years ago
|
||
Attachment #574623 -
Flags: review?(blassey.bugs)
Assignee | ||
Updated•13 years ago
|
Attachment #574592 -
Attachment is obsolete: true
Updated•13 years ago
|
Attachment #574623 -
Flags: review?(blassey.bugs) → review+
Comment 4•13 years ago
|
||
https://hg.mozilla.org/projects/birch/rev/e84fa7cf77ad
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
tracking-fennec: --- → 11+
Updated•12 years ago
|
status-firefox11:
--- → fixed
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•