Closed Bug 1403945 Opened 5 years ago Closed 5 years ago

Utility functions to recognize OS X 10.13

Categories

(Core :: Graphics, enhancement, P1)

Product:
Core
Component:
Graphics
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla58
Tracking Flags:
Tracking Status
firefox57 --- fixed
firefox58 --- fixed

People

(Reporter: milan, Assigned: milan)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

Bug 1403945: Add utility functions to recognize OS X 10.13.
59 bytes, text/x-review-board-request
mstange
: review+
Sylvestre
: approval-mozilla-beta+
Details 
Not making any decisions based on these versions, but making sure the underlying utility functions are in place.
Assignee: nobody → milan
status-firefox57: --- → affected
status-firefox58: --- → affected
Priority: -- → P1
Whiteboard: [gfx-noted]
Comment on attachment 8913221 [details]
Bug 1403945: Add utility functions to recognize OS X 10.13.

https://reviewboard.mozilla.org/r/184624/#review189812
Attachment #8913221 - Flags: review?(mstange) → review+
Pushed by msreckovic@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8063ebcb25db
Add utility functions to recognize OS X 10.13. r=mstange
https://hg.mozilla.org/mozilla-central/rev/8063ebcb25db
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox58: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Do we want to uplift that to 57?
Flags: needinfo?(milan)
Comment on attachment 8913221 [details]
Bug 1403945: Add utility functions to recognize OS X 10.13.

Approval Request Comment
This gives us flexibility in case something like bug 1400057 gets a 10.13 specific solution.  Right now, the code is unused, so it is low risk, but if we need to gate specific functionality, or add downloadable blocklist item for 10.13, we need this patch.
Flags: needinfo?(milan)
Attachment #8913221 - Flags: approval-mozilla-beta?
Comment on attachment 8913221 [details]
Bug 1403945: Add utility functions to recognize OS X 10.13.

In the (likely) case we need that, taking it. Should be in 57b5
Attachment #8913221 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.