Closed
Bug 725119
(skia-android)
Opened 13 years ago
Closed 8 years ago
Get Azure/Skia content rendering working on Android
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla51
Tracking | Status | |
---|---|---|
firefox51 | --- | fixed |
People
(Reporter: gw280, Assigned: lsalzman)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
(deleted),
patch
|
mchang
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mchang
:
review+
|
Details | Diff | Splinter Review |
Tracking bug to track progress of Azure/Skia content rendering on Android
Reporter | ||
Updated•13 years ago
|
Reporter | ||
Comment 1•13 years ago
|
||
Current status: content rendering works; canvas rendering works (but sometimes doesn't render if both content and canvas rendering via azure are enabled). Content rendering can be done via either basic layers or tiled thebes layers with my patches.
Fonts are proving to be problematic. I have a refactor sitting around, but there's still issues with it. Sometimes glyphs get rendered at the incorrect size, and sometimes extents are calculated at the wrong size for the glyphs being rendered. There also may be issues with unnecessary loading of font data from the disk.
Reporter | ||
Comment 2•13 years ago
|
||
My git branch with my work-in-progress patches:
https://github.com/gw280/mozilla-central/commits/gwright/skia_linux
Comment 3•13 years ago
|
||
George, is this related to bug 740200, or they are completely separate?
Reporter | ||
Comment 4•13 years ago
|
||
They used to be fairly similar, but they're starting to diverge. Mainly that the gfxFont implementation on Linux is not the same as for Android, and Linux uses Thebes Layers and Android now uses Tiled Thebes Layers.
It would be fabu if the tiled thebeslayer reached consilience with non-tiled thebeslayer. If it's a worthy optimization we should do it everywhere.
Not to unnecessarily block the work here though. And the azure code working for one case but not the other would also be suspect.
Reporter | ||
Comment 6•13 years ago
|
||
I already have azure rendering working in both cases, and support for basic thebes layers is actually r+d and ready to land.
Updated•9 years ago
|
Blocks: Backendnuken
Updated•8 years ago
|
Alias: skia-android
Assignee | ||
Updated•8 years ago
|
Depends on: skia-linux
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Comment 8•8 years ago
|
||
Due to a bug/oversight, we are not actually using Skia for content at all on Android yet. Now that I figured out what was preventing Skia content from being enabled there, there are unresolved issues with reftests I need to take care of before we can go ahead with this.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Assignee | ||
Comment 9•8 years ago
|
||
This fixes the bugs that were preventing Skia from working as a content backend for Android.
Attachment #8786897 -
Flags: review?(mchang)
Assignee | ||
Comment 10•8 years ago
|
||
This is mostly small fuzz adjustments and fixing cases where the skiaContent conditions overrode preexisting fails-if/skip-if cases for Android.
Assignee: gwright → lsalzman
Status: REOPENED → ASSIGNED
Attachment #8786898 -
Flags: review?(mchang)
Updated•8 years ago
|
Attachment #8786897 -
Flags: review?(mchang) → review+
Updated•8 years ago
|
Attachment #8786898 -
Flags: review?(mchang) → review+
Comment 11•8 years ago
|
||
Pushed by lsalzman@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7083a4e7ebc0
allow Skia as a content backend in gfxPlatform. r=mchang
https://hg.mozilla.org/integration/mozilla-inbound/rev/2bb5563041e5
fuzz for reftests with Skia content on Android. r=mchang
Comment 12•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7083a4e7ebc0
https://hg.mozilla.org/mozilla-central/rev/2bb5563041e5
Status: ASSIGNED → RESOLVED
Closed: 8 years ago → 8 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Comment 13•8 years ago
|
||
bugherder |
You need to log in
before you can comment on or make changes to this bug.
Description
•