Closed Bug 1530471 Opened 6 years ago Closed 6 years ago

Remove SkiaGL glue code

Categories

(Core :: Graphics: Canvas2D, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: lsalzman, Assigned: lsalzman)

References

Details

(Keywords: feature, Whiteboard: [gfx-noted])

Attachments

(7 files)

With bug 1468801, we have had SkiaGL disabled by default for several release and ESR cycles to test whether this is a viable decision long-term. Aside from some synthetic benchmarks, we have managed to mostly succeed at leaving this pref disabled.

The graphics team then met to decide whether to follow-through and remove support for this feature. We ultimately agreed that we can remove support for SkiaGL, with the caveat that high-end content producers should transition to WebGL where high performance is desired and better supported.

Some other use-cases might warrant some additional investigation as to bottlenecks both outside and inside the existing Skia software Canvas2D code support.

In the fulfillment of this decision, this bug will track removal of the SkiaGL glue code that was implementing support for this feature.

Depends on D21050

Depends on D21051

Depends on D21052

Depends on D21053

Depends on D21054

Depends on D21055

Blocks: 1531417
Pushed by lsalzman@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d1e480941a33 remove --disable-skia-gpu since Skia GPU is no longer used r=glandium https://hg.mozilla.org/integration/autoland/rev/9dac1bac3859 remove SkiaGL Canvas2D functionality r=jrmuizel https://hg.mozilla.org/integration/autoland/rev/4b820d7d2c13 don't build Skia GPU support r=rhunt https://hg.mozilla.org/integration/autoland/rev/62aabc4a6541 remove Moz2D/thebes SkiaGL glue r=jrmuizel https://hg.mozilla.org/integration/autoland/rev/6d0e1f99b8ac remove GLContext SkiaGL glue r=jgilbert https://hg.mozilla.org/integration/autoland/rev/d2af6ff2f10d remove layers SkiaGL glue r=jrmuizel https://hg.mozilla.org/integration/autoland/rev/484f62205c8d remove prefs for related to SkiaGL canvas r=jrmuizel
Blocks: 1126055
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: