1244754 - Turn on Skia canvas on linux in 48

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Comment 1

[Milan Sreckovic [:milan] (needinfo for best results)]

Release Note Request (optional, but appreciated)
[Why is this notable]: We're changing how canvas gets rendered on Linux.
[Suggested wording]: Let's figure out the wording as it gets closer to being done.  We should have information about performance at that point as well.

Comment 2

[Lee Salzman [:lsalzman]]

Attachment: part 1 - backport zero-length gradient interval fix from Skia upstream

This is just a straight backport of a fix I already had upstreamed but was not incorporated in time before Skia upstream tagged the m49 branch. It fixes some failing reftests.

Comment 3

[Lee Salzman [:lsalzman]]

Attachment: part 2 - enable Skia canvas on Linux and fix web-platform-tests/reftests

The main effect of this patch is to pref on Skia canvas on Linux.

It removes a lot of meta-data we had for web-platform-test fails with Cairo. Since now Linux is not a special snowflake on canvas results, using Skia like everything else, a lot of "unexpected-pass"es showed up, so this removes those. Adds one actual WPT failure in Linux that matches existing failures on Mac and Windows. Another reftest that is insane and actually skipped on all platforms but Linux (drawCustomFocusRing.html) had to be fuzzed due to 4 corner pixels blending weirdly from inside an onFocus handler.

Comment 4

[Lee Salzman [:lsalzman]]

Try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=91b8128f7267

Everything looks green, ignoring the current intermittent reftest flu that is currently going around on some talos results.

Comment 5

[Lee Salzman [:lsalzman]]

(In reply to Lee Salzman [:lsalzman] from comment #4)
> Try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=91b8128f7267
> 
> Everything looks green, ignoring the current intermittent reftest flu that
> is currently going around on some talos results.

It should also be noted that the talos results are more or less unchanged. Bigger differences we might have previously noticed were addressed when we disabled xrender by default with Cairo still in place. So if nothing else, we at least don't regress and now our canvas results should be more or less consistent across all platforms.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a37ac3dbec9f
https://hg.mozilla.org/integration/mozilla-inbound/rev/180f3f0494a7

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/826afa1625f4

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a37ac3dbec9f
https://hg.mozilla.org/mozilla-central/rev/180f3f0494a7
https://hg.mozilla.org/mozilla-central/rev/826afa1625f4

Comment 9

[Milan Sreckovic [:milan] (needinfo for best results)]

We're going to let go with 48 to dev edition, but if we encounter any blocking problems, may keep it twice on dev edition.

Comment 10

[Sylvestre Ledru [:Sylvestre]]

Added to the release notes with "Skia enabled by default on GNU/Linux" as wording

Comment 11

[Sylvestre Ledru [:Sylvestre]]

Milan, can we have more information or a blog post?
For now "Skia enabled by default on GNU/Linux" is cryptic, even for geek like us...

Comment 12

[Sylvestre Ledru [:Sylvestre]]

"Skia, the graphic library, enabled by default on GNU/Linux for better performances" => I updated to that

Comment 13

[Milan Sreckovic [:milan] (needinfo for best results)]

"The use of Skia, the graphic library, enabled by default on GNU/Linux for better canvas performance."

It is just canvas, so it's worth mentioning that.

Comment 14

[Sylvestre Ledru [:Sylvestre]]

thanks, updated!