Closed Bug 1037570 Opened 5 years ago Closed 5 years ago

Remove gfxPath

Categories

(Core :: Graphics, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla33

People

(Reporter: jwatt, Assigned: jwatt)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Continuing on from bug 939589, we can now get rid of gfxPath now that bug 933019 is fixed.
Attached patch patchSplinter Review
Attachment #8454578 - Flags: review?(matt.woodrow)
Can you mark as needs-landing once you review, Matt?
Attachment #8454578 - Flags: review?(matt.woodrow) → review+
Matt, Johnathan, could you provide a try link here ?
Tomcat, it's a fairly trivial dead-code removal patch, I doubt we need a try push for it.
https://hg.mozilla.org/mozilla-central/rev/665539a1bbb3
Status: NEW → RESOLVED
Closed: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
(In reply to Matt Woodrow (:mattwoodrow) from comment #4)
> Tomcat, it's a fairly trivial dead-code removal patch, I doubt we need a try
> push for it.

Per the policy announcement 2 months ago, we require them.
https://groups.google.com/d/msg/mozilla.dev.platform/9w0-Bh_3vVI/xWebYK64GdwJ

For a simple patch like this, even a basic "does it build?" sanity check would suffice.
You need to log in before you can comment on or make changes to this bug.