1330473 - graphite2 cleanup

Status: RESOLVED DUPLICATE of bug 1335223

(Core :: Graphics: Text, defect)

Description

[Tom Ritter [:tjr] (OOTO until 4/30 at least)]

We have accumulated a few places where we have graphite has gotten out of sync with earlier versions or have misc. unneeded headers.

Comment 1

[Tom Ritter [:tjr] (OOTO until 4/30 at least)]

Attachment: Bug 1330473 This graphite2 include is not needed

Review commit: https://reviewboard.mozilla.org/r/104050/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/104050/

Comment 2

[Tom Ritter [:tjr] (OOTO until 4/30 at least)]

Attachment: Bug 1330473 These files were present in earlier versions of graphite, but have since been removed and are vestigial.

Review commit: https://reviewboard.mozilla.org/r/104052/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/104052/

Comment 3

[Tom Ritter [:tjr] (OOTO until 4/30 at least)]

Try Build submitted:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=44539e63c6a82ab47ede54accd908e750f9c123f

(Expecting it to pass, if not I will investigate.)

Comment 4

[Ryan VanderMeulen [:RyanVM]]

As discussed in bug 1335223, I've rolled this work into that patch. Thanks for noticing this, Tom!