remove OS X opengl workaround from Mozilla bug 603134

RESOLVED FIXED in Firefox 57

Status

()

Core
Widget: Cocoa
RESOLVED FIXED
2 months ago
a month ago

People

(Reporter: Josh Aas, Assigned: Josh Aas)

Tracking

Trunk
mozilla57
Points:
---

Firefox Tracking Flags

(firefox57 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

2 months ago
Created attachment 8902087 [details] [diff] [review]
fix v1.0

I think after 7 years it's worth a shot at getting rid of the workaround from bug 603134. OS X has changed a lot since then, as has Firefox.

Recommend taking this patch, keeping an eye out for problems after it lands, we can just put it right back if we see problems.
Attachment #8902087 - Flags: review?(mstange)
Comment on attachment 8902087 [details] [diff] [review]
fix v1.0

Review of attachment 8902087 [details] [diff] [review]:
-----------------------------------------------------------------

I agree, this is definitely worth trying out. Thanks!
Attachment #8902087 - Flags: review?(mstange) → review+
(Assignee)

Comment 2

2 months ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=b0eaa0211a192cc33ce0823339e4fceea32e3fbf
(Assignee)

Comment 3

2 months ago
Created attachment 8903959 [details] [diff] [review]
fix v1.0 with commit message
(Assignee)

Updated

a month ago
Keywords: checkin-needed
Attachment #8902087 - Attachment is obsolete: true

Comment 4

a month ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/10a488d8110e
Remove OS X opengl workaround from Mozilla bug 603134. r=mstange
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/10a488d8110e
Status: NEW → RESOLVED
Last Resolved: a month ago
status-firefox57: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.