Closed Bug 1271262 Opened 9 years ago Closed 9 years ago

Re-enable Graphite text shaping by default for release-channel builds

Categories

(Core :: Graphics: Text, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed
relnote-firefox --- 49+

People

(Reporter: jfkthame, Assigned: jfkthame)

Details

Attachments

(1 file)

In mozilla-45 and following trains, we temporarily preffed off graphite2 on the release channels due to the number of fuzz-bugs that were being found, some of which could have security implications. Now that the graphite2 code seems much more robust and stable -- we're on release 1.3.8 on all trains, since bug 1262846, and fuzzing is no longer finding new issues on a daily or even weekly basis -- I think it's time to revert that change and let graphite once again be preffed on for all channels.
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
cc: Tyson -- would you agree that the graphite2 fuzz-fest has settled down, such that it's OK to do this?
Flags: needinfo?(twsmith)
(In reply to Jonathan Kew (:jfkthame) from comment #2) > cc: Tyson -- would you agree that the graphite2 fuzz-fest has settled down, > such that it's OK to do this? Yes I'm confident that we have found the majority of the issues. Other than a few instances of Undefined Behavior nothing has come up over the last month of fuzzing.
Flags: needinfo?(twsmith)
Attachment #8750280 - Flags: review?(bugs) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Release Note Request (optional, but appreciated) [Why is this notable]: [Suggested wording]: Graphite2 font shaping is enabled by default. [Links (documentation, blog post, etc)]: http://graphite.sil.org/
relnote-firefox: --- → ?
49+ for release notes, added "Graphite2 font shaping is enabled by default"
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: