If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Turn on Skia canvas on linux in 48

RESOLVED FIXED in Firefox 48

Status

()

Core
Graphics
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: jrmuizel, Assigned: lsalzman)

Tracking

(Blocks: 1 bug)

unspecified
mozilla48
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox48 fixed, relnote-firefox 48+)

Details

(Whiteboard: [gfx-noted])

Attachments

(2 attachments)

Comment hidden (empty)
(Assignee)

Updated

2 years ago
Blocks: 1141625
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.
relnote-firefox: --- → ?
(Reporter)

Updated

2 years ago
Whiteboard: [gfx-noted]
(Assignee)

Comment 2

2 years ago
Created attachment 8733430 [details] [diff] [review]
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.
Assignee: nobody → lsalzman
Status: NEW → ASSIGNED
Attachment #8733430 - Flags: review?(jmuizelaar)
(Assignee)

Comment 3

2 years ago
Created attachment 8733432 [details] [diff] [review]
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.
Attachment #8733432 - Flags: review?(jmuizelaar)
(Assignee)

Comment 4

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

Comment 5

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

Updated

2 years ago
Attachment #8733430 - Flags: review?(jmuizelaar) → review+
(Reporter)

Updated

2 years ago
Attachment #8733432 - Flags: review?(jmuizelaar) → review+

Comment 6

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

Comment 7

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

Comment 8

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a37ac3dbec9f
https://hg.mozilla.org/mozilla-central/rev/180f3f0494a7
https://hg.mozilla.org/mozilla-central/rev/826afa1625f4
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Depends on: 1259621
Depends on: 1261699
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.
Added to the release notes with "Skia enabled by default on GNU/Linux" as wording
relnote-firefox: ? → 48+
Blocks: 1186552
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...
Flags: needinfo?(milan)
"Skia, the graphic library, enabled by default on GNU/Linux for better performances" => I updated to that
"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.
Flags: needinfo?(milan)
thanks, updated!
(Assignee)

Updated

a year ago
Blocks: 1100744
(Assignee)

Updated

a year ago
Blocks: 1181006
Depends on: 1287144
You need to log in before you can comment on or make changes to this bug.